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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] Switch to the new API key scheme for Blink wallet #2881

Closed
openoms opened this issue Nov 21, 2023 · 1 comment · Fixed by #2887
Closed

[Feature] Switch to the new API key scheme for Blink wallet #2881

openoms opened this issue Nov 21, 2023 · 1 comment · Fixed by #2887
Labels
idea A feature idea that needs to be more specification

Comments

@openoms
Copy link
Contributor

openoms commented Nov 21, 2023

Feature description

Blink has a new way to manage API keys: https://dashboard.blink.sv/

image

The header has changed to:

--header 'X-API-KEY' 'blink_...'

from the previously used:

--header "Authorization: Bearer ory_..."

The old way is only used in the mobile wallet and should not be used for Alby any more as that key can only be revoked by deleting the whole account while the API keys can be created and revoked on demand.

Describe the solution

Will update the connection instructions and correct the header used.

Describe alternatives

No response

Additional context

No response

Are you working on this?

Yes

@openoms openoms added the idea A feature idea that needs to be more specification label Nov 21, 2023
@rolznz
Copy link
Contributor

rolznz commented Nov 22, 2023

@openoms thanks for working on this.

Would you also be interested by any chance in also adding Blink wallet as a connector to Bitcoin Connect? https://github.com/getAlby/bitcoin-connect

It provides WebLN in the same way as the Alby extension but the user does not need to install an extension, so works on all browsers, including mobile and PWAs.

In case you are interested I created an issue here: getAlby/bitcoin-connect#114

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea A feature idea that needs to be more specification
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants