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.

Sign up for GitHub

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

Jump to bottom

Pre-Authorization + Persist Authorizations... don't write anything #441

Closed
nynymike opened this issue Jan 17, 2017 · 0 comments
Closed

Pre-Authorization + Persist Authorizations... don't write anything #441

nynymike opened this issue Jan 17, 2017 · 0 comments
Assignees
@qbert2k
Labels
enhancement libs update, re-factroring, etc.
Milestone
CE 3.0.0

Comments

@nynymike
Copy link
Contributor

If a client has pre-authorization=true, there is no point to create the entry under ou=clientAuthorizations it will negatively impact performance, grow the size of the ldap database, and serve no purpose... it's only possible that someone may mis-configure this. We can mitigate in the GUI too.
@nynymike nynymike added the enhancement libs update, re-factroring, etc. label Jan 17, 2017
@nynymike nynymike added this to the CE 3.0.0 milestone Jan 17, 2017
qbert2k added a commit that referenced this issue Jan 20, 2017
@qbert2k
oxAuth #441
3ae70ef 
Pre-Authorization + Persist Authorizations... don't write anything
qbert2k added a commit that referenced this issue Jan 20, 2017
@qbert2k
Merge pull request #453 from GluuFederation/persist_authorizations
f91b37f 
oxAuth #440 #441
qbert2k added a commit that referenced this issue Jan 20, 2017
@qbert2k
oxAuth #440 #441
831e813
qbert2k added a commit that referenced this issue Jan 20, 2017
@qbert2k
oxAuth #440 #441
7419809
qbert2k added a commit that referenced this issue Jan 20, 2017
@qbert2k
Merge pull request #456 from GluuFederation/persist_authorizations
62a3a9f 
oxAuth #440 #441
@qbert2k qbert2k closed this as completed Jan 21, 2017
@qbert2k qbert2k mentioned this issue Jan 17, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qbert2k qbert2k

Labels
enhancement libs update, re-factroring, etc.
Projects
None yet
Milestone
CE 3.0.0
Development

No branches or pull requests
2 participants
@qbert2k @nynymike