Skip to content

Don't require TOTP vault key in CE - #4317

Merged
zoldar merged 1 commit into
masterfrom
fallback-to-secret-key-base-if-no-totp-secret
Jul 15, 2024
Merged

Don't require TOTP vault key in CE#4317
zoldar merged 1 commit into
masterfrom
fallback-to-secret-key-base-if-no-totp-secret

Conversation

@ruslandoga

@ruslandoga ruslandoga commented Jul 7, 2024

Copy link
Copy Markdown
Contributor

Changes

This PR removes the requirement for TOTP_VAULT_KEY by generating it from SECRET_KEY_BASE if not set.

Tests

  • Automated tests have been added

Changelog

  • Entry has been added to changelog

Documentation

  • Docs have been updated

Dark mode

  • This PR does not change the UI

@ruslandoga
ruslandoga force-pushed the fallback-to-secret-key-base-if-no-totp-secret branch from ce957dd to 73a8f51 Compare July 7, 2024 09:46
@ruslandoga
ruslandoga marked this pull request as ready for review July 7, 2024 09:46
@ruslandoga
ruslandoga requested a review from zoldar July 7, 2024 09:46
@ruslandoga ruslandoga added the self-hosting Anything self-hosted label Jul 7, 2024
@ruslandoga
ruslandoga force-pushed the fallback-to-secret-key-base-if-no-totp-secret branch 2 times, most recently from d8ed09f to 5d78b6e Compare July 7, 2024 10:07
@ruslandoga
ruslandoga force-pushed the fallback-to-secret-key-base-if-no-totp-secret branch from 5d78b6e to c3bc9ca Compare July 7, 2024 10:32
@zoldar zoldar changed the title don't require totp vault key in ce Don't require TOTP vault key in CE Jul 7, 2024
@zoldar
zoldar merged commit 14d5a80 into master Jul 15, 2024
@zoldar
zoldar deleted the fallback-to-secret-key-base-if-no-totp-secret branch July 15, 2024 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

self-hosting Anything self-hosted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants