-
Notifications
You must be signed in to change notification settings - Fork 81
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
HTTPS support #118
Comments
Hello and thanks for caring about Polaris! I agree this is a very desirable feature. I looked into it a couple years ago and could not find a solution that satisfied all my requirements:
I will look into this again at some point. In the mean time, I know some users work around this by deploying Polaris behind a |
I realise this is a little late, but I was recently looing to do something similar and thought I'd share my experience. For just a regular reverse proxy
For SSL
Restart apache and it should be ready to go. |
Oh, right, let me contribute too. If you're using Nginx, you can use this as a starter config:
See https://letsencrypt.org/ for obtaining a certificate. |
One solution to offer this in Polaris would be to use a crate like https://crates.io/crates/acme-micro. Personally, I wouldn't use it though, since the reverse proxy solution works better for me. @stukkat you can use fenced code blocks for code snippets. |
Thanks for that @lnicola, please forgive my github noobishness :P |
This is exactly what I would want to bundle in Polaris, if it wasn't for the OpenSSL dependency. I had that library as a dependency in the early days of Polaris and it was a constant headache for building and distributing on Windows. |
SOZUuse https://github.com/sozu-proxy/sozu-acme for configuring Let's Encrypt
Create systemd service
Enable and start the service
HAPROXY
Enable service
|
Hey @stukkat ! Sorry for the bump, but I'm running apache as well, but it has nextcloud listening to 80 and 443. ProxyPass /polaris http://localhost:5050/ <Location /polaris> But this opens a black site for me, nothing is displayed. Any pointers is appreciated! |
For anyone having trouble getting ssl to work using apache (reverse proxy)I've struggled with that for a bit myself. I basically had the same directives @stukkat used and added
|
I'm new to polaris. I'm running it in a docker container and trying to get it working behind nginx. With the most basic of nginx configs, its appears to work but tracks will randomly be cut short. When this happens, the browser (firefox) shows the error NS_ERROR_NET_PARTIAL_TRANSFER. I'm using the latest nginx, firefox and polaris (from the git hub repo). I do not appear to have the same issue if I connect directly to the polaris service on port 5050, bypassing the nginx proxy. I tried using the settings from the suggested config from @lnicola but have the same issue. I've tried many variants of similar settings but none resolve the issue. Is there any tried and tested nginx config example? |
I suggest creating a new issue for this, since this appears to be either due to your specific nginx configuration, or the Firefox browser itself. Before creating a new issue, try chrome, and try Firefox, and try one of the Polaris apps. Mention the platforms on each that were tested (Windows, Linux, Android, iOS). |
Also, for what its worth, I don't believe polaris should worry about acme integration. If polaris should do anything, it should just create a self-signed certificate (Rustls) in some ./ssl directory and use that. Anyone who is reverse-proxying who needs SSL can ignore the cert check, anyone not reverse proxying can ignore the "untrusted cert", and if they care they can get their own certs and use those instead. |
Thanks @Lyamc. I agree that Polaris does not need to so anything itself for ssl. I was just hoping someone would have a known working nginx config, ssl or not. I'm trying in another browser for now. I'll give the Android app a good test too. |
Since polaris exposes a login form, it would be desirable if the application had support for HTTPS, so that credentials aren't sent unencrypted.
The text was updated successfully, but these errors were encountered: