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

[5.x] Add url friendly base64 en-/decoding for Glide #11299

Open
wants to merge 3 commits into
base: 5.x
Choose a base branch
from

Conversation

marcorieser
Copy link
Contributor

@marcorieser marcorieser commented Dec 22, 2024

This PR implements the two custom implementations (mentioned here) for base64_encode and base64_decode in Glide. Sometimes the internal base64_encode produced URLs with undesired slashes, which then broke the routing for Glide.

Fixes: #11176

@marcorieser marcorieser marked this pull request as ready for review December 22, 2024 15:24
@marcorieser marcorieser changed the title [5.x] Use custom url compliant base64 en-/decoding [5.x] Use custom url compliant base64 en-/decoding for Glide Dec 22, 2024
@marcorieser marcorieser changed the title [5.x] Use custom url compliant base64 en-/decoding for Glide [5.x] Add url friendly base64 en-/decoding for Glide Dec 22, 2024
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.

Images fail to load if the base64-encoded path contains multiple slashes
1 participant