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

Doesn't properly redirect #4

Open
Theelx opened this issue Jan 30, 2022 · 1 comment
Open

Doesn't properly redirect #4

Theelx opened this issue Jan 30, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@Theelx
Copy link

Theelx commented Jan 30, 2022

I'm on Brave and the 404 page for redirects simply doesn't show up for https://s.godi.se/allidoiswin. Let me know what diagnostics I need to post.

@gustavwilliam gustavwilliam added the bug Something isn't working label Jan 31, 2022
@gustavwilliam
Copy link
Owner

I noticed a similar issue on Snapchat. It works, but an error popup from snapchat is being shown before redirecting, saying that an error occurred. This isn't too good UX.

The main issue here is that Shortly currently relies on showing a 404 page in order to redirect. This is suboptimal for two reasons:

  1. It's slow
  2. Some browsers/apps, like Brave and Snapchat, have custom 404 messages, which might be difficult to bypass

I think it's worth looking into using a different method for redirecting, using CF workers/rules or something similar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants