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

Token exchange request warning when SMART scopes are not combined (Test 1.4.06) #538

Open
Justin-Ramm opened this issue Aug 1, 2024 · 1 comment

Comments

@Justin-Ramm
Copy link

Inferno currently gives a warning if SMART v2.0.0 scopes are returned separately instead of in a combined single scope string. For example, if the token endpoint returns a scope string that includes patient/AllergyIntolerance.r patient/AllergyIntolerance.s, Inferno will have a warning that says:

Token exchange response did not include all requested scopes. These may have been denied by user: patient/AllergyIntolerance.rs

The SMART spec does "recommend" combining scopes into the shortest string possible, but does not make it a strict requirement (in the Scope Equivalence section).

While this is only a warning, I am curious if Inferno should accept separate scope strings as valid without a warning. Perhaps the warning makes sense, but wanted to raise the issue in case this wasn't intended.

@Justin-Ramm
Copy link
Author

The following is a full list of tests where I have observed this warning due to the issue noted above, when using (g)(10) test kit options US Core 6.1.0 / USCDI v3, SMART App Launch 2.0.0, Bulk Data 1.0.1:

Test section Affected test steps
1 Standalone Patient App 1.4.06
3 EHR Practitioner App 3.4.08
9.2 Public Client Launch 9.2.06
9.9 EHR Launch with Patient Scopes 9.9.08
9.11 Token Introspection 9.11.1.2.06
9.12 Asymmetric Client Launch 9.12.2.06
9.13 Launch with v1 Scopes 9.13.2.06
9.14 SMART Launch with Fine-Grained Scopes 9.14.1.1.2.06
9.14.2.1.2.06

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

1 participant