Don't allow claim escalation
Some checks failed
Some checks failed
This commit is contained in:
@@ -457,6 +457,16 @@ class OidcController < ApplicationController
|
|||||||
|
|
||||||
# POST /oauth/token
|
# POST /oauth/token
|
||||||
def token
|
def token
|
||||||
|
# Reject claims parameter - per OIDC security, claims parameter is only valid
|
||||||
|
# in authorization requests, not at the token endpoint
|
||||||
|
if params[:claims].present?
|
||||||
|
render json: {
|
||||||
|
error: "invalid_request",
|
||||||
|
error_description: "claims parameter is not allowed at the token endpoint"
|
||||||
|
}, status: :bad_request
|
||||||
|
return
|
||||||
|
end
|
||||||
|
|
||||||
grant_type = params[:grant_type]
|
grant_type = params[:grant_type]
|
||||||
|
|
||||||
case grant_type
|
case grant_type
|
||||||
|
|||||||
@@ -158,7 +158,7 @@ This checklist ensures Clinch meets security, quality, and documentation standar
|
|||||||
|
|
||||||
### Performance
|
### Performance
|
||||||
- [ ] Review N+1 queries
|
- [ ] Review N+1 queries
|
||||||
- [ ] Add database indexes where needed
|
- [x] Add database indexes where needed
|
||||||
- [ ] Test with realistic data volumes
|
- [ ] Test with realistic data volumes
|
||||||
- [ ] Review token cleanup job performance
|
- [ ] Review token cleanup job performance
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user