Skip to content

Commit

Permalink
qBittorrent Reverse Proxy enabled
Browse files Browse the repository at this point in the history
  • Loading branch information
TommyE123 committed Feb 18, 2018
1 parent 68f7d73 commit e3c95d6
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 2 deletions.
1 change: 1 addition & 0 deletions qbittorrent-nox/qbittorrent-nox-constants.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,4 @@ APPSYSTEMDLOC=$SCRIPTPATH'/qbittorrent-nox/'$APPSYSTEMD
APPINITD='qbittorrent-nox'
APPINITDLOC=$SCRIPTPATH'/qbittorrent-nox/qbittorrent-nox.init'
NGINXCONFNAME='qbittorrent'
PROXYREQSUFFIX='NO'
16 changes: 14 additions & 2 deletions utils/nginx/locations-available/qbittorrent.atomic.conf
Original file line number Diff line number Diff line change
@@ -1,4 +1,16 @@
# Version=1.0
location /qbittorrent/ {
proxy_pass http://127.0.0.1:8086;
location /qbittorrent
{
rewrite ^(.*[^/])$ $1/ permanent;
}

location ~ /qbittorrent/(?<url>.*)
{
auth_basic "off";

This comment has been minimized.

Copy link
@nemchik

nemchik Feb 19, 2018

I don't think this is needed. It's not mentioned here

This comment has been minimized.

Copy link
@TommyE123

TommyE123 Feb 19, 2018

Author Owner

The first bit I got from below, as it fails the test I do to check its working without it. Do you think its ok?
qbittorrent/qBittorrent#5693

This comment has been minimized.

Copy link
@nemchik

nemchik Feb 19, 2018

It probably doesn't hurt to have it, I just wasn't sure if it was actually needed or if it messes with qb's ability to password protect its interface.

proxy_pass http://127.0.0.1:8086/$url;
proxy_hide_header Referer;
proxy_hide_header Origin;
proxy_set_header Referer '';
proxy_set_header Origin '';
add_header X-Frame-Options "SAMEORIGIN";

This comment has been minimized.

Copy link
@nemchik

nemchik Feb 19, 2018

We already set this here

This comment has been minimized.

Copy link
@TommyE123

TommyE123 Feb 19, 2018

Author Owner

I've removed the add_header X-Frame-Options "SAMEORIGIN"; do i need to remove the rest?

This comment has been minimized.

Copy link
@nemchik

nemchik Feb 19, 2018

Nope, just that. We already have it elsewhere so it shouldn't be needed here because this is a child of where we have it.

}

0 comments on commit e3c95d6

Please sign in to comment.