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

NIP-11 for publishing server metadata #51

Merged
merged 3 commits into from
Jan 14, 2022

Conversation

scsibug
Copy link
Contributor

@scsibug scsibug commented Jan 3, 2022

This NIP proposes a way of publishing server metadata.

closes #50

nips/11.md Outdated
}]
```

Additional fields may be present, but they MUST be prefixed with `x-` to indicate they are not standard.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm strongly against the x- stuff. If people want to return extra fields they should be able to. And if others want to rely on these fields then they are a standard -- even if they're not covered on these documents.

Also software implementations must know reasonably well what they're supposed to return and what they must look for.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, I can remove the x- suggestion.

nips/11.md Show resolved Hide resolved
nips/11.md Show resolved Hide resolved
nips/11.md Outdated

### Supported NIPs ###

As the Nostr protocol evolves, some functionality may only be available by relays that implement a specific `NIP`. This field is an array of the string identifiers of `NIP`s that are implemented in the relay. The string should match the title of the `NIP` document. Examples would include `"NIP-01"` and `"NIP-09"`. Client-side `NIPs` SHOULD not be advertised, and can be ignored by clients.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if we made this a list of integers instead of strings? It's less prone to typo errors.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No strong preference from me here. It's all going to be machine-read anyways, so we can take the position that all NIPs will be integers. The only thing to consider is we have a lot more flexibility with strings, but people can make up numbers just as easily.

@fiatjaf
Copy link
Member

fiatjaf commented Jan 4, 2022

Looks good! Thank you!

@scsibug scsibug changed the title New NIP for publishing server metadata NIP-11 for publishing server metadata Jan 4, 2022
@fiatjaf fiatjaf merged commit bdc11b6 into nostr-protocol:master Jan 14, 2022
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.

Feature Req: Relay identification command
2 participants