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

Add support for base64url strings #3711

Closed
marvinruder opened this issue Aug 16, 2024 · 0 comments · Fixed by #3712
Closed

Add support for base64url strings #3711

marvinruder opened this issue Aug 16, 2024 · 0 comments · Fixed by #3712

Comments

@marvinruder
Copy link
Contributor

marvinruder commented Aug 16, 2024 •
edited
Loading

The .base64() function validates strings that contain binary information encoded in Base 64. A modification of Base 64 that can safely be used in URLs and filenames is given by the base64url standard specified in RFC 4648 § 5. As summarized here,

it uses the same algorithm as the main standard, but differs in the following:

  • Replaces “+” by “-” (minus)
  • Replaces “/” by “_” (underline)
  • Does not require a padding character
  • Forbids line separators

I propose a method .base64url() that is similar to .base64(), but instead matches against a regular expression that describes base64url-encoded values.
marvinruder added a commit to marvinruder/zod that referenced this issue Aug 16, 2024
@marvinruder
Add support for base64url strings
070c899 
Fixes colinhacks#3711

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>
@marvinruder marvinruder mentioned this issue Aug 16, 2024
Merged
marvinruder added a commit to marvinruder/zod that referenced this issue Aug 30, 2024
@marvinruder
Add support for base64url strings
93fdf5a 
Fixes colinhacks#3711

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>
colinhacks pushed a commit to marvinruder/zod that referenced this issue Dec 10, 2024
@marvinruder @colinhacks
Add support for base64url strings
1725ef4 
Fixes colinhacks#3711

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>
colinhacks added a commit that referenced this issue Dec 10, 2024
@marvinruder @colinhacks
Add support for base64url strings (#3712)
b85686a 
* Add support for `base64url` strings

Fixes #3711

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>

* Add comments

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>

* Change test structure for `base64()`, `base64url()`

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>

* Prettier

* Avoid test.each

---------

Signed-off-by: Marvin A. Ruder <signed@mruder.dev>
Co-authored-by: Colin McDonnell <colinmcd94@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for base64url strings marvinruder/zod
1 participant
@marvinruder