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

New API endpoint to sign and broadcast #126

Merged
merged 7 commits into from
Mar 16, 2022
Merged

New API endpoint to sign and broadcast #126

merged 7 commits into from
Mar 16, 2022

Conversation

jasny
Copy link
Member

@jasny jasny commented Mar 4, 2022

  • Accept Authorization Bearer in addition to X-API-KEY and fixed Swagger to send the Authorization header.
    Fixed Content-Type issue when using Swagger.
    Fixes Swagger UI API Key not applied in test node #122

  • Removed /transactions/sign/{sender}, /transactions/sponsor/{sponsor} and /transactions/sponsor.
    Instead, use /transactions/sign with sender and/or sponsor in JSON body.
    TxRequest class resolves the sender and sponsor. This is no longer done as custom logic in TransactionApiRoute.

  • Added /transactions/submit and /transactions/submit/{type}.
    These endpoints will sign and broadcast a transaction.

Remove deprecated api_key header.
Remove default API key in application.conf.
Fixed `Content-Type` issue when using Swagger.

Removed `/transactions/sign/{sender}`, `/transactions/sponsor/{sponsor}` and `/transactions/sponsor`.
Instead, use `/transactions/sign` with sender and/or sponsor in JSON body.
TxRequest class resolves the sender and sponsor. This is no longer done as custom logic in TransactionApiRoute.

Added `/transactions/submit` and `/transactions/submit/{type}`.
These endpoints will sign and broadcast a transaction.
@jasny jasny merged commit 7f3d5af into master Mar 16, 2022
@jasny jasny deleted the submit-transaction branch March 16, 2022 12:03
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.

Swagger UI API Key not applied in test node
1 participant