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

KEYCLOAK-4222 Remove slash from state parameter #3760

Merged
merged 1 commit into from Jan 20, 2017

Conversation

sldab
Copy link
Contributor

@sldab sldab commented Jan 13, 2017

This was causing us lots of trouble. See https://issues.jboss.org/browse/KEYCLOAK-4222 for details.

@stianst
Copy link
Contributor

stianst commented Jan 18, 2017

I don't understand why the counter is there in the first place. If that can be removed there's no need to have a delimiter at all.

@sldab sldab force-pushed the state-param-no-slash branch 4 times, most recently from 7cbf400 to 7a8add6 Compare January 18, 2017 21:33
@stianst stianst merged commit 1373708 into keycloak:master Jan 20, 2017
123445666 pushed a commit to 123445666/keycloak that referenced this pull request Mar 10, 2023
Bumps [cypress](https://github.com/cypress-io/cypress) from 10.11.0 to 11.0.1.
- [Release notes](https://github.com/cypress-io/cypress/releases)
- [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md)
- [Commits](cypress-io/cypress@v10.11.0...v11.0.1)

---
updated-dependencies:
- dependency-name: cypress
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

None yet

2 participants