-
Notifications
You must be signed in to change notification settings - Fork 829
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
/oauth/token does not allow authentication via post parameters #308
Comments
We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/113274509. |
@omazhary Please add version and curl command thanks |
We are using UAA as shipped with CloudFoundry 226. The curl command is: You will need to create a grafana oauth client too, and a code. We just picked a code from the browser's network log during login. |
* Added a cloudfoundry provider (cloudfoundry.go) to interface with cloudfoundry's UAA [1] * Provide defaults for local Cloudfoundry installation on BOSH lite [2] * One workaround for UAA bug [3] in provider_default is necessary, but only applied if provider "cloudfoundry" is used [1] https://github.com/cloudfoundry/uaa [2] https://github.com/cloudfoundry/bosh-lite [3] cloudfoundry/uaa#308
Already fixed via in a later version via #258 |
great. So I understand this is fixed starting from UAA 3.0.0 i.e. CF v229 https://github.com/cloudfoundry/cf-release/releases/tag/v229 |
* UAA 3.0.0 and later (contained in Cloudfoundry 229 and later) implements the fix (cloudfoundry/uaa@8cb166e) that solves this problem, making our workaround obsolete.
@jsievers That is correct |
* Added a cloudfoundry provider (cloudfoundry.go) to interface with cloudfoundry's UAA [1] * Provide defaults for local Cloudfoundry installation on BOSH lite [2] * One workaround for UAA bug [3] in provider_default is necessary, but only applied if provider "cloudfoundry" is used [1] https://github.com/cloudfoundry/uaa [2] https://github.com/cloudfoundry/bosh-lite [3] cloudfoundry/uaa#308
* UAA 3.0.0 and later (contained in Cloudfoundry 229 and later) implements the fix (cloudfoundry/uaa@8cb166e) that solves this problem, making our workaround obsolete.
* Added a cloudfoundry provider (cloudfoundry.go) to interface with cloudfoundry's UAA [1] * Provide defaults for local Cloudfoundry installation on BOSH lite [2] * One workaround for UAA bug [3] in provider_default is necessary, but only applied if provider "cloudfoundry" is used [1] https://github.com/cloudfoundry/uaa [2] https://github.com/cloudfoundry/bosh-lite [3] cloudfoundry/uaa#308
* UAA 3.0.0 and later (contained in Cloudfoundry 229 and later) implements the fix (cloudfoundry/uaa@8cb166e) that solves this problem, making our workaround obsolete.
* Added a cloudfoundry provider (cloudfoundry.go) to interface with cloudfoundry's UAA [1] * Provide defaults for local Cloudfoundry installation on BOSH lite [2] * One workaround for UAA bug [3] in provider_default is necessary, but only applied if provider "cloudfoundry" is used [1] https://github.com/cloudfoundry/uaa [2] https://github.com/cloudfoundry/bosh-lite [3] cloudfoundry/uaa#308
* UAA 3.0.0 and later (contained in Cloudfoundry 229 and later) implements the fix (cloudfoundry/uaa@8cb166e) that solves this problem, making our workaround obsolete.
According to docs, /oauth/token should allow either BASIC auth or or client_id and client_secret can be provided as url encoded form parameters.
We tried this in the context of https://github.com/bitly/oauth2_proxy and it turns out it"s currently only possible via BASIC authentication.
here is the error we get although we do set client_id and client_secret form parameters:
It looks like the filter enforces BASIC auth and does not check for form parameters as a fallback before failing.
Unfortunately this makes integration with https://github.com/bitly/oauth2_proxy harder (we will have to introduce UAA-specific workarounds).
The text was updated successfully, but these errors were encountered: