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

Scope limit to the minimum necessary #84

Open
wants to merge 4 commits into
base: argonaut-dev
Choose a base branch
from
Open

Scope limit to the minimum necessary #84

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 17, 2015

The grants, scope, and period of time requested by the client should be guided by the minimum necessary rule, per HIPAA, RFC 6749, and foundation security principle.

bakerdb added 4 commits August 17, 2015 10:57
The grants, scope, and period of time requested by the client should be guided by the minimum necessary rule, per HIPAA and basic security principle.
Added required and optional parameters included in the authorization request.  Content that was here was inconsistent with the Argonaut SMART on FHIR profile.  

Via a separate pull request, I have also requested that the content regarding use of the OS's native browser also should be removed from this section.
@jmandel
Copy link
Member

jmandel commented Aug 17, 2015

I would mark these as points of discuss with @whitehatguy - e.g. leaving redirect_uri optional sounds like a reasonable choice for apps that have registered only a single URI, but then again there's a certain value to being explicit and consistent (even if slightly verbose).

In any case, this is an example of a breaking change from our current spec that needs to be highlighted and raised as a question.

@jbenhamou
Copy link
Contributor

A strict interpretation of HIPAA, RFC 6749 would demand fine-grained access control (at the data element, rather than resource level). How we reconcile SMART on FHIR with HIPAA mandates has been a source of friction at several healthcare organizations I have worked with.

We need to consider the practicality of demanding that any application owner, provide evidence of necessity for specific elements within a resource. Application owners can generally find a way to justify their needs for all elements in a resource (e.g. Patient). Still this is a very interesting question, especially as it relates to broad resource types (e.g. Observation).

Is this still an open question, are are we still working under the understanding that IaM is outside the purview of SMART?

@jmandel
Copy link
Member

jmandel commented Dec 8, 2016

This is out of scope for what we could reasonably define at the SMART specification level. One clear demonstration of this fact: the "minimum necessarily" condition applies to HIPAA authorized access, but isn't relevant for Patient API access. In other words, the decision about what is necessary in the context of a provider facing app needs to be up to the provide (or provider organization), but for patient access apps it's entirely up to the patient.

@jbenhamou
Copy link
Contributor

jbenhamou commented Dec 8, 2016 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants