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

Intuit quickbooks OIDC sandbox configuration stored at a non-standard location #445

Closed
djcopley opened this issue Jan 3, 2025 · 2 comments

Comments

@djcopley
Copy link

djcopley commented Jan 3, 2025

The intuit quickbooks OIDC implementation stores the sandbox testing configuration at a nonstandard url: https://developer.api.intuit.com/.well-known/openid_sandbox_configuration

As a result, the provider urls are incorrect when trying to use the sandbox environment.

@ericchiang
Copy link
Collaborator

Hey @djcopley,

Thanks for opening the issue. The spec is pretty clear around what the well-known URL has to be. I would open an issue with intuit. If you need to integrate with go-oidc, please parse the JSON document yourself and use the following API:

https://pkg.go.dev/github.com/coreos/go-oidc/v3/oidc#ProviderConfig.NewProvider

@djcopley
Copy link
Author

djcopley commented Jan 3, 2025

Thanks @ericchiang!

This is exactly what I was looking for. Not sure how I missed it...

@djcopley djcopley closed this as completed Jan 3, 2025
ericchiang added a commit to ericchiang/go-oidc that referenced this issue Jan 4, 2025
This PR adds JSON tags to allow parsing a ProviderConfig directly from
the OpenID Connect JSON metadata document. Since this is the preferred
workaround for providers that don't support discovery in a
spec-compliant way, such as returning the wrong issuer, or requiring a
URL parameter, make this path easier and add an example to the godoc.

Updates coreos#445
Updates coreos#444
Updates coreos#439
Updates coreos#442
Updates coreos#344
Fixes coreos#290
ericchiang added a commit to ericchiang/go-oidc that referenced this issue Jan 4, 2025
This PR adds JSON tags to allow parsing a ProviderConfig directly from
the OpenID Connect JSON metadata document. Since this is the preferred
workaround for providers that don't support discovery in a
spec-compliant way, such as returning the wrong issuer, or requiring a
URL parameter, make this path easier and add an example to the godoc.

Updates coreos#445
Updates coreos#444
Updates coreos#439
Updates coreos#442
Updates coreos#344
Fixes coreos#290
ericchiang added a commit to ericchiang/go-oidc that referenced this issue Jan 4, 2025
This PR adds JSON tags to allow parsing a ProviderConfig directly from
the OpenID Connect JSON metadata document. Since this is the preferred
workaround for providers that don't support discovery in a
spec-compliant way, such as returning the wrong issuer, or requiring a
URL parameter, make this path easier and add an example to the godoc.

Updates coreos#445
Updates coreos#444
Updates coreos#439
Updates coreos#442
Updates coreos#344
Fixes coreos#290
ericchiang added a commit that referenced this issue Jan 4, 2025
This PR adds JSON tags to allow parsing a ProviderConfig directly from
the OpenID Connect JSON metadata document. Since this is the preferred
workaround for providers that don't support discovery in a
spec-compliant way, such as returning the wrong issuer, or requiring a
URL parameter, make this path easier and add an example to the godoc.

Updates #445
Updates #444
Updates #439
Updates #442
Updates #344
Fixes #290
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

No branches or pull requests

2 participants