Skip to content

feat: include get decrypted transaction call for shutterized txns#845

Merged
aloknerurkar merged 5 commits intoprimev:mainfrom
blockchainluffy:main
Nov 5, 2025
Merged

feat: include get decrypted transaction call for shutterized txns#845
aloknerurkar merged 5 commits intoprimev:mainfrom
blockchainluffy:main

Conversation

@blockchainluffy
Copy link
Copy Markdown
Contributor

Describe your changes

This PR introduces the GetDecryptedTransaction method in the provider API to retrieve decrypted shutterized transactions from the sequencer. This serves as the initial step toward the Shutter integration.

In future iterations, this implementation will be updated to utilize the keyper P2P network for lower latency and improved performance.

Additionally, this PR adds the eon_id field to facilitate easier coordination and identification of the keyper set.

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

Copy link
Copy Markdown
Collaborator

@aloknerurkar aloknerurkar left a comment

Choose a reason for hiding this comment

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

There is some issue with the PR title again. I think camel case is also not supported in the title.

@blockchainluffy blockchainluffy changed the title feat: include GetDecryptedTransaction call for shutterized transactions feat: include get_decrypted_transaction call for shutterized transactions Nov 4, 2025
@blockchainluffy
Copy link
Copy Markdown
Contributor Author

@aloknerurkar thanks, updated title!

Comment thread p2p/pkg/rpc/provider/service.go Outdated
ctx context.Context,
req *providerapiv1.GetDecryptedTransactionRequest,
) (*providerapiv1.GetDecryptedTransactionResponse, error) {
txHashes := req.GetTxHash()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: I m guessing it only supports 1 transaction at a time. Better to name it txHash in that case?

@aloknerurkar aloknerurkar changed the title feat: include get_decrypted_transaction call for shutterized transactions feat: include GetDecryptedTransaction call for shutterized txns Nov 5, 2025
@aloknerurkar aloknerurkar changed the title feat: include GetDecryptedTransaction call for shutterized txns feat: include get decrypted transaction call for shutterized txns Nov 5, 2025
@aloknerurkar aloknerurkar merged commit 21e6fef into primev:main Nov 5, 2025
5 of 8 checks passed
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.

2 participants