Skip to content

implement NIP-42 AUTH support in the context of NIP-70 protected events #137

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fiatjaf
Copy link
Contributor

@fiatjaf fiatjaf commented Feb 19, 2025

No description provided.

@ZenenTreadwell
Copy link

https://github.com/hoytech/strfry/actions/runs/13416417248/workflow?pr=137#L37

Can you change this to v4 so the build test passes?

@fiatjaf
Copy link
Contributor Author

fiatjaf commented Mar 13, 2025

Didn't work.

@ZenenTreadwell
Copy link

ZenenTreadwell commented Mar 13, 2025

https://github.com/hoytech/strfry/actions/runs/13835507990/workflow?pr=137#L19

Try adding build-essential to the packages that get installed here

Edit: I think it needs to be installed here, actually - https://github.com/hoytech/strfry/actions/runs/13835507990/workflow?pr=137#L30

@vitorpamplona
Copy link

@fiatjaf can we extend this PR to have and check against a set of authed pubkeys as per nostr-protocol/nips#1881 ?

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