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

Add /r/tx/<txid> #4147

Merged
merged 10 commits into from
Jan 27, 2025
Merged

Add /r/tx/<txid> #4147

merged 10 commits into from
Jan 27, 2025

Conversation

samedcildir
Copy link
Contributor

This endpoint bypasses the index and acts as a proxy to get raw transaction hex from bitcoin cli. It returns hex encoded raw transaction data as a JSON string.

PR also includes recursion docs update to reflect the new endpoint and tests for the new endpoint.

@samedcildir
Copy link
Contributor Author

@casey @raphjaph is there any progress about this PR? I guess we've gone through all scenarios about this endpoint in the Issue #4108 and we can use the endpoint with an immutable transaction parsing library. Thank you for sharing your insights and time.

@raphjaph raphjaph changed the title Add /r/txhex/<txid> for tx-hex recursion Add /r/tx/<txid> Jan 27, 2025
Copy link
Collaborator

@raphjaph raphjaph left a comment

Choose a reason for hiding this comment

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

LGTM

@raphjaph raphjaph enabled auto-merge (squash) January 27, 2025 20:38
@raphjaph raphjaph merged commit 7d3a16d into ordinals:master Jan 27, 2025
5 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