Skip to content

feat: add option to log in with apikey#5

Merged
charlesthomas merged 1 commit intocharlesthomas:mainfrom
lunarys:main
Apr 22, 2025
Merged

feat: add option to log in with apikey#5
charlesthomas merged 1 commit intocharlesthomas:mainfrom
lunarys:main

Conversation

@lunarys
Copy link
Copy Markdown
Contributor

@lunarys lunarys commented Apr 13, 2025

I had some issues with being prompted to enter a verification code sent to my email when trying to log in with username and password, thus I added the option to log in via apikey. The password is still needed to unlock the vault.

Contributing back as this is the image referenced from the external-secrets example.

Using the username+password login may promt for a verification code. Use apikey authentication instead.
@charlesthomas
Copy link
Copy Markdown
Owner

hey @lunarys thanks for the PR!

i'm aware that the image is in the docs. for a while they linked to an image that didn't exist anymore, which is why i created this repo!

i've done my best to make the Dockerfile & entrypoint.sh script match the docs (with the exception of the checksum verification). as a result i'm a little hesitant to merge this because it would be another mismatch.

would you be willing to open a PR to their docs & include your changes so that they will match? then once the docs include this extra logic i could merge this. (bonus points if you include the checksum verification logic in the Dockerfile)

@echo-devnull
Copy link
Copy Markdown

The thing is though, currently the bitwarden cli cannot log in anymore with just a regular password.:

So I would not wait for the documentation to change before merging this PR.

@lunarys
Copy link
Copy Markdown
Contributor Author

lunarys commented Apr 22, 2025

Hi @charlesthomas !
The changes were merged into the external-secrets docs.

@charlesthomas charlesthomas merged commit 383e0e7 into charlesthomas:main Apr 22, 2025
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.

3 participants