Crash when // is used as Link's to

[rpominov]

What version of Remix are you using?

1.12.0

Are all your remix dependencies & dev-dependencies using the same version?

• Yes

Steps to Reproduce

1. Add this code to a page <Link to="//">...</Link>
2. Open the page in the browser

Same problem with some other values of to for example "http://", so I think this should be fixed by wrapping new URL() into a try..catch rather than checking for known fail cases.

Expected Behavior

Page doesn't crash, instead <a href="//">...</a> is rendered

Actual Behavior

The page crashes with a error Failed to construct 'URL': Invalid URL

[rpominov]

To give more context:

We at @webstudio-is have a case where the to value comes from user input. We would like to just have a link that doesn't work in this case instead of entire app crushing.

[krall12]

This sounds related to a bug I had filed a few months ago #4422

Looks like they have it fixed for an upcoming version.

[github-actions]

This issue has been automatically marked stale because we haven't received a response from the original author in a while 🙈. This automation helps keep the issue tracker clean from issues that are not actionable. Please reach out if you have more information for us or you think this issue shouldn't be closed! 🙂 If you don't do so within 7 days, this issue will be automatically closed.

[brophdawg11]

This is fixed by remix-run/react-router#10367 and will be available once Remix 1.16.0 is released.

[rpominov]

Great work! Thank you!