Making WordPress.org

Opened 7 years ago

Closed 3 years ago

Last modified 3 years ago

#2575 closed defect (bug) (fixed)

Codex: #dotorgredirect does not redirect to Rosetta sites

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: otto42's profile Otto42
Milestone: Priority: normal
Component: Codex Keywords:
Cc:

Description (last modified by SergeyBiryukov)

I'm trying to redirect an old "Forum rules" page on Codex to a new page on the forums.

The #dotorgredirect directive is used for redirecting pages to developer.w.org, so I thought this would work:

{{#dotorgredirect:https://ru.wordpress.org/support/forum-rules/}}

However, it just displays "Invalid Redirect" on the page.

Is there a whitelist in the redirect script? Could we add forums to that list?

Change History (21)

#1 @SergeyBiryukov
7 years ago

  • Description modified (diff)

#2 @samuelsidler
7 years ago

This is based on a whitelist. @Otto42 should be able to help here.

#3 @dd32
6 years ago

  • Owner set to dd32
  • Status changed from new to accepted

Just to note, the plugin only allows redirects to the following hostnames:

  • developer.wordpress.org
  • make.wordpress.org

I don't see any harm in allowing *.wordpress.org / wordpress.org myself though.
Any thoughts from anyone as to why it's as it is? @otto42?

#4 follow-up: @Otto42
6 years ago

I wrote it that way because that's the way I was told to write it. No other reason. It was originally intended only for redirection of moved documentation. It can be expanded in scope if needed.

#5 in reply to: ↑ 4 ; follow-up: @SergeyBiryukov
5 years ago

Replying to Otto42:

It was originally intended only for redirection of moved documentation. It can be expanded in scope if needed.

With the Systems request deployed, I thought a redirect to https://ru.wordpress.org/support/forum-rules/ would now work, but still getting an "Invalid Redirect" error.

Apparently Rosetta domains like *.wordpress.org are still not whitelisted, only the main domain is?

#6 in reply to: ↑ 5 ; follow-up: @dd32
5 years ago

Replying to SergeyBiryukov:

Replying to Otto42:

It was originally intended only for redirection of moved documentation. It can be expanded in scope if needed.

With the Systems request deployed, I thought a redirect to https://ru.wordpress.org/support/forum-rules/ would now work, but still getting an "Invalid Redirect" error.

Apparently Rosetta domains like *.wordpress.org are still not whitelisted, only the main domain is?

Correct. Only wordpress.org was added.

Last edited 5 years ago by dd32 (previous) (diff)

#7 @dd32
5 years ago

  • Owner changed from dd32 to Otto42
  • Status changed from accepted to assigned

#8 in reply to: ↑ 6 @SergeyBiryukov
5 years ago

Replying to dd32:

Correct. Only wordpress.org was added.

Thanks! While we're at it, could we add *.wordpress.org then? I tried looking into it myself, but apparently the plugin is not in the dotorg repo.

This ticket was mentioned in Slack in #meta by sergey. View the logs.


5 years ago

#10 @SergeyBiryukov
5 years ago

  • Summary changed from Codex: #dotorgredirect does not redirect to forums to Codex: #dotorgredirect does not redirect to Rosetta sites

#12 @Otto42
5 years ago

Okay, I added wordpress.org in deploy:8223 but forgot about *.wordpress.org. I'll make that change shortly too.

#13 follow-up: @Otto42
5 years ago

Change made to deploy revision 8740. It will accept wordpress.org and *.wordpress.org. It was also send 301's for the redirects instead of 302's.

#14 in reply to: ↑ 13 ; follow-up: @SergeyBiryukov
5 years ago

Replying to Otto42:

Change made to deploy revision 8740. It will accept wordpress.org and *.wordpress.org.

Thanks! Is it deployed yet? Still getting "Invalid Redirect" for this:

{{#dotorgredirect:https://ru.wordpress.org/support/forum-rules/}}

This ticket was mentioned in Slack in #meta by sergey. View the logs.


4 years ago

This ticket was mentioned in Slack in #forums by sergey. View the logs.


4 years ago

This ticket was mentioned in Slack in #meta by sergey. View the logs.


3 years ago

#18 in reply to: ↑ 14 ; follow-up: @dd32
3 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

Replying to SergeyBiryukov:

{{#dotorgredirect:https://ru.wordpress.org/support/forum-rules/}}

This seems to be working for me now.

#19 in reply to: ↑ 18 @SergeyBiryukov
3 years ago

Replying to dd32:

This seems to be working for me now.

Indeed, now works for me too :)

This ticket was mentioned in Slack in #docs by yui. View the logs.


3 years ago

This ticket was mentioned in Slack in #meta by sergey. View the logs.


3 years ago

Note: See TracTickets for help on using tickets.