Page MenuHomePhabricator
Create Task
Maniphest T324311

Redirect https://phabricator.wikimedia.org/r/ to https://gerrit.wikimedia.org/g/
Closed, DeclinedPublic
Actions

Description

We removed the outdated (cf T194145) static Gerrit code repo list content at https://phabricator.wikimedia.org/r/ in rPHEX6dce59e1313340547407c6585b97bf2482b01da6 for T228507.

So https://phabricator.wikimedia.org/r/* serves a 404 now.

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
phabricator: add test for /r/ redirect to gerritoperations/puppetproduction+7 -0
Redirect phabricator.wikimedia.org/r/ to gerrit.wikimedia.org/g/operations/puppetproduction+3 -0
phabricator: rewrite https://phabricator.wikimedia.org/r/ to gerritoperations/puppetproduction+4 -0
httpbb: add tests for phabricator, git, bugzillaoperations/puppetproduction+94 -1
Customize query in gerrit

Related Objects

Event Timeline

Aklapper created this task.Dec 2 2022, 8:31 AM
Aklapper triaged this task as Low priority.
gerritbot added a comment.Dec 2 2022, 8:32 AM

Change 863229 had a related patch set uploaded (by Aklapper; author: Aklapper):

[operations/puppet@production] Redirect phabricator.wikimedia.org/r/ to gerrit.wikimedia.org/g/

https://gerrit.wikimedia.org/r/863229

gerritbot added a project: Patch-For-Review.Dec 2 2022, 8:32 AM
LSobanski reassigned this task from Aklapper to Dzahn.Dec 5 2022, 4:53 PM
LSobanski moved this task from Incoming to Backlog on the collaboration-services board.
Mainframe98 awarded a token.Dec 10 2022, 2:21 PM
gerritbot added a comment.Dec 20 2022, 10:46 PM

Change 869853 had a related patch set uploaded (by Dzahn; author: Dzahn):

[operations/puppet@production] phabricator: rewrite https://phabricator.wikimedia.org/r/ to gerrit

https://gerrit.wikimedia.org/r/869853

Dzahn added a comment.Jan 3 2023, 10:08 PM

also see T228507

Dzahn mentioned this in T228507: Remove "Browse Gerrit Projects" on https://phabricator.wikimedia.org/r/.Jan 3 2023, 10:08 PM
Dzahn added a parent task: T228507: Remove "Browse Gerrit Projects" on https://phabricator.wikimedia.org/r/.
gerritbot added a comment.Jan 10 2023, 10:10 PM

Change 878203 had a related patch set uploaded (by Dzahn; author: Dzahn):

[operations/puppet@production] httpbb: add tests for phabricator, git, bugzilla

https://gerrit.wikimedia.org/r/878203

gerritbot added a comment.Jan 10 2023, 10:38 PM

Change 878203 merged by Dzahn:

[operations/puppet@production] httpbb: add tests for phabricator, git, bugzilla

https://gerrit.wikimedia.org/r/878203

gerritbot added a comment.Jan 11 2023, 9:15 PM

Change 879137 had a related patch set uploaded (by Dzahn; author: Dzahn):

[operations/puppet@production] phabricator: add test for /r/ redirect to gerrit

https://gerrit.wikimedia.org/r/879137

gerritbot added a comment.Jan 20 2023, 7:54 PM

Change 869853 merged by Dzahn:

[operations/puppet@production] phabricator: rewrite https://phabricator.wikimedia.org/r/ to gerrit

https://gerrit.wikimedia.org/r/869853

gerritbot added a comment.Feb 13 2023, 10:42 PM

Change 863229 abandoned by Aklapper:

[operations/puppet@production] Redirect phabricator.wikimedia.org/r/ to gerrit.wikimedia.org/g/

Reason:

Superseded by https://gerrit.wikimedia.org/r/c/operations/puppet/+/869853 and https://gerrit.wikimedia.org/r/c/operations/puppet/+/881863

https://gerrit.wikimedia.org/r/863229

Dzahn added a comment.Apr 6 2023, 7:44 PM

There is new activity on T228507. Let's see how this affects this ticket.

Dzahn removed Dzahn as the assignee of this task.Apr 6 2023, 7:48 PM
Dzahn subscribed.

watching, just wanted to make clear it's up for grabs

Aklapper moved this task from To Triage to External on the Phabricator board.Apr 25 2023, 10:11 AM
gerritbot added a comment.Jun 22 2023, 9:18 PM

Change 879137 abandoned by Dzahn:

[operations/puppet@production] phabricator: add test for /r/ redirect to gerrit

Reason:

no such rewrite rule is here yet - but we might want it later

https://gerrit.wikimedia.org/r/879137

Maintenance_bot removed a project: Patch-For-Review.Jun 22 2023, 9:31 PM
Aklapper removed a parent task: T228507: Remove "Browse Gerrit Projects" on https://phabricator.wikimedia.org/r/.Jun 2 2024, 4:24 PM
Aklapper updated the task description. (Show Details)Jun 2 2024, 4:26 PM
Pppery added a project: Wikimedia-Apache-configuration.Jun 2 2024, 4:51 PM
hashar closed this task as Declined.Jun 3 2024, 9:12 AM
hashar subscribed.

The ability to browse Gerrit project from Phabricator ( at https://phabricator.wikimedia.org/r/ ) has been removed via T228507 . I don't think we should maintain a redirect for that given the feature was barely advertised and looking at Apache logs there were only 58 hits for the last 90 days.

hashar removed a project: Wikimedia-Apache-configuration.Jun 3 2024, 9:12 AM
Aklapper added a comment.Jun 3 2024, 9:32 AM

I'm curious how you got to the number 58 (I see way more hits because random url.paths like e.g. /r/p/feed/mediawiki/extensions/CollaborationKit/).

But in any case I do agree that we should decline this! Thanks.

hashar added a comment.Jun 3 2024, 11:09 AM

Oups that is because I have used a filter url.path=/r/ :)

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits