Skip to content

Add JWT authorization + Fixes #27

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

Merged
merged 8 commits into from
May 6, 2023
Merged

Add JWT authorization + Fixes #27

merged 8 commits into from
May 6, 2023

Conversation

PJColombo
Copy link
Member

@PJColombo PJColombo commented May 3, 2023

It adds the following:

  • Add JWT auth to blobscan client requests and implement a token refreshing mechanism.
  • Fix the problem about blob entities matching an incorrect tx.
  • Include the following code refactoring:
    • Remove unused general types
    • Move entity types (block, tx, and blob) to their respective client package.
    • Implement entity conversion logic through the TryForm and Form traits.
    • Abstract utility mappings creation logic into their own functions.
    • Rename API components to "client" for clarity.
    • Simplify client instantiation logic.
    • Move context.rs and env.rs out of /utils folder to improve code discoverability.

@PJColombo PJColombo changed the title Add JWT authentication Add JWT authorization May 3, 2023
@PJColombo PJColombo marked this pull request as ready for review May 3, 2023 14:07
@PJColombo PJColombo requested a review from 0xGabi May 3, 2023 14:08
@PJColombo PJColombo linked an issue May 3, 2023 that may be closed by this pull request
@PJColombo PJColombo changed the title Add JWT authorization Add JWT authorization + Fixes May 3, 2023
@PJColombo PJColombo merged commit dd1eace into master May 6, 2023
@PJColombo PJColombo deleted the jwt-auth branch May 6, 2023 17:05
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.

Add JWT Auth
1 participant