Skip to content

add hidden RPC getorphantxs#435

Merged
tcharding merged 1 commit intorust-bitcoin:masterfrom
0xB10C:2025-12-add-hidden-getorphantxs
Jan 9, 2026
Merged

add hidden RPC getorphantxs#435
tcharding merged 1 commit intorust-bitcoin:masterfrom
0xB10C:2025-12-add-hidden-getorphantxs

Conversation

@0xB10C
Copy link
Contributor

@0xB10C 0xB10C commented Dec 22, 2025

getorphantxs ( verbosity )
Shows transactions in the tx orphanage.

This implements the hidden getorphantxs RPC in the last commit. Depends on #432 and #433. Part of #333.

Since the expiry and entry fields were dropped for v30 (this is an experimental RPC after all), there is a into.rs implementation for both v29 and v30.

Copy link
Collaborator

@jamillambert jamillambert left a comment

Choose a reason for hiding this comment

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

Most of it looks good. A few specific comments below.

@0xB10C 0xB10C force-pushed the 2025-12-add-hidden-getorphantxs branch from 12204f8 to ca15500 Compare December 29, 2025 12:45
@0xB10C
Copy link
Contributor Author

0xB10C commented Dec 29, 2025

Addresses review. Thanks!

@jamillambert
Copy link
Collaborator

Almost, for the client macros I meant that the list of macros in the hidden section for both v29 and v30 should be in alphabetical order like the other sections.

@0xB10C 0xB10C force-pushed the 2025-12-add-hidden-getorphantxs branch from ca15500 to 04006e5 Compare December 29, 2025 13:42
@0xB10C
Copy link
Contributor Author

0xB10C commented Dec 29, 2025

I see. Ignoring the _v29__ part in the sorting now. Did the same for v30.

jamillambert
jamillambert previously approved these changes Dec 29, 2025
Copy link
Collaborator

@jamillambert jamillambert left a comment

Choose a reason for hiding this comment

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

ACK 04006e5

@0xB10C 0xB10C dismissed jamillambert’s stale review December 29, 2025 22:51

The merge-base changed after approval.

@0xB10C 0xB10C force-pushed the 2025-12-add-hidden-getorphantxs branch from 04006e5 to a501481 Compare December 30, 2025 10:00
@0xB10C
Copy link
Contributor Author

0xB10C commented Dec 30, 2025

No changes, just rebased. Will rebase again after #433.

@0xB10C 0xB10C force-pushed the 2025-12-add-hidden-getorphantxs branch from a501481 to c274943 Compare January 1, 2026 23:51
@0xB10C 0xB10C force-pushed the 2025-12-add-hidden-getorphantxs branch from c274943 to 426e7b3 Compare January 1, 2026 23:54
@0xB10C
Copy link
Contributor Author

0xB10C commented Jan 2, 2026

rebased with #433 merged! un-drafting

@0xB10C 0xB10C marked this pull request as ready for review January 2, 2026 00:06
Copy link
Collaborator

@jamillambert jamillambert left a comment

Choose a reason for hiding this comment

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

ACK 426e7b3

Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK 426e7b3

@tcharding tcharding merged commit 0ca7342 into rust-bitcoin:master Jan 9, 2026
30 checks passed
tcharding added a commit that referenced this pull request Jan 19, 2026
a7a8785 Add hidden RPC getrawaddrman (deadmanoz)

Pull request description:

  Add support for the hidden `getrawaddrman` RPC that returns raw address manager table entries (added v26). This RPC is marked as experimental and hidden from the API docs by default.

  The `mapped_as` and `source_mapped_as` fields are `Option<u32>` following the established pattern for version-specific fields (no `into.rs` required because these are additions rather than removals or changed types). These fields were added in Bitcoin Core v28 and will deserialise as `None` for earlier versions or when `-asmap` is not configured.

  Note: will have conflicts with #435 and #433, happy to rebase after either merges

ACKs for top commit:
  tcharding:
    ACK a7a8785
  jamillambert:
    ACK a7a8785

Tree-SHA512: 592c499e6f129fafdc764c4fad729f18315a6f9baca3c16a74471929337b49366771db297789e2db89a7fe3602e2a2644f4de6221280098e8c136138a209756c
0xB10C added a commit to 0xB10C/corepc that referenced this pull request Feb 3, 2026
A concrete rust-bitcoin type was added to the version-specific
`GetOrphanTxs` struct during rust-bitcoin#435. This is corrected by using a list
of strings and converting to the model type via `into_model`.

Affects v29 and v30.

Fixes rust-bitcoin#484
0xB10C added a commit to 0xB10C/corepc that referenced this pull request Feb 6, 2026
A concrete rust-bitcoin type was added to the version-specific
`GetOrphanTxs` struct during rust-bitcoin#435. This is corrected by using a list
of strings and converting to the model type via `into_model`.

Affects v29 and v30.

Also, use the explicit error in the itegration tests to check that
the re-exports are good.

Fixes rust-bitcoin#484
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants