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

LDAP Authorization cache doesn't appear to be working #116

Closed
derek63 opened this issue Mar 17, 2015 · 4 comments
Closed

LDAP Authorization cache doesn't appear to be working #116

derek63 opened this issue Mar 17, 2015 · 4 comments
Assignees
Labels

Comments

@derek63
Copy link
Member

derek63 commented Mar 17, 2015

The changes implemented in #99 were recently deployed to our dev environment, and we lost all ability to access any of the REST endpoints (403 returned from JBoss in all cases). Upon investigation, no information was being written to the access.log file, so it seemed like LDAP wasn't even being consulted for user roles. I reverted the login module code to the version prior to the above referenced PR, and was able to access the services again. So it appears that something related to authorization cache is not working correctly. I didn't debug in detail, as I was primarily focused on restoring service in dev, so @luan-cestari, can you take a look at this when you have a chance? Perhaps we can replicate the behavior in a unit test using an LDAP instance in order to get it resolved.

@luan-cestari
Copy link
Collaborator

@derek63 What about I make a PR with more log statements so we could have a better picture of the problem and i parallel I will also work creating integration tests to this module (which will take more time to finish)?

@derek63
Copy link
Member Author

derek63 commented Mar 17, 2015

Sure, more logging statements could help to isolate where the issue lies, and we can use that to troubleshoot. Let me know when you get that created and I can merge it in.

@luan-cestari
Copy link
Collaborator

Created the PR for the log. Feel free to review it & merge it -> https://github.com/lightblue-platform/lightblue-rest/pull/117/files

@luan-cestari
Copy link
Collaborator

New PR for this problem -> #119

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants