Skip to content

SMART Implementation #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Plofstoffel opened this issue Jan 17, 2019 · 3 comments
Open

SMART Implementation #6

Plofstoffel opened this issue Jan 17, 2019 · 3 comments

Comments

@Plofstoffel
Copy link

Hi,

I have been playing around with the Dev-Sandbox and would like to confirm the following.

  • Where is the SMART implementation?
  • Is the Docker Image in this sandbox implementing an Interceptor as per the Hapi server documentation?

Sorry for creating an issue about this but I cannot get any SMART requests to work via Postman.

@vlad-ignatov
Copy link
Contributor

vlad-ignatov commented Jan 17, 2019

This is a package of several projects linked together. HAPI is used as FHIR server only. The SMART authentication layer is handled by what we call the sandbox. It is basically a NodeJS proxy server, intercepting requests to handle the authentication and then sending them upstream to the HAPI servers.

Perhaps if you share your Postman experiments I might be able to help more...

@alekline
Copy link

alekline commented Jan 19, 2019

Hi,

Thank you for fixing this @vlad-ignatov

  • I found a couple of other places where the HOST variable must be added so this works everywhere:

For the Patient Browser to work, these files need to be updated:

  • The patient-browser section of the docker-compose.yml file (line ~59)
  • smart-dev-sandbox/patient-browser/dstu2-local.tpl (line 4)
  • smart-dev-sandbox/patient-browser/dstu3-local.tpl (line 4)

vlad-ignatov added a commit that referenced this issue Jan 21, 2019
@vlad-ignatov
Copy link
Contributor

That is correct. I had do some additional changes to support the HOST variable. Please update.

kherock pushed a commit to kherock/smart-dev-sandbox that referenced this issue Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants