Normalize authentication errors from SVN.

Review Request #6275 — Created Aug. 26, 2014 and submitted

Information

Review Board
release-2.0.x
8578084...

Reviewers

When failing to authenticate with a repository, the SVN backends will
return an error of "callback_get_login required." This looks like a bug
in Review Board, but is actually just an error message meaning that the
credentials were bad.

We now normalize these errors, converting them into an
authentication-related error. The SSL error handling code already did
this, but now all call sites use it.

Manually tested several of the call sites. Unfortunately, it's a bit
tricky writing unit tests for this right now.

Unit tests pass.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/pysvn.py
        reviewboard/webapi/resources/repository_info.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/pysvn.py
        reviewboard/webapi/resources/repository_info.py
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (8d8c31d)
Loading...