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

OAuth2 token endpoint does not allow GET method but reads query parameters #160

Closed
aeneasr opened this issue Jul 24, 2016 · 1 comment
Closed
Assignees
Labels
bug Something is not working.
Milestone

Comments

@aeneasr
Copy link
Member

aeneasr commented Jul 24, 2016

This should not work: http://localhost:4444/oauth2/token?grant_type=client_credentials&scope=core

@aeneasr aeneasr added the bug Something is not working. label Jul 24, 2016
@aeneasr aeneasr added this to the 0.2.0 milestone Jul 24, 2016
@aeneasr aeneasr self-assigned this Jul 24, 2016
@aeneasr
Copy link
Member Author

aeneasr commented Jul 24, 2016

upstream fosite

aeneasr pushed a commit that referenced this issue Jul 24, 2016
@aeneasr aeneasr closed this as completed Jul 24, 2016
aeneasr pushed a commit that referenced this issue Jul 24, 2016
aeneasr pushed a commit that referenced this issue Aug 9, 2016
* warden: rename `assertion` to `token` - closes #158
* config: do not log database credentials - closes #147
* oauth2: upgrade fosite - close #160
* config: do not store database config in hydra config - closes #164
* oauth2: id_token at_hash / c_hash is null - closes #129
* jwk: improve error message of wrong system secrect - closes #104
* readme: improve images, add benchmarks - closes #161
* cmd: improve connect dialogue - closes #170
* cmd: fix --dry option - closes #157
* firewall: document warden interface sdk
* readme: link openid connect and oauth2 introduction
* cmd: introduce FORCE_ROOT_CLIENT_CREDENTIALS env var - closes #140
* readme: document error redirect to identity provider - closes #96
* internal: fosite store must be consistent to avoid errors - closes #176
* client: add GetConcreteClient to http manager
* cmd: host process now logs basic information on all http requests - closes #178
* all: add memory profiling - closes #179
* warden: resolve nil pointer issue - closes #181
* cmd: clean up env to struct mapping, add more controls
* cmd: bcrypt cost should be configurable - closes #184
* cmd: token lifespans should be configurable - closes #183
* cmd: resolve issues with envirnoment config - closes #182
* cmd: implement tls termination capability - closes #177
* cmd: resolve issues with redirect logic and TLS
* oauth2: implement default oauth2 consent endpoint - closes #185
* warden - closes #188 
* oauth2: id token claims should be set by using id_token - closes #188
* oauth2: oauth2 implicit flow should allow custom protocols - closes #180
* oauth2: core scope should not be mandatory - closes #189
* warden: warden sdk should not make distinction between token and request - closes #190
* warden: rename authorized / allowed endpoints to something more meaningful - closes #162
* ci: improve travis config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

No branches or pull requests

1 participant