-
Notifications
You must be signed in to change notification settings - Fork 988
Updating documentation for custom claim headers #918
Conversation
|
Build status: failure (Full Log) |
…ink with valid current url to owasp
|
Build status: warning (Full Log) |
|
This is to be merged with the pull request |
|
@abstractj didn't we move the docs? Or plan to? If so docs PRs should be sent to Louketo repo right? |
| https://account.live.com/developers/applications/create | ||
| https://developer.twitter.com/apps/ | ||
| https://developer.twitter.com/apps/ | ||
| https://docs.jboss.org/author/display/WFLY10/Admin+Guide* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to fix these URLs rather than ignore them
| * Secure token and logout endpoint were included in Gatekeeper. Thanks to https://github.com/fredbi[fredbi] | ||
| * There was a bug on Gatekeeper which was making cookies to be applied to subdomains. Thanks to https://github.com/daniel-ac-martin[daniel-ac-martin] the issue was fixed | ||
| * Now Gatekeeper provides support to https://www.owasp.org/index.php/SameSite[Same-site cookies]. Thanks to https://github.com/fiji-flo[fiji-flo] | ||
| * Now Gatekeeper provides support to https://cheatsheetseries.owasp.org/cheatsheets/Cross-Site_Request_Forgery_Prevention_Cheat_Sheet.html#samesite-cookie-attribute[Same-site cookies]. Thanks to https://github.com/fiji-flo[fiji-flo] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should just remove this link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because we dont support the feature any more ? samesite cookie attribute ?
|
Build status: warning (Full Log) |
@stianst yes, we moved the docs to Louketo. I just wonder if we should update the Keycloak documentation, by removing the Gatekeeper section and pointing people to Louketo. Wdyt? |
|
@allupaku thanks for contributing to this. Although, Louketo reache it's EOL and we no longer support it. For this reason, I'm closing the issues here. |
This is for updating changes made via pull request louketo/louketo-proxy#532