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

% in URL param results in incorrect behaviour #7600

Closed
simonbrent opened this issue Jun 24, 2020 · 4 comments · Fixed by #8748
Closed

% in URL param results in incorrect behaviour #7600

simonbrent opened this issue Jun 24, 2020 · 4 comments · Fixed by #8748

Comments

@simonbrent
Copy link

Version

v2.12.2

Reproduction link

https://32kd4.sse.codesandbox.io/

Steps to reproduce

  • Go to https://32kd4.sse.codesandbox.io/
  • Click on link to "A working page" - see that the page displays correctly
  • Click on link to "An invalid page" - see a "This page could not be found"
  • Go back and click on link to "A broken page" - see an error page with the message "URI malformed"
  • Go back and click on link to "A differently broken page" - see an ngnix error page

What is expected ?

The three non-working links should all display the "This page could not be found" page

What is actually happening?

  • A URL param which cannot be decoded throws a URI malformed error
  • A URL param which is a single % causes a more severe error

Additional comments?

See also #4977

This bug report is available on Nuxt community (#c10841)
@ghost ghost added the cmty:bug-report label Jun 24, 2020
@stale
Copy link

stale bot commented Jul 25, 2020

Thanks for your contribution to Nuxt.js!
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you would like this issue to remain open:

  1. Verify that you can still reproduce the issue in the latest version of nuxt-edge
  2. Comment the steps to reproduce it

Issues that are labeled as pending will not be automatically marked as stale.

@stale stale bot added the stale label Jul 25, 2020
@simonbrent
Copy link
Author

This issue is still present in nuxt-edge 2.14.0-26582020.6ab65fbf

@stale stale bot removed the stale label Jul 27, 2020
@stale
Copy link

stale bot commented Aug 29, 2020

Thanks for your contribution to Nuxt.js!
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you would like this issue to remain open:

  1. Verify that you can still reproduce the issue in the latest version of nuxt-edge
  2. Comment the steps to reproduce it

Issues that are labeled as pending will not be automatically marked as stale.

@stale stale bot added the stale label Aug 29, 2020
@simonbrent
Copy link
Author

This issue is still present in nuxt-edge 2.14.5-26648675.2c600d52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants