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

Handle Subroutines ABORT #246

Open
BasileiosKal opened this issue Feb 21, 2023 · 5 comments
Open

Handle Subroutines ABORT #246

BasileiosKal opened this issue Feb 21, 2023 · 5 comments
Labels
core Associated to the core spec editorial ready-for-pr
Milestone

Comments

@BasileiosKal
Copy link
Contributor

2 sugestions for the Ciphersuite format:

A. Points serialization:

The point serialization we use is curve agnostic. No reason to be defined in the Ciphersuite. We could simplify things and define them as global parameters.

B. expand_len definition:

After #243 is merged, we can define expand_len in the ciphersuite and impose requirements for its value in a way that expand_message will never fail. We can then remove all the if uniform_bytes is INVALID, return INVALID steps (and potentially even make hash_to_scalar to never fail if we remove the check for 0s).

@tplooker tplooker added core Associated to the core spec ready-for-pr labels Feb 22, 2023
@BasileiosKal
Copy link
Contributor Author

Discussed on WG on 6th of Mar. Will revisit for draft 03 after we simplify Sign

@BasileiosKal
Copy link
Contributor Author

Discussed on WG call on 13th of Mar. The current point serialization cannot be curve agnostic. Will update the test vectors to use un-compressed format and re visit this after.

@BasileiosKal
Copy link
Contributor Author

Discussed on WG call 20th of Mar. Will consider having PK as a point, to not restrict key representation.

This was referenced Mar 20, 2023
@BasileiosKal
Copy link
Contributor Author

Discussed on WG call on 27th of Mar. Consensus is to define ciphersuite specific encodings with the addition to define PKs as points in the operation's input

@BasileiosKal BasileiosKal reopened this Mar 27, 2023
@tplooker tplooker added this to the draft-03 milestone May 8, 2023
@BasileiosKal
Copy link
Contributor Author

BasileiosKal commented May 22, 2023

Discussed on the WG on the 22nd of May. Will address after PR #257 is merged

This was referenced Jun 12, 2023
@BasileiosKal BasileiosKal changed the title Ciphersuite format Handle Subroutines ABORT Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Associated to the core spec editorial ready-for-pr
Projects
None yet
Development

No branches or pull requests

2 participants