Skip to content

authorization list support on send-transaction #837

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 4 commits into from
Mar 18, 2025
Merged

authorization list support on send-transaction #837

merged 4 commits into from
Mar 18, 2025

Conversation

d4mr
Copy link
Member

@d4mr d4mr commented Jan 27, 2025

PR-Codex overview

This PR introduces error handling for invalid BigInt values and integrates authorization handling into transaction schemas and processing. It enhances type safety and validation for transactions, particularly with authorizationList.

Detailed summary

  • Added badBigIntError function for error handling in src/server/middleware/error.ts.
  • Updated InsertedTransaction type to include authorizationList in src/shared/utils/transaction/types.ts.
  • Introduced requiredBigInt function for validating BigInt values in src/shared/utils/primitive-types.ts.
  • Created authorizationSchema and authorizationListSchema in src/server/schemas/transaction/authorization.ts.
  • Implemented toParsedAuthorization function for parsing authorization data.
  • Modified sendTransaction handler to include authorizationList in src/server/routes/backend-wallet/send-transaction.ts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

github-actions bot commented Feb 6, 2025

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@github-actions github-actions bot closed this Feb 9, 2025
@d4mr d4mr reopened this Feb 18, 2025
Copy link

zeet-co bot commented Feb 18, 2025

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://tw-unreal-demo-engine-vwn0-production.engine-aws-usw2.zeet.app before merging 😉

Copy link

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

Copy link

github-actions bot commented Mar 6, 2025

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@d4mr d4mr merged commit a2eca4e into main Mar 18, 2025
6 of 7 checks passed
@d4mr d4mr deleted the pb/eip-7702 branch March 18, 2025 20:16
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.

3 participants