mailmaps: Add an endpoint to fetch the list
ClosedPublic
Actions

Authored by vlorentz on Feb 7 2022, 2:50 PM.

Details

Reviewers

anlambert

Group Reviewers

Reviewers

Commits

rDWAPPS5640edd82b4e: mailmaps: Add an endpoint to fetch the list

Summary

This will be useful for debugging and checking records are indeed
configured.

Diff Detail

Repository

rDWAPPS Web applications

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz created this revision.Feb 7 2022, 2:50 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptFeb 7 2022, 2:50 PM

Build has FAILED

Patch application report for D7095 (id=25747)

Rebasing onto d4ca7bede2...

Current branch diff-target is up to date.

Changes applied before test

commit 98bdf12aea75bdcc2e3e96b49515112987daa164
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Feb 7 14:49:54 2022 +0100

    mailmaps: Add an endpoint to fetch the list
    
    This will be useful for debugging and checking records are indeed
    configured.

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1342/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1342/console

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 7 2022, 3:10 PM

Harbormaster failed remote builds in B26576: Diff 25747!

Build is green

Patch application report for D7095 (id=25747)

Rebasing onto d4ca7bede2...

Current branch diff-target is up to date.

Changes applied before test

commit 98bdf12aea75bdcc2e3e96b49515112987daa164
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Feb 7 14:49:54 2022 +0100

    mailmaps: Add an endpoint to fetch the list
    
    This will be useful for debugging and checking records are indeed
    configured.

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

Harbormaster completed remote builds in B26576: Diff 25747.Feb 7 2022, 5:08 PM

vlorentz requested review of this revision.Feb 7 2022, 5:08 PM

Looks good to me.

swh/web/tests/auth/test_mailmap.py
51–52	Ah right it crashes because the endpoint does not have a HTML doc declared with apidoc decorator and `check_api_get_responses` checks the `text/html` content type.. You can remove the TODO I think.