Page MenuHomePhabricator

When users visit a Phame post URI with an old blog ID, canonicalize the URI instead of 404'ing
ClosedPublic

Authored by epriestley on Wed, Jul 31, 6:40 PM.

Details

Summary

Fixes T13353. If you:

  • Visit a blog post and save the URI.
  • Move the blog post to a different blog.
  • Revisit the old URI.

...we currently 404. We know what you're trying to do and should just redirect you to the new URI instead. We already do this if you visit a URI with a noncanonical slug.

Test Plan
  • Created post A.
  • Copied the live URI.
  • Moved it to a different blog.
  • Visited the saved URI from the earlier step.
  • Before: 404.
  • After: Redirect to the canonical URI.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Wed, Jul 31, 6:40 PM
epriestley requested review of this revision.Wed, Jul 31, 6:41 PM
This revision was not accepted when it landed; it landed in state Needs Review.Wed, Jul 31, 6:44 PM
This revision was automatically updated to reflect the committed changes.