Skip to content
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

Added client_id OAuth 2.0 support to CDSHooks service #701

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

c-schuler
Copy link
Contributor

The OAuth 2.0 client_id header may be required to request access to patient data. This PR provides the default client_id header name along with a way to configure that to be environment-specific (like Epic-Client-ID for instance). See the CDSHooks docs for more information, especially the subject element in the fhirAuthorization request parameter.

Copy link

@alackerbauer alackerbauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good +1

Copy link

@alackerbauer alackerbauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good +1

@alackerbauer
Copy link

looks good +1

@c-schuler I approved the request but still the "review required" check pops up. I may not have the role to provide full approval (but i was able to click the approve radio button.

Copy link
Contributor

@sliver007 sliver007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@c-schuler c-schuler merged commit 37475b0 into master Apr 14, 2023
@c-schuler c-schuler deleted the cdshooks-clientId-config branch April 14, 2023 16:12
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

Successfully merging this pull request may close these issues.

3 participants