Skip to content

Conversation

@aryanjassal
Copy link
Contributor

@aryanjassal aryanjassal commented Jun 18, 2025

Description

This PR adds the polykey auth login command, which takes in an encoded JWT, signs it, and returns the new token. This is meant to verify the identity of the node, which would be as part of a returned field labelled nodeId.

Issues Fixed

Tasks

  • 1. Create polykey auth subcommand
  • 2. Create polykey auth login command
  • 3. Integrate AuthSignToken handler from Polykey
  • 4. Write tests for this behaviour

Final checklist

  • Domain specific tests
  • Full tests
  • Updated inline-comment documentation
  • Lint fixed
  • Squash and rebased
  • Sanity check the final build

@aryanjassal aryanjassal self-assigned this Jun 18, 2025
Copy link
Contributor

@tegefaulkes tegefaulkes left a comment

Choose a reason for hiding this comment

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

Some small comments.

@aryanjassal aryanjassal force-pushed the feature-polykey-auth-login branch from 8aa67b3 to be4d177 Compare June 19, 2025 03:54
@aryanjassal aryanjassal merged commit 83ed8d1 into staging Jun 19, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants