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

Site Redirect appends "/" to the destination URL #60072

Open
devNigel opened this issue Jan 14, 2022 · 9 comments
Open

Site Redirect appends "/" to the destination URL #60072

devNigel opened this issue Jan 14, 2022 · 9 comments
Labels
[Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Impact] High [Pri] Normal Triaged To be used when issues have been triaged. [Type] Bug User Report This issue was created following a WordPress customer report

Comments

@devNigel
Copy link

Quick summary

When you set up a Site Redirection for a destination such as http://aozhou10btn.com/who-invented-the-internet.html , WordPress.com automatically appends "/" to the URL.

This causes 404 issues on servers that are not configured to automatically redirect to the correct URL on their end.

WordPress.com also does not allow users to prevent appending of "/" to the destination URL.

Steps to reproduce

  1. Create a Site Redirect subscription from here
  2. Enter destination URL as http://aozhou10btn.com/who-invented-the-internet.html
  3. Try visiting the custom domain, WordPress.com will send you to http://aozhou10btn.com/who-invented-the-internet.html/ instead of http://aozhou10btn.com/who-invented-the-internet.html which gives 404 error.

If you check the redirection settings, you will not find any trailing /.

image

What you expected to happen

WordPress.com should redirect to the correct URL without appending /.

What actually happened

A / gets appended to the URL.

Context

Chat: 33648197-hc
Follow up: 4685859-zen
Slack discussion: p1642128829211400-slack-C03TY6J1A

Simple, Atomic or both?

Simple, Atomic

Theme-specific issue?

No.

Browser, operating system and other notes

No response

Reproducibility

Consistent

Severity

All

Available workarounds?

No and the platform is unusable

Workaround details

No workaround apart from setting another redirection in destination servers ( which is not possible for the user in most cases) . The site redirection feature is unusable for users if they redirect to URLs like the one mentioned in this issue.

@devNigel devNigel added [Type] Bug [Pri] BLOCKER [Pri] High [Impact] High User Report This issue was created following a WordPress customer report labels Jan 14, 2022
@devNigel devNigel added this to Needs triage in HE Triaging Board (new bugs) via automation Jan 14, 2022
@kerrynicl kerrynicl added [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. Triaged To be used when issues have been triaged. labels Jan 17, 2022
@kerrynicl kerrynicl moved this from Needs triage to Triaged in HE Triaging Board (new bugs) Jan 17, 2022
@cometgrrl
Copy link
Contributor

After chatting with @delputnam, it seems this behavior is working as designed. The redirect is intended to be to a domain or directory path, not a file (as in this example, it's redirecting to a .html file).

More context here: p2MSmN-1kA-p2#comment-3401

HE Triaging Board (new bugs) automation moved this from Triaged to Closed Jan 21, 2022
@cometgrrl cometgrrl added [Pri] Normal [Closed] Won't Fix No intention to address issue. and removed [Pri] BLOCKER [Pri] High labels Jan 21, 2022
@inaikem
Copy link
Contributor

inaikem commented Jan 23, 2022

Docs update request:

1427-gh-Automattic/en.support-docs-content

@Greatdane
Copy link

@cometgrrl, @delputnam, we have recently had another example of a user not wanting to share a file directly, but a google doc. Because the URL does not end in a "/" the redirect is not working.

See; p2EDhh-1qV-p2

An example would be this text page URL
https://docs.google.com/spreadsheets/d/e/2PACX-1vT3ohhKd2BxUinxkgSln6OG_l3ln5njr1tROtGgM3OhYOhJGCO1Z8uZwcMDBcjvzvMoKD-3Y8KRu_Ut/pubhtml?gid=0&single=true

@devNigel devNigel reopened this Jan 26, 2022
HE Triaging Board (new bugs) automation moved this from Closed to Needs triage Jan 26, 2022
@devNigel
Copy link
Author

Re-opened this issue. I think we should revisit the use-case of Site Redirect subscription. It does not make sense to assume Site Redirect will be only used by users to point to a WordPress site hosted elsewhere. That comment is also almost 8 years old and WordPress.com has come a long way since then.

@davipontesblog davipontesblog removed the [Closed] Won't Fix No intention to address issue. label Jan 26, 2022
@delputnam
Copy link
Contributor

Nomado will add this to our backlog for consideration in a future sprint later in 2022. But as we have a very full roadmap, this one is likely to remain a low priority for the time being.

If there are additional reports of this issue, it would be helpful to note them here to aid in prioritization.

@cometgrrl
Copy link
Contributor

@cometgrrl, @delputnam, we have recently had another example of a user not wanting to share a file directly, but a google doc. Because the URL does not end in a "/" the redirect is not working.

See; p2EDhh-1qV-p2

An example would be this text page URL
https://docs.google.com/spreadsheets/d/e/2PACX-1vT3ohhKd2BxUinxkgSln6OG_l3ln5njr1tROtGgM3OhYOhJGCO1Z8uZwcMDBcjvzvMoKD-3Y8KRu_Ut/pubhtml?gid=0&single=true

This is very interesting. It does seem like a corner case, I wonder how many people use site redirects in this manner?

@formosattic formosattic moved this from Needs triage to Triaged in HE Triaging Board (new bugs) Feb 15, 2022
@cometgrrl cometgrrl moved this from Triaged to Prioritized - Bugs in HE Triaging Board (new bugs) Feb 15, 2022
@syds87
Copy link

syds87 commented Mar 23, 2022

I got a similar report here: 4864591-zen

In this case, the user has a site on square.site and it seems like the trailing / is causing a multiple redirect and would show a blank screen.

A temporary workaround that seems to be helpful in this case was suggesting adding a ? at the end of the Redirection URL. In this case, the redirect is working on my test site.

@mdtanjid0
Copy link

  • 6083768-zen ( needs to follow up ).

I got another report here and suggested the workaround as @syds87 suggested, which didn't work for this user.
I checked that too, and it's the same for me!

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

Support References

This comment is automatically generated. Please do not edit it.

  • 4685859-zen
  • 4864591-zen
  • 6083768-zen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Impact] High [Pri] Normal Triaged To be used when issues have been triaged. [Type] Bug User Report This issue was created following a WordPress customer report
Projects
Development

No branches or pull requests

9 participants