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

Support InResponseTo validations in MultiSaml #350

Merged

Conversation

stavros-wb
Copy link
Contributor

Either use cache provided by user, or a default memory
cache to store InResponse parameters. This cache is not
yet partitioned per provider, which means a malicious
provider could do replay attacks by using anothers
unconsummed InResponse values

#334

@stavros-wb stavros-wb force-pushed the support_default_cache_in_multisaml branch 2 times, most recently from a012ff2 to 182c6eb Compare February 8, 2019 08:04
@stavros-wb stavros-wb changed the title Support InResponseTo validations through in MultiSaml Support InResponseTo validations in MultiSaml Feb 8, 2019
@stavros-wb
Copy link
Contributor Author

@markstos PTAL

@stavros-wb stavros-wb force-pushed the support_default_cache_in_multisaml branch 3 times, most recently from 0e3ff48 to 7045d8a Compare February 8, 2019 08:17
Either use cache provided by user, or a default memory
cache to store InResponse parameters. This cache is not
yet partitioned per provider, which means a malicious
provider could do replay attacks by using anothers
unconsummed `InResponse` values

node-saml#334
@stavros-wb stavros-wb force-pushed the support_default_cache_in_multisaml branch from 7045d8a to 5baeddf Compare February 8, 2019 08:20
@markstos markstos merged commit 2afa1ba into node-saml:master Feb 8, 2019
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

Successfully merging this pull request may close these issues.

2 participants