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.

Sign up for GitHub

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

Jump to bottom

Increase the RelayState maximum length on Stub IdP #1207

Open
stukalin opened this issue Jun 10, 2020 · 1 comment
Open

Increase the RelayState maximum length on Stub IdP #1207

stukalin opened this issue Jun 10, 2020 · 1 comment
Labels
enhancement

Comments

@stukalin
Copy link

stukalin commented Jun 10, 2020 •
edited

Currently the max length of the RelayState is set to 80 symbols. I suggest increasing it to, say, 1024 in order to support longer URLs.

image

If you wish I can create a small PR for that.
@AndersAbel
Copy link
Member

That should be fixed (among a bunch of other stuff with the StubIdp). The thing is that the public version at https://stubidp.sustainsys.com is currently frozen and not accepting new deploys. This is to be able to rework it in the master branch for v3 without destroying the public version.
So a PR would be accepted against master. But, to get it deployed to https://stubidp.sustainsys.com is a bit more complicated.

@stukalin stukalin mentioned this issue Jun 11, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Increase relay state max length for stup idp stukalin/Saml2
2 participants
@AndersAbel @stukalin