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

feat(node): implement node.AuthVerify and node.AuthNew #2370

Merged
merged 2 commits into from
Jun 15, 2023

Conversation

distractedm1nd
Copy link
Collaborator

Implements node.AuthVerify and node.AuthNew.
Removes the jwt.Signer from the Node struct and replaces it with the nodebuilder.Node module, along with the other services.

Includes a minor version bump to the API.

@distractedm1nd distractedm1nd added the area:api Related to celestia-node API label Jun 15, 2023
@distractedm1nd distractedm1nd self-assigned this Jun 15, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #2370 (2de376d) into main (d0e4353) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2370      +/-   ##
==========================================
- Coverage   50.80%   50.54%   -0.26%     
==========================================
  Files         155      155              
  Lines        9766     9826      +60     
==========================================
+ Hits         4962     4967       +5     
- Misses       4369     4426      +57     
+ Partials      435      433       -2     
Impacted Files Coverage Δ
nodebuilder/node.go 58.57% <ø> (ø)

... and 10 files with indirect coverage changes

renaynay
renaynay previously approved these changes Jun 15, 2023
Implements node.AuthVerify and node.AuthNew.
Removes the jwt.Signer from the Node struct and replaces it with the nodebuilder.Node module, along with the other services.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api Related to celestia-node API kind:feat Attached to feature PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants