Differential D25543

Replace all phurl.io short URIs with target URIs
ClosedPublic
Actions

Authored by aklapper on Feb 28 2024, 11:31.

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15746: https://phurl.io is a HTTP 503 error

Commits

rP7cee42ca2601: Replace all phurl.io short URIs with target URIs

Summary

As of February 2024, phurl.io (which is not run by Phorge.it) shows HTTP 503 errors instead of redirecting to the target URIs.

Thus replace any URIs pointing to phurl.io with the corresponding target URIs, based on the mapping listed on https://secure.phabricator.com/phurl/.

Closes T15746

Test Plan

Carefully read the diff.

Diff Detail

Repository

Branch

phurlIo503 (branched from master)

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 1078
Build 1078: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	101 ms	PhabricatorCelerityTestCase::testCelerityMaps Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:27): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
	11 ms	PhabricatorConduitTestCase::testConduitMethods 1 assertion passed.
	0 ms	PhabricatorInfrastructureTestCase::testApplicationsInstalled 1 assertion passed.
	0 ms	PhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries 1 assertion passed.
	1,472 ms	PhabricatorLibraryTestCase::testEverythingImplemented 1 assertion passed.
		View Full Test Results (1 Failed · 7 Passed)

Event Timeline

aklapper created this revision.Feb 28 2024, 11:31

Owners added a reviewer: O1: Blessed Committers.Feb 28 2024, 11:31

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptFeb 28 2024, 11:31

aklapper requested review of this revision.Feb 28 2024, 11:31

Harbormaster completed remote builds in B1078: Diff 1720.Feb 28 2024, 11:31

Clean up my local testing mess

Harbormaster completed remote builds in B1079: Diff 1721.Feb 28 2024, 11:39

Have you digged Internet Archive to fix this? :)

Thanks. Confirmed and documented a bit in T15746

This revision is now accepted and ready to land.Feb 28 2024, 12:17

Closed by commit rP7cee42ca2601: Replace all phurl.io short URIs with target URIs. · Explain WhyFeb 28 2024, 13:59

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP7cee42ca2601: Replace all phurl.io short URIs with target URIs.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

config/

check/

PhabricatorManualActivitySetupCheck.php

8 lines

O1: Blessed Committers

PhabricatorPHPPreflightSetupCheck.php

4 lines

O1: Blessed Committers

search/

controller/

PhabricatorApplicationSearchController.php

2 lines

O1: Blessed Committers

docs/

user/

configuration/

configuring_inbound_email.diviner

4 lines

O1: Blessed Committers

configuring_outbound_email.diviner

4 lines

O1: Blessed Committers

userguide/

multi_factor_auth.diviner

2 lines

O1: Blessed Committers

support/

startup/

PhabricatorStartup.php

2 lines

O1: Blessed Committers

Diff 1720

src/applications/config/check/PhabricatorManualActivitySetupCheck.php

Loading...

src/applications/config/check/PhabricatorPHPPreflightSetupCheck.php

Loading...

src/applications/search/controller/PhabricatorApplicationSearchController.php

Loading...

src/docs/user/configuration/configuring_inbound_email.diviner

Loading...

src/docs/user/configuration/configuring_outbound_email.diviner

Loading...

src/docs/user/userguide/multi_factor_auth.diviner

Loading...

support/startup/PhabricatorStartup.php

Loading...