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

Fix issue url query #190

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

coicoronado
Copy link

As stated in issue #188 , this resolves the need to parse correctly a given URL with query params as it is a valid option displayed in the Azure documentation for the discovery of the configuration of the issuer.

src/authorization_service_configuration.ts Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@coicoronado
Copy link
Author

Hi any update on this??

@coicoronado
Copy link
Author

Hi @tikurahul Do you have any further comments?

@TechnoChimp
Copy link

Any chance this is going to be merged in? I'm running into this same issue trying to use this package with our Okta endpoint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants