Page MenuHomeSoftware Heritage

fix cvs pserver authentication error
ClosedPublic

Authored by stsp on Oct 1 2021, 2:45 PM.

Details

Summary

Testing against cvs.savannah.gnu.org revealed that the CVS module
name should not be included in the authentication request.

Diff Detail

Repository
rDLDCVS CVS Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6388 (id=23237)

Rebasing onto 62b0f560ef...

Current branch diff-target is up to date.
Changes applied before test
commit 2b41f5205f62d0778cff55c326f256977d162b28
Author: Stefan Sperling <stsp@stsp.name>
Date:   Fri Oct 1 14:43:53 2021 +0200

    fix pserver authentication error
    
    Testing against cvs.savannah.gnu.org revealed that the CVS module
    name should not be included in the authentication request.

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

stsp requested review of this revision.Oct 1 2021, 2:47 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/loader/cvs/cvsclient.py
91

forgot to remove this

This revision is now accepted and ready to land.Oct 1 2021, 3:12 PM
This revision was automatically updated to reflect the committed changes.