Skip to content

add a note about prf sorting#2

Open
Gozala wants to merge 1 commit into
mainfrom
feat/prf
Open

add a note about prf sorting#2
Gozala wants to merge 1 commit into
mainfrom
feat/prf

Conversation

@Gozala

@Gozala Gozala commented Nov 29, 2022

Copy link
Copy Markdown

Signed-off-by: Irakli Gozalishvili contact@gozala.io

Signed-off-by: Irakli Gozalishvili <contact@gozala.io>
Comment thread README.md
3. [`dag-json`](https://ipld.io/specs/codecs/dag-json/spec/) encoding MUST be used
4. The resulting JWT MUST be [base64url](https://datatracker.ietf.org/doc/html/rfc4648#section-5) encoded per [RFC 7519]
5. All segments MUST be joined with `.`s, per [RFC 7519](https://www.rfc-editor.org/rfc/rfc7519)
3. Elements in `prf`s field MUST be sorted alphabetically.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we want them alphabetically, or numerically on the underlying bits?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I guess this will be represented as a string, so UTF8 sorting makes sense 👍

@expede

expede commented Aug 14, 2023

Copy link
Copy Markdown
Member

Thanks @Gozala — all good to merge :)

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.

2 participants