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

Accessing cache file by cni throwing KeyError on SLO #909

Open
glaand opened this issue May 30, 2023 · 0 comments
Open

Accessing cache file by cni throwing KeyError on SLO #909

glaand opened this issue May 30, 2023 · 0 comments

Comments

@glaand
Copy link

glaand commented May 30, 2023

Im trying to implement djangosaml2 into the project for the company i work for and they use SLO for logging users out. It turns out that the order of communication matther. The Idp will communicate with the SP and logging the user out but then after the user tries to refresh the browser, it is trying to access a dictionary that has been already emptied. Also the way the dictionary is been accessed is not totally correct. Should it not be wrapped wit ha try exception case? I guess if an exception is thrown just return an empty list?

Code Version

Latest Version

Expected Behavior

Returning empty list instead keyerror

Current Behavior

Keyerror

Possible Solution

reason is above

Steps to Reproduce

[

4.](

return list(self._db[cni].keys())
)

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

No branches or pull requests

1 participant