Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create davis.subdomain.conf.sample #739

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

Create davis.subdomain.conf.sample #739

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 61 additions & 0 deletions davis.subdomain.conf.sample
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
## Version 2025/01/15
# make sure that your davis container is named davis
# make sure that your dns has a cname set for davis
Copy link
Member

Choose a reason for hiding this comment

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

Need to add a note about the official container having standard and standalone versions, the standalone indeed exposes port 9000 but the standard compose seems to include nginx which maps 9000 on the host to 80 on the container, meaning that it would require set $upstream_port 80;.


server {
listen 443 ssl;
listen [::]:443 ssl;

server_name davis.*;

include /config/nginx/ssl.conf;

client_max_body_size 0;

# enable for ldap auth (requires ldap-location.conf in the location block)
#include /config/nginx/ldap-server.conf;

# enable for Authelia (requires authelia-location.conf in the location block)
#include /config/nginx/authelia-server.conf;

# enable for Authentik (requires authentik-location.conf in the location block)
#include /config/nginx/authentik-server.conf;

location / {
# enable the next two lines for http auth
#auth_basic "Restricted";
#auth_basic_user_file /config/nginx/.htpasswd;

# enable for ldap auth (requires ldap-server.conf in the server block)
#include /config/nginx/ldap-location.conf;

# enable for Authelia (requires authelia-server.conf in the server block)
#include /config/nginx/authelia-location.conf;

# enable for Authentik (requires authentik-server.conf in the server block)
#include /config/nginx/authentik-location.conf;

include /config/nginx/proxy.conf;
include /config/nginx/resolver.conf;
set $upstream_app davis;
set $upstream_port 9000;
set $upstream_proto http;
proxy_pass $upstream_proto://$upstream_app:$upstream_port;

}

# Required for autodiscovery
rewrite ^/.well-known/caldav /dav/ redirect;
Copy link
Member

Choose a reason for hiding this comment

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

Please move the rewrites above the location brackets, or put them in a location bracket the same way we have it in https://github.com/linuxserver/docker-nextcloud/blob/master/root/defaults/nginx/site-confs/default.conf.sample#L89

rewrite ^/.well-known/carddav /dav/ redirect;
charset utf-8;

location ~ (/davis)?/dav {
include /config/nginx/proxy.conf;
include /config/nginx/resolver.conf;
set $upstream_app davis;
set $upstream_port 9000;
set $upstream_proto http;
proxy_pass $upstream_proto://$upstream_app:$upstream_port;

}
}
Loading