Skip to content
New issue

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

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

Already on GitHub? # to your account

As an Azure API for FHIR user I want to be able to configure the service to use any hostname for resources without using the host header #1481

Closed
dlouwers opened this issue Nov 23, 2020 · 3 comments
Labels
VSTS-Backlog On VSTS Backlog

Comments

@dlouwers
Copy link

User story
As an Azure API for FHIR user I want to be able to configure the service to use any hostname for resources without using the host header so that I can run the Azure API for FHIR SaaS behind any reverse proxy usign our own domain name.
The specific use-case that we are running into is using our own SMART on FHIR proxy and making sure that resource links are referring to our domain name that we expose. We can achieve this using the open source version by changing the host header to be whatever we want, but since the host header is being used to identify the tenant in the SaaS version this is not an option.
From what I see it would make most sense to use the X-Forwarded-Host header for this in the hope it would not break current mechanics in the SaaS version but would allow users to set their own resource locations.

Acceptance criteria

  1. When I set the X-Forwarded-Host header in a request to the FHIR API this header will be used as the hostname for any resources being stored or queried.
@CaitlinV39 CaitlinV39 added Review Tag for PM/Dev Review VSTS-Pending Prioritization We would like to add this to the backlog labels Nov 23, 2020
@CaitlinV39 CaitlinV39 added VSTS-Backlog On VSTS Backlog and removed VSTS-Pending Prioritization We would like to add this to the backlog Review Tag for PM/Dev Review labels Dec 2, 2020
@emyklebost
Copy link

Support for X-Forwarded-PathBase and X-Forwarded-Proto is also applicable to this issue.

@CaitlinV39 CaitlinV39 added this to the backlog milestone Feb 25, 2021
@CaitlinV39
Copy link
Contributor

CaitlinV39 commented Mar 10, 2021

AB#77955

@CaitlinV39
Copy link
Contributor

Closing as we deprioritized

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
VSTS-Backlog On VSTS Backlog
Projects
None yet
Development

No branches or pull requests

3 participants