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

fix(nuxt): don't encode location header #26893

Merged
merged 2 commits into from
Apr 22, 2024
Merged

fix(nuxt): don't encode location header #26893

merged 2 commits into from
Apr 22, 2024
2 −16

Conversation

huang-julien
Copy link
Member

@huang-julien huang-julien commented Apr 22, 2024 •
edited
Loading

🔗 Linked issue

fix #26882
fix #26888

📚 Description

Hi 👋 this PR reverts a change. It remove the encodeURI for the location header.

@stackblitz StackBlitz
Copy link

stackblitz bot commented Apr 22, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions github-actions bot added the 3.x label Apr 22, 2024
@huang-julien huang-julien changed the title revert(nuxt): revert(nuxt): don't encode location header Apr 22, 2024
@huang-julien huang-julien deleted the revert/encodde_location branch April 22, 2024 20:33
@huang-julien huang-julien restored the revert/encodde_location branch April 22, 2024 20:33
@huang-julien huang-julien reopened this Apr 22, 2024
@danielroe danielroe changed the title revert(nuxt): don't encode location header fix(nuxt): don't encode location header Apr 22, 2024
@danielroe danielroe merged commit 6229681 into main Apr 22, 2024
37 checks passed
@danielroe danielroe deleted the revert/encodde_location branch April 22, 2024 21:11
This was referenced Apr 22, 2024
Closed
Merged
@abovedave abovedave mentioned this pull request Apr 23, 2024
Closed
@huang-julien huang-julien mentioned this pull request May 20, 2024
Merged
@npcomplete13 npcomplete13 mentioned this pull request Jul 4, 2024
Open
@BalaKadiyala BalaKadiyala mentioned this pull request Jul 10, 2024
Open
This was referenced Jul 17, 2024
Open
Open
@bmvermeer bmvermeer mentioned this pull request Jul 27, 2024
Open
@BalaKadiyala BalaKadiyala mentioned this pull request Jul 27, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielroe danielroe danielroe approved these changes

Assignees
No one assigned
Labels
3.x
Projects
None yet
Milestone
No milestone
2 participants
@huang-julien @danielroe