Skip to content

Made sanitizeToken more strict and use correct alphabet#20

Open
JoelLinn wants to merge 1 commit intojaneczku:masterfrom
JoelLinn:fix-sanitizetoken
Open

Made sanitizeToken more strict and use correct alphabet#20
JoelLinn wants to merge 1 commit intojaneczku:masterfrom
JoelLinn:fix-sanitizetoken

Conversation

@JoelLinn
Copy link

@JoelLinn JoelLinn commented Feb 3, 2019

  • + and = are illegal in challenge tokens (url safe variant of base64 without padding)
  • instead of replacing illegal characters (gsub) only accept the token if it only contains valid characters at every position and is at least one character long
  • newline at end of file (posix standard)

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.

1 participant