Skip to content

feat(validator): first implementation#1687

Merged
paologallinaharbur merged 2 commits intomainfrom
feat/verifier
Sep 17, 2025
Merged

feat(validator): first implementation#1687
paologallinaharbur merged 2 commits intomainfrom
feat/verifier

Conversation

@paologallinaharbur
Copy link
Member

@paologallinaharbur paologallinaharbur commented Sep 16, 2025 •
edited
Loading

What this PR does / why we need it

This Pr introduces the first iteration of the verifier.

This PR assumes:

Consequently, an EdDSA public key is represented by a single parameter though it represents a point with two coordinates. This becomes clearly visible in the JWK format of an EdDSA public key, where the parameter x holds the encoded public key:

"kty": "OKP",
"kid": "-1909572257",
"alg":	"EdDSA",
"crv":	"Ed25519",
"x":	"XWxGtApfcqmKI7p0OKnF5JSEWMVoLsytFXLEP7xZ_l8"

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Provided a meaningful title following conventional commit style.
  • Included a detailed description for the Pull Request.
  • Documentation under docs is aligned with the change.
  • Follows guidelines for Pull Requests in CONTRIBUTING.md.
  • Follows log level guidelines.

Base automatically changed from feat/handle-algorithm to main September 17, 2025 06:22
@paologallinaharbur paologallinaharbur marked this pull request as ready for review September 17, 2025 11:32
@paologallinaharbur paologallinaharbur requested a review from a team as a code owner September 17, 2025 11:32
sigilioso
sigilioso reviewed Sep 17, 2025
View reviewed changes
Copy link
Contributor

@sigilioso sigilioso left a comment

Choose a reason for hiding this comment

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

Thanks! Find some comments below

sigilioso
sigilioso approved these changes Sep 17, 2025
View reviewed changes
Copy link
Contributor

@sigilioso sigilioso left a comment

Choose a reason for hiding this comment

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

LGTM

@paologallinaharbur paologallinaharbur merged commit 761fec7 into main Sep 17, 2025
28 checks passed
@paologallinaharbur paologallinaharbur deleted the feat/verifier branch September 17, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sigilioso sigilioso sigilioso approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@paologallinaharbur @sigilioso