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

feat: modify auth token url based on instance param #260

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Commits on Oct 8, 2024

  1. feat: set api url from jwt audience claim

    Snyk's OAuth implementation is capable of indicating the environment
    which the user is authenticated into and authorized to access.
    
    This is specified in the audience JWT claim ("aud"). Snyk's
    implementation of this claim contains an array of strings, per RFC 7519.
    
    If set and non-empty, the first audience URL is taken as the default API
    URL that the client should use, unless the endpoint was specifically
    configured.
    cmars committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    0a646dd View commit details
    Browse the repository at this point in the history
  2. chore: ignore test jwt in secret scanner

    It's a test JWT created on jwt.io for testing the parsing of claims
    and cannot be used as a valid authorization anywhere.
    cmars committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    5661185 View commit details
    Browse the repository at this point in the history

Commits on Oct 9, 2024

  1. feat: modify auth token url based on instance param

    If an instance parameter is provided in the redirect, use it to modify
    the URL from where the oauth token is obtained.
    
    The instance provided is the Snyk region domain, minus the api. host
    prefix.
    cmars committed Oct 9, 2024
    Configuration menu
    Copy the full SHA
    345bc9e View commit details
    Browse the repository at this point in the history