Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

feat!: Allow paths for fhirUser and launch claims #40

Merged
merged 2 commits into from
Jun 22, 2021
Merged

Conversation

rsmayda
Copy link
Contributor

@rsmayda rsmayda commented Jun 21, 2021

BREAKING CHANGE

Closing: #39

Description of changes:

  • allow the fhirUser claim to be nested in the access_token.
  • allow the launch context claim to be nested in the access_token.

example below is now valid if fhirUserClaimPath = "ext.fhirUser":

{
  "ext": {
    "fhirUser": "Patient/1234"
  }
}

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rsmayda rsmayda requested a review from carvantes June 21, 2021 20:57
@rsmayda rsmayda self-assigned this Jun 21, 2021
carvantes
carvantes previously approved these changes Jun 22, 2021
Copy link
Contributor

@carvantes carvantes 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. Only minor comments not related to the implementation.

package.json Outdated Show resolved Hide resolved
src/smartHandler.test.ts Outdated Show resolved Hide resolved
@rsmayda rsmayda merged commit 332806d into mainline Jun 22, 2021
@rsmayda rsmayda deleted the paths branch June 22, 2021 16:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants