Skip to content

#24 Added support for CBOR request parsing#25

Merged
b3y0urs3lf merged 8 commits intomainfrom
issue-24
Feb 23, 2026
Merged

#24 Added support for CBOR request parsing#25
b3y0urs3lf merged 8 commits intomainfrom
issue-24

Conversation

@martti007
Copy link
Copy Markdown
Contributor

No description provided.

@martti007 martti007 requested a review from Copilot February 10, 2026 10:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds CBOR-aware routing parameter extraction for specific JSON-RPC methods.

Changes:

  • Introduces CBOR deserialization + hex decode/encode to extract requestId for certification_request.
  • Adds special-case requestId extraction for get_inclusion_proof.v2.
  • Simplifies virtual thread invocation by removing an unnecessary local variable.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java Outdated
Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java Outdated
Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java Outdated
@martti007 martti007 requested a review from MastaP February 10, 2026 16:31
@martti007 martti007 changed the title #24 Added support for CBOR parsing #24 Added support for CBOR request parsing Feb 12, 2026
@martti007 martti007 linked an issue Feb 12, 2026 that may be closed by this pull request
@MastaP MastaP requested a review from Copilot February 13, 2026 12:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java
Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java Outdated
Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java
Comment thread src/main/java/org/unicitylabs/proxy/RequestHandler.java
@b3y0urs3lf b3y0urs3lf merged commit 3c9d52b into main Feb 23, 2026
1 check 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.

[TGE] CBOR support for Subscription Gateway

4 participants