🔨Fix websockets connectivity

[sinclairpaul]

Proposed Changes

As per https://github.com/dani-garcia/bitwarden_rs/wiki/Enabling-WebSocket-notifications
Rocket does not support Websockets, so they must be proxied through a reverse proxy to work.

Summary of changes:

• Add nginx and standard config to image
• Move SSL to nginx
• Change Rocket port internally to 80, Websockets endpoint to 8080 (always listening)
• Remove websocket port from exposed list as it is not used (everything is proxied through 7277)
• Align Max Body Size in Nginx with config value

I have tested between the Web App, Chrome extension and Android app. Prior to changes the apps would need to be refreshed to get updated entries, after they appear automatically.

Related Issues

None

GitHub

dani-garcia/bitwarden_rs

Unofficial Bitwarden compatible server written in Rust - dani-garcia/bitwarden_rs

[addons-assistant]

💖 Thanks for opening this pull request! 💖 If your PR gets accepted and merged in, we will invite you to the project 🎉

[reharmsen]

I already have a ngix proxy with the correct configuration running in my HassOS instance with ngix proxy manager.
Ngix Proxy Manager also manages my Let's Encrypt certificates.

Can I still use this to have my certificates managed?

[sinclairpaul]

I already have a ngix proxy with the correct configuration running in my HassOS instance with ngix proxy manager.
Ngix Proxy Manager also manages my Let's Encrypt certificates.

Can I still use this to have my certificates managed?

@reharmsen Yes, but as this is not merged yet, the comments are really for review.

[reharmsen]

I already have a ngix proxy with the correct configuration running in my HassOS instance with ngix proxy manager.
Ngix Proxy Manager also manages my Let's Encrypt certificates.
Can I still use this to have my certificates managed?

@reharmsen Yes, but as this is not merged yet, the comments are really for review.

I am just worried something could break or not possible.
I guess others will better be able to judge that.

Changing this, might make the configuration a lot simpler.

[frenck]

LGTM! 👍

Thanks m8!

Happy New Year! 🍾

[addons-assistant]

Congrats on merging your first pull request! 🎉🎉🎉

[addons-assistant]

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.