Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-18667

Add support to display_firefly for obtaining and passing along an authorization token for Firefly

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: display_firefly, SUIT
    • Labels:
      None
    • Story Points:
      3
    • Epic Link:
    • Sprint:
      SUIT Sprint 2019-04, SUIT Sprint 2019-05, SUIT Sprint 2019-06, SUIT Sprint 2019-07, SUIT Sprint 2019-08
    • Team:
      Science User Interface

      Description

      Based on the API support for authentication tokens to be added to firefly_client under DM-18666, this ticket asks that display_firefly be modified to support both:

      • manually supplying an authorization token for Firefly when the display object is created, and
      • automatically picking up an appropriate token from a TBD mechanism in the Python process environment established by the Notebook Aspect (e.g., this may be from an environment variable with an agreed-upon name, or from a dot-file).

      The second subtask requires, of course, that the "TBD mechanism" has been defined. This will require work on the Notebook Aspect / Nublado as well.

      In each case, the token should then be passed down to the underlying firefly_client object.

      Care must be taken to be aware the different servers that might be used and to respect overrides that may be in place. The "second subtask", it-just-works, method is initially only required to work for whatever the active LSP-instance default Firefly server is (including any system-configured overrides to that). If the user wishes to use a different server, s/he may be required to supply an appropriate token manually.

        Attachments

          Issue Links

            Activity

            Hide
            gpdf Gregory Dubois-Felsmann added a comment -

            Raised priority to critical to match the associated firefly_client ticket DM-18666.

            Show
            gpdf Gregory Dubois-Felsmann added a comment - Raised priority to critical to match the associated firefly_client ticket DM-18666 .
            Hide
            shupe David Shupe added a comment -

            DM-18901 provides for passing the token in environment variable ACCESS_TOKEN. Passing a secret will be a later story. For now, the implementation can be to look for ACCESS_TOKEN and use it if it is provided.

            Show
            shupe David Shupe added a comment - DM-18901 provides for passing the token in environment variable ACCESS_TOKEN. Passing a secret will be a later story. For now, the implementation can be to look for ACCESS_TOKEN and use it if it is provided.
            Hide
            shupe David Shupe added a comment -

            The implementation has been tested on the lsst-lsp-int platform. The code changes are small since most of the work is done in firefly_client.

            Show
            shupe David Shupe added a comment - The implementation has been tested on the lsst-lsp-int platform. The code changes are small since most of the work is done in firefly_client .
            Hide
            shupe David Shupe added a comment -

            Reviewed on Github PR by Tatiana Goldina

            Show
            shupe David Shupe added a comment - Reviewed on Github PR by Tatiana Goldina

              People

              Assignee:
              shupe David Shupe
              Reporter:
              gpdf Gregory Dubois-Felsmann
              Reviewers:
              Gregory Dubois-Felsmann
              Watchers:
              Adam Thornton, David Shupe, Gregory Dubois-Felsmann, Xiuqin Wu [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.