Uploaded image for project: 'FHIR Specification Feedback'
  1. FHIR Specification Feedback
  2. FHIR-32186

Scope scope is unclear and needs further clarification

    XMLWordPrintableJSON

Details

    • Icon: Change Request Change Request
    • Resolution: Persuasive with Modification
    • Icon: Highest Highest
    • Bulk Data (FHIR)
    • 2.0.0
    • FHIR Infrastructure
    • STU
    • SMART Backend Services Authorization
    • 3.7
    • Hide

      We'll address these details as part of the SMART App Launch spec. Our intention is to:

      1. Update backend service examples to use SMARTv2 scopes
      2. Document the definition of "system/" scopes (without defining in them in terms of "parallel to user/ scopes" — already done in SMARTv2 ballot
      3. Explicitly document that that an app following the SMART App Launch MAY request a combination patient/, user/, and system/ scopes (to be granted with some combination of authority from the user and from the underlying client registration) – TODO during SMARTv2 ballot reconciliation. #FHIR-32251 clarifies the model for authorizing a client with "user/" scopes; we can remove that issue from the upcoming SMART App Launch block vote and extend its resolution to include these considerations.
      4. Discuss during the SMART App Launch ballot reconciliation: renaming "system/" scopes to "client/" scopes.

       

      Show
      We'll address these details as part of the SMART App Launch spec. Our intention is to: Update backend service examples to use SMARTv2 scopes Document the definition of "system/" scopes (without defining in them in terms of "parallel to user/ scopes" — already done in SMARTv2 ballot Explicitly document that that an app following the SMART App Launch MAY request a combination patient/, user/, and system/ scopes (to be granted with some combination of authority from the user and from the underlying client registration) – TODO during SMARTv2 ballot reconciliation. # FHIR-32251 clarifies the model for authorizing a client with "user/" scopes; we can remove that issue from the upcoming SMART App Launch block vote and extend its resolution to include these considerations. Discuss during the SMART App Launch ballot reconciliation: renaming "system/" scopes to "client/" scopes.  
    • Bas van den Heuvel / Dan Gottlieb: 10-0-0
    • Clarification
    • Compatible, substantive

    Description

      There are multiple unclarities in this section which makes the meaning of the scopes unclear.

      1. A back-end service based infrastructure can still have a current context. A client could potentially still access a context through FHIRcast. The client could be part of a set of systems that operate within the context of a current patient (e.g. the set of services in an operating theater). In the case, the launch scope could be used to signal patient and encounter id's. In such case, a patient scope still would have meaning. It could also be used to limit the resource access to a single patient.
      2. What the system scope means is unclear and needs further clarification.
        The section indicates that the system scopes "parallel" user scopes. This concept is a bit vague. A granted user scope for a device means the device can use the resource(s) in line with what the user is allowed to do. So read access can be granted while not all resources will be returned. The resources a device can access will be different depending on the authorizations of the user.
        Without a user, what does a system code mean? Access to the resources within the authorizations of the device? In this case a device that is not allowed to read patients, can be granted a system/Patient.read scope which will result in empty Bundles? Does it imply access to all resources? In this case subsets as defined by SmartV2 are essential. In this case the request to system/Patient.read would be denied for the case described above.
        If it is a subset of the device rights, 'device' would be better than 'system'.
      3. What will happen if system scopes are requested in a normal smart launch? Is this allowed (I suggest it would)? If so, what will it mean (see previous discussion)?
      4. The scopes are expressed in SmartV1 style, I would update them to SmartV2 style.

      Please revise the section to clarify these issues.

      Attachments

        Activity

          People

            Unassigned Unassigned
            bvdh Bas van den Heuvel
            Bas van den Heuvel
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: