Page MenuHomeSoftware Heritage

Fix for snapshot content redirection
ClosedPublic

Authored by jayeshv on Nov 26 2021, 2:49 PM.

Details

Summary

Snapshot content route is redirected with the query param snapshot_id instead of snapshot.
Now considering snapshot_id param and snapshot ar /content route. (snapshot takes priority)

Diff Detail

Repository
rDWAPPS Web applications
Branch
redirect-fix
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25191
Build 39359: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 39358: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D6694 (id=24317)

Rebasing onto 20a9d18559...

Current branch diff-target is up to date.
Changes applied before test
commit 806623eb53f944efde4cfeac30f433aa73b3adb3
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Fri Nov 26 14:45:09 2021 +0100

    Fix for snapshot content redirection
    
    Snapshot content route is redirected with the query param snapshot_id.
    Considering snapshot_id param along with snapshot. (snapshot takes priority)

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1246/ for more details.

This revision is now accepted and ready to land.Nov 26 2021, 3:10 PM
This revision was automatically updated to reflect the committed changes.