Skip to content
Snippets Groups Projects

Scitoken authentication

Merged Duncan Meacher requested to merge scitokens into master

Updated version of MR4 to handle scitokens instead of X.509 certificates. Functionality remains similar. One change is that the scitoken data is now set in the gwdatafind-server.ini file instead of at the top of authentication.py. This allows for different values to be used for testing purposes, where locally created SciTokens are used.

This has been tested using CILogon issued SciTokens, though further testing is required.

Gitlab CI unit tests have been added that repeat all the X509 view tests with the use of Scitokens. For this locally issued tokens are used, using the same setup functions as are used here: https://github.com/duncanmmacleod/igwn-auth-utils/blob/main/igwn_auth_utils/tests/conftest.py The public PEM key is then stored in a scitoken keycache, that can be accessed and used in authentication.py to deserialise the test token. One if "TEST" statements is used for the validator. A workaround needs to be developed before this can pushed to production.

Closes #3 (closed).

Edited by Duncan Meacher

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Duncan Macleod
  • Duncan Meacher added 1 commit

    added 1 commit

    • dea5cb39 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Duncan Meacher added 1 commit

    added 1 commit

    • e2531ed9 - Addressed comments for authentication.py

    Compare with previous version

  • Duncan Meacher added 1 commit

    added 1 commit

    Compare with previous version

  • Duncan Meacher added 1 commit

    added 1 commit

    • d2ca7bb8 - disable flake8 error for known F821 bug

    Compare with previous version

  • Duncan Macleod
  • Duncan Meacher added 1 commit

    added 1 commit

    Compare with previous version

  • Duncan Meacher added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading