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.

Sign up for GitHub

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

Jump to bottom

Avail TurboDA Upgrade #3584

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Avail TurboDA Upgrade #3584

wants to merge 1 commit into from

Conversation

christineRPM
Copy link

What ❔

This PR upgrades Avail Turbo DA from v0.1 to v1 with significant API changes and improvements to the authentication system. The upgrade includes a complete overhaul of the API endpoints structure, authentication mechanism (moving from JWT to API keys), and internal architecture improvements.

Why ❔

These changes are implemented to:

  • Improve security through better API key management and support for multiple keys per user
  • Enhance performance with better latency and higher throughput capabilities
  • Create a more maintainable and extensible codebase through modular design
  • Establish proper API versioning to better manage future updates and backwards compatibility
  • Standardize endpoint structures by removing redundant path segments

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zkstack dev fmt and zkstack dev lint.

@christineRPM
migrate to new turboda
2839f3e 
- More well managed API keys.
- Multiple API keys against a single user.
- Better latency response
- Higher throughput
- Modular design to easily add new features.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
external-contribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@christineRPM