Modify Bitwarden client to handle missing email

[Nemirtingas]

Remove email parameter requirement and fetch email if not provided.

Closes #34

[Lujeni]

@Nemirtingas thanks for the MR

Can you update the tests pls ?

[Nemirtingas]

That should do it.

[Lujeni]

@Nemirtingas thanks for the MR

Can you update the tests pls ?

@Nemirtingas Sorry if I wasn't clear, looks like you've fixed the CI/linter thing (good btw:)). My original ask was about adding a unit test though.

[Nemirtingas]

Well, I don't know. I always get a
httpx.HTTPStatusError: Client error '429 Too Many Requests' for url 'http://127.0.0.1/identity/connect/token'

Any idea why it would send a 429 response?

[Lujeni]

Well, I don't know. I always get a httpx.HTTPStatusError: Client error '429 Too Many Requests' for url 'http://127.0.0.1/identity/connect/token'

Any idea why it would send a 429 response?

imo you moved the BitwardenAPIClient creation, so every tests generated new client (and more requests)

[Nemirtingas]

Ah, yes thanks! That was it. All green now.