Staging instance, all changes can be removed at any time

Skip to content

browse: Disambiguate URLs related to origin / snapshot context

Antoine Lambert requested to merge generated-differential-D3046-source into generated-differential-D3046-target

Still on the road to #2342 (closed), I thought removing ambiguities in current browse URL scheme was a prerequisite to ensure all archive objects can be correctly browsed and referenced with updated SWHIDs.

So disambiguate and/or deprecate the following browse URLs:

  • GET /browse/origin/(origin_url)/content/(path)/

  • GET /browse/origin/(origin_url)/directory/(path)/

  • GET /browse/origin/(origin_url)/visit/(timestamp)/branches/

  • GET /browse/origin/(origin_url)/visit/(timestamp)/content/(path)/

  • GET /browse/origin/(origin_url)/visit/(timestamp)/directory/(path)/

  • GET /browse/origin/(origin_url)/visit/(timestamp)/log/

  • GET /browse/origin/(origin_url)/visit/(timestamp)/releases/

  • GET /browse/snapshot/(snapshot_id)/content/(path)/

  • GET /browse/snapshot/(snapshot_id)/directory/(path)/

Replace their use by providing timestamp and path as query parameters to these endpoints for consistency:

  • GET /browse/origin/(origin_url)/content/

  • GET /browse/origin/(origin_url)/directory/

  • GET /browse/origin/(origin_url)/log/

  • GET /browse/origin/(origin_url)/branches/

  • GET /browse/origin/(origin_url)/releases/

  • GET /browse/snapshot/(snapshot_id)/content/

  • GET /browse/snapshot/(snapshot_id)/directory/

Endpoints documentation have also been updated to reflect these changes.

Sorry for the huge diff but I did not have a choice to process all URLs in one batch.

Closes #2115 (closed) Closes #2135 (closed)

Migrated from D3046 (view on Phabricator)

Merge request reports