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

Add support for Openstak Fernet tokens #409

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

sampsapenna
Copy link
Member

Description

This PR changes the login token syntax check to perform a rudimentary check for the token being a Fernet token. It does this by checking the magic byte and valid decoding of base64. (not much else we can do without the symmetric encryption key)

Related issues

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update (this needs a follow up PR)

Changes Made

  • Add rudimentary check's for the token being an Openstack Fernet token

Testing

  • Unit Tests
  • Integration Tests
  • Tests do not apply
  • Needs testing (start an issue or do a follow up PR about it)

Mentions

@sampsapenna sampsapenna added the enhancement New feature or request label Nov 24, 2021
@sampsapenna sampsapenna self-assigned this Nov 24, 2021
@sampsapenna sampsapenna force-pushed the feature/add-fernet-os-token-support branch from bb09e7b to 34919c6 Compare November 24, 2021 16:44
@sampsapenna sampsapenna merged commit 5db12cc into devel Nov 25, 2021
@sampsapenna sampsapenna deleted the feature/add-fernet-os-token-support branch November 25, 2021 13:17
@blankdots blankdots linked an issue Nov 29, 2021 that may be closed by this pull request
@blankdots blankdots mentioned this pull request Nov 29, 2021
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Usabilty with any Openstack environment
2 participants