Summary

Fix public SVN repository support with Subvertpy

Review Request #10087 — Created July 16, 2018 and submitted July 18, 2018, 4:57 p.m.

Information

Owner

easyb

Repository

Review Board

Branch

release-3.0.x

Bugs

Depends On

Commit

e0361de...

Reviewers

Groups

reviewboard

People

Description

When using Subvertpy as an SCM backend, adding a repository not
requiring authentication would cause a segfault. This is due to a call
to `subvertpy.ra.Auth.auth.get_parameter()` although the parameter was
not previously set.

Testing Done


Created Docker container form easybe/alpine-reviewboard:wip
Sucessfully added an SVN repository not requiring authentication
Sucessfully added an SVN repository requiring authentication

Issues

Description	From	Last Updated
E501 line too long (82 > 79 characters)	reviewbot	July 16, 2018, 3:01 p.m.
E501 line too long (82 > 79 characters)	reviewbot	July 16, 2018, 3:01 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/scmtools/svn/subvertpy.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (82 > 79 characters)
```
reviewboard/scmtools/svn/subvertpy.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (82 > 79 characters)
```