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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/vaultwarden upgrade vaultwarden #799

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LiquidITGuy
Copy link
Contributor

First of all, thank you for your contribution! 馃槃

鈽戯笍 Self Check before Merge

  • I have tested the template using the method described in README.md thoroughly
  • I have ensured that I put as much default values as possible (except passwords) to ensure minimum effort required for end users to get started.
  • I have ensured that I am not using the "latest" tag as this tag is dynamically changing and might break the one-click app. Use a fixed version.
  • I have made sure that instructions.start and instructions.end are clear and self-explanatory.
  • Icon is added as a png file to the logos directory.

@arthuragone
Copy link

Since 1.30.0 NGIX congig has to change.

As stated in official GitHub Doc:

Note: The WebSockets service for live sync has been integrated in the main HTTP server, which means simpler proxy setups that don't require a separate rule to redirect WS traffic to port 3012. Please check the updated examples in the wiki. It's recommended to migrate to this new setup as using the old server on port 3012 is deprecated, won't receive new features and will be removed in a future release.

Maybe someone could make an update to the oneclick config? (On my side I can't make it work but neither previous version)

@adamshand
Copy link
Contributor

We already have BitwardenRS which is the old name for Vaultwarden (but the recipe is current). Perhaps we can delete this and just rename BitwardenRS to Vaultwarden?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants