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

Fix: Add support for Base64 strings without padding (Issue #2501) #2509

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jamro
Copy link

@jamro jamro commented Dec 13, 2024

fix(isBase64): Add support for Base64 strings without padding (Issue #2501)

  • Updated isBase64 function to handle Base64 strings with missing padding.
  • Includes tests for both padded and unpadded Base64 strings.
  • Introduced a config option to toggle padding handling behaviour (ignorePadding).

Fix: #2501

Reference: Base64 RFC

3.2 Padding of Encoded Data
   In some circumstances, the use of padding ("=") in base-encoded data
   is not required or used.

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)
  • Tests written (where applicable)
  • References provided in PR (where applicable)

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.

isBase64 padding support
2 participants