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.

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

Incorrect documentation on reverse proxy #183

Open
lumintulogic-admin opened this issue Sep 21, 2024 · 4 comments
Open

Incorrect documentation on reverse proxy #183

lumintulogic-admin opened this issue Sep 21, 2024 · 4 comments
Labels
documentation Improvements or additions to documentation needs info

Comments

@lumintulogic-admin
Copy link

  • For Apache, it was written using ws as the protocol, it should be http:
ProxyPass /whiteboard http://localhost:3002/
RewriteEngine on
RewriteCond %{HTTP:Upgrade} websocket [NC]
RewriteCond %{HTTP:Connection} upgrade [NC]
RewriteRule ^/?whiteboard/(.*) "http://localhost:3002/$1" [P,L]
  • And for Nginx, it was missing a forward slash at the end of the proxy_pass directive, it should be:
location /whiteboard/ {
	proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
	proxy_set_header Host $host;	
	proxy_pass http://localhost:3002/;	
	proxy_http_version 1.1;
	proxy_set_header Upgrade $http_upgrade;
	proxy_set_header Connection "upgrade";
}
@kinimodmeyer
Copy link

kinimodmeyer commented Sep 22, 2024

Are you sure it should be http? With http:// instead of ws:// the admin-page can not connect to it.

@lumintulogic-admin
Copy link
Author

Are you sure it should be http? With http:// instead of ws:// the admin-page can not connect to it.

I tried mine with nginx config as written in my correction above, and the backend url used in the admin page is the nextcloud instance's URL (using https) concatenated with "/whiteboard"
Capture

@ralfi
Copy link

ralfi commented Sep 23, 2024

For Apache you can also rewrite everything if you have a sub-domain with LE Cert like this:

Auswahl_005

@juliushaertl juliushaertl added the documentation Improvements or additions to documentation label Oct 8, 2024
@juliushaertl
Copy link
Member

Are you sure it should be http? With http:// instead of ws:// the admin-page can not connect to it.

Yes, I also double checked and the config for apache and nginx work fine for me as they are stated in the readme. Can you describe what issues you experience with those?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation needs info
Projects
None yet
Development

No branches or pull requests

4 participants