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

Several improvements to ts_authorize

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      This ticket is for several ts_authorize issues:

      • Make sure that the request_authorize script handles + and - in user and CSC lists properly
        • Both command line and internally
      • Convert the MochWebServer into a context manager

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          wvreeven Wouter van Reeven added a comment -

          Regarding the context manager, see https://github.com/lsst-ts/ts_authorize/pull/13#discussion_r999475120

          Show
          wvreeven Wouter van Reeven added a comment - Regarding the context manager, see https://github.com/lsst-ts/ts_authorize/pull/13#discussion_r999475120
          Hide
          Permalink
          wvreeven Wouter van Reeven added a comment -

          The MockWebServer now is as async context manager.

          The +/- support for the request_authorize script already is in place. I failed to notice that there is a command line argument for that.

          Show
          wvreeven Wouter van Reeven added a comment - The MockWebServer now is as async context manager. The +/- support for the request_authorize script already is in place. I failed to notice that there is a command line argument for that.
          Hide
          Permalink
          wvreeven Wouter van Reeven added a comment -

          Since this is a code improvement and no functionality was added or removed, I didn't update the version history.

          Show
          wvreeven Wouter van Reeven added a comment - Since this is a code improvement and no functionality was added or removed, I didn't update the version history.
          Hide
          Permalink
          wvreeven Wouter van Reeven added a comment -

          ts_authorize PR: https://github.com/lsst-ts/ts_authorize/pull/14

          Show
          wvreeven Wouter van Reeven added a comment - ts_authorize PR: https://github.com/lsst-ts/ts_authorize/pull/14
          Hide
          Permalink
          rowen Russell Owen added a comment -

          Reviewed on github

          Show
          rowen Russell Owen added a comment - Reviewed on github

            People

            Assignee:
            wvreeven Wouter van Reeven
            Reporter:
            wvreeven Wouter van Reeven
            Reviewers:
            Russell Owen
            Watchers:
            Russell Owen, Wouter van Reeven
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.