Summary:

Fix authentication backends.

Review Request #10045 — Created June 28, 2018 and submitted July 3, 2018, 12:08 a.m.

Information
Owner:	easyb
Repository:	Review Board
Branch:	release-3.0.x
Bugs:
Depends On:
Commit:	b54e827...
Reviewers
Groups:	reviewboard
People:

Description

BaseAuthBackend.get_user() was removed with commit:
a12a370fd - Modernize the documentation for BaseAuthBackend.

The function is called by Django when the user is authenticated and

therefore required.
Also, the Active Directory backend needed fixing.

Testing Done


Created Docker container form easybe/alpine-reviewboard:wip
Successfully logged in with Active Directory credentials

Issues

Description	From	Last Updated
F821 undefined name 'get_object_or_none'	reviewbot	June 28, 2018, 11:01 p.m.
F821 undefined name 'User'	reviewbot	June 28, 2018, 11:01 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/accounts/backends/base.py (Diff revision 1)
Show all issues
```
F821 undefined name 'get_object_or_none'
```
reviewboard/accounts/backends/base.py (Diff revision 1)
Show all issues
```
F821 undefined name 'User'
```

Change Summary:

fix

Commit:

-	0f4adf823aa9fd9410f459bba1e0dfa83c6535cc
+	b54e827d05f461fae7632a05d27a3e46b36074e2

Diff:

Revision 2 (+17 -1)

Show changes

	reviewboard/accounts/backends/ad.py
	reviewboard/accounts/backends/base.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

reviewboard/accounts/backends/base.py (Diff revision 2)

Or should this be implemented in the deriving classes?

Description:

~		`BaseAuthBackend.get_user()` was accidentally removed with commit:
	~	`BaseAuthBackend.get_user()` was removed with commit:

		`a12a370fd - Modernize the documentation for BaseAuthBackend.`


		Also, the Active Directory backend needed fixing.

Description:

		`BaseAuthBackend.get_user()` was removed with commit:

		`a12a370fd - Modernize the documentation for BaseAuthBackend.`


	+	The function is called by Django when the user is authenticated and
	+	therefore required.
	+
		Also, the Active Directory backend needed fixing.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (4a35952)