Skip to content
This repository has been archived by the owner on Feb 19, 2021. It is now read-only.

Add WSGIPassAuthorization On to wsgi to support basic auth #713

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Whisprin
Copy link

@Whisprin Whisprin commented Dec 9, 2020

Basic auth is used by the paperless api. In order to use basic auth with wsgi this directive is required:

WSGIPassAuthorization On
https://www.django-rest-framework.org/api-guide/authentication/#setting-the-authentication-scheme

Basic auth is used by the paperless api. In order to use basic auth with wsgi this directive is required:

`WSGIPassAuthorization On`
https://www.django-rest-framework.org/api-guide/authentication/#setting-the-authentication-scheme
Copy link
Member

@pitkley pitkley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor nit/suggestion regarding a difference in whitespace to the surrounding lines, other than that LGTM!

@@ -342,6 +342,7 @@ and mod_wsgi, with a Paperless installation in ``/home/paperless/``:
WSGIScriptAlias / /home/paperless/paperless/src/paperless/wsgi.py
WSGIDaemonProcess example.com user=paperless group=paperless threads=5 python-path=/home/paperless/paperless/src:/home/paperless/.env/lib/python3.6/site-packages
WSGIProcessGroup example.com
WSGIPassAuthorization On
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
WSGIPassAuthorization On
WSGIPassAuthorization On

The remaining content is indented using spaces, whereas the added line uses tabs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants