Skip to content

fix: missing TraceHttp logs#129

Merged
gregorydemay merged 3 commits intomainfrom
gdemay/XC-378-missing-logs
May 27, 2025
Merged

fix: missing TraceHttp logs#129
gregorydemay merged 3 commits intomainfrom
gdemay/XC-378-missing-logs

Conversation

@gregorydemay
Copy link
Contributor

@gregorydemay gregorydemay commented May 27, 2025 •
edited
Loading

Logs at level TraceHttp where not added to the canister response when queried via the HTTP endpoint /logs (was probably forgotten in #13).

@gregorydemay gregorydemay changed the title fix: missing logs fix: missing TraceHttp logs May 27, 2025
@gregorydemay gregorydemay requested review from lpahlavi and ninegua May 27, 2025 05:24
@gregorydemay gregorydemay marked this pull request as ready for review May 27, 2025 05:24
@gregorydemay gregorydemay requested a review from a team as a code owner May 27, 2025 05:24
@lpahlavi
Copy link
Contributor

lpahlavi commented May 27, 2025

Thank you for looking into this and for this fix @gregorydemay!

@gregorydemay gregorydemay merged commit c5f5629 into main May 27, 2025
14 of 15 checks passed
@gregorydemay gregorydemay deleted the gdemay/XC-378-missing-logs branch May 27, 2025 06:26
@gregorydemay gregorydemay mentioned this pull request May 27, 2025
Merged
gregorydemay added a commit to dfinity/evm-rpc-canister that referenced this pull request May 27, 2025
@gregorydemay
fix: missing TraceHttp logs (#421)
20d3901 
Logs at level `TraceHttp` where not added to the canister response when
queried via the HTTP endpoint `/logs`. Similar to
dfinity/sol-rpc-canister#129.
@github-actions github-actions bot mentioned this pull request May 27, 2025
Merged
gregorydemay added a commit that referenced this pull request May 27, 2025
@github-actions @gregorydemay
chore: release v0.2.0 (#131)
11c9802 
## 🤖 New release

* `sol_rpc_canister`: 0.2.0

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

## [0.2.0] - 2025-05-27

### Added

- Add client method to sign a transaction
([#113](#113))
- Add Chainstack RPC provider
([#118](#118))
- Add support for `getSignaturesForAddress`
([#106](#106))
- Add support for `getSignatureStatuses` RPC method
([#96](#96))
- Add support for `getTokenAccountBalance` RPC method
([#90](#90))
- Add support for `getTransaction` RPC method
([#68](#68))
- Add `getBlock` RPC method
([#53](#53))
- Add `sendTransaction` RPC method
([#59](#59))
- Add NOTICE to Apache license
([#60](#60))
- Add `getAccountInfo` RPC method
([#49](#49))
- Add metrics
([#41](#41))
- Add logging crate
([#13](#13))
- Add support for override providers
([#12](#12))

### Changed

- Bump version to 0.2
- Candid NonZeroU8
([#108](#108))
- Add `RoundingError` to `sol_rpc_types`
([#105](#105))
- Use secure primitive types for `Pubkey`, `Signature` and `Hash`
([#98](#98))
- Add support for `getRecentPrioritizationFees`
([#92](#92))
- Simplify API keys provisioning script
([#89](#89))
- Release v0.1.0
([#88](#88))
- Use `canlog_derive` and `canlog` from crates.io
([#84](#84))
- Release pipeline
([#4](#4))
- Clean-up TODOs
([#81](#81))
- Support method `getBalance`
([#74](#74))
- Remove http_types module and use external ic-http-types crate
([#73](#73))
- Rename some enum variants to camel case when serializing
([#72](#72))
- Use constant size JSON-RPC request ID
([#62](#62))
- Use method from JSON-RPC request for metric
([#61](#61))
- Cycles cost
([#52](#52))
- Client builder
([#54](#54))
- Round result from `getSlot` RPC method
([#48](#48))
- Use `canhttp` `multi` feature
([#46](#46))
- Implement a method for making generic RPC request
([#39](#39))
- Implement getSlot RPC method
([#33](#33))
- Add some tested RPC providers for Solana Mainnet and Devnet
([#15](#15))
- Streamline providers
([#32](#32))
- Update rust toolchain to 1.85
([#21](#21))
- Remove unnecessary Storable implementations
([#14](#14))
- Add support for API keys
([#10](#10))
- Hard-code SOL RPC providers
([#9](#9))
- Reproducible build
([#3](#3))
- Initial cargo workspace and build pipeline
([#2](#2))

### Fixed

- Missing `TraceHttp` logs
([#129](#129))
- End-to-end tests for `sendTransaction`
([#104](#104))
- Unit test for `getRecentPrioritizationFees` parameters serialization
([#107](#107))
- Integration test for `verifyApiKey`
([#82](#82))
- Set `maxSupportedTransactionVersion` to zero for end-to-end tests
([#85](#85))
- API keys ([#58](#58))
- End-to-end tests
([#45](#45))
- Correct Solana cluster for dRPC and Helius providers
([#47](#47))
- E2e test with Solana test validator
([#20](#20))
- Create test canister on ICP mainnet
([#8](#8))

### Removed

- Remove default/non-default providers
([#122](#122))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: gregorydemay <gregory.demay@dfinity.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot mentioned this pull request Jun 12, 2025
Merged
gregorydemay added a commit that referenced this pull request Jun 12, 2025
@github-actions @gregorydemay
chore: release v1.0.0 (#146)
e3e416a 
## 🤖 New release

* `sol_rpc_canister`: 1.0.0

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

## [1.0.0] - 2025-06-12

### Added

- Add support for `transactionDetails=accounts`
([#139](#139))
- Add support for `rewards` parameter for `getBlock`
([#135](#135))
- Add helper methods for request builders
([#136](#136))
- Add client method to sign a transaction
([#113](#113))
- Add Chainstack RPC provider
([#118](#118))
- Add support for `getSignaturesForAddress`
([#106](#106))
- Add support for `getSignatureStatuses` RPC method
([#96](#96))
- Add support for `getTokenAccountBalance` RPC method
([#90](#90))
- Add support for `getTransaction` RPC method
([#68](#68))
- Add `getBlock` RPC method
([#53](#53))
- Add `sendTransaction` RPC method
([#59](#59))
- Add NOTICE to Apache license
([#60](#60))
- Add `getAccountInfo` RPC method
([#49](#49))
- Add metrics
([#41](#41))
- Add logging crate
([#13](#13))
- Add support for override providers
([#12](#12))

### Changed

- Bump to 1.0.0
- Inline request parameters `is_default_config` methods
([#138](#138))
- Release v0.2.0
([#131](#131))
- Bump version and use a release notes template
([#130](#130))
- Candid NonZeroU8
([#108](#108))
- Add `RoundingError` to `sol_rpc_types`
([#105](#105))
- Use secure primitive types for `Pubkey`, `Signature` and `Hash`
([#98](#98))
- Add support for `getRecentPrioritizationFees`
([#92](#92))
- Simplify API keys provisioning script
([#89](#89))
- Release v0.1.0
([#88](#88))
- Use `canlog_derive` and `canlog` from crates.io
([#84](#84))
- Release pipeline
([#4](#4))
- Clean-up TODOs
([#81](#81))
- Support method `getBalance`
([#74](#74))
- Remove http_types module and use external ic-http-types crate
([#73](#73))
- Rename some enum variants to camel case when serializing
([#72](#72))
- Use constant size JSON-RPC request ID
([#62](#62))
- Use method from JSON-RPC request for metric
([#61](#61))
- Cycles cost
([#52](#52))
- Client builder
([#54](#54))
- Round result from `getSlot` RPC method
([#48](#48))
- Use `canhttp` `multi` feature
([#46](#46))
- Implement a method for making generic RPC request
([#39](#39))
- Implement getSlot RPC method
([#33](#33))
- Add some tested RPC providers for Solana Mainnet and Devnet
([#15](#15))
- Streamline providers
([#32](#32))
- Update rust toolchain to 1.85
([#21](#21))
- Remove unnecessary Storable implementations
([#14](#14))
- Add support for API keys
([#10](#10))
- Hard-code SOL RPC providers
([#9](#9))
- Reproducible build
([#3](#3))
- Initial cargo workspace and build pipeline
([#2](#2))

### Fixed

- Missing `TraceHttp` logs
([#129](#129))
- End-to-end tests for `sendTransaction`
([#104](#104))
- Unit test for `getRecentPrioritizationFees` parameters serialization
([#107](#107))
- Integration test for `verifyApiKey`
([#82](#82))
- Set `maxSupportedTransactionVersion` to zero for end-to-end tests
([#85](#85))
- API keys ([#58](#58))
- End-to-end tests
([#45](#45))
- Correct Solana cluster for dRPC and Helius providers
([#47](#47))
- E2e test with Solana test validator
([#20](#20))
- Create test canister on ICP mainnet
([#8](#8))

### Removed

- Remove default/non-default providers
([#122](#122))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: gregorydemay <gregory.demay@dfinity.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: gregorydemay <112856886+gregorydemay@users.noreply.github.com>
@github-actions github-actions bot mentioned this pull request Jul 31, 2025
Closed
rileyHoward92 added a commit to rileyHoward92/evm-rpc-canister that referenced this pull request Sep 25, 2025
@rileyHoward92
fix: missing TraceHttp logs (#421)
f8ecc3e 
Logs at level `TraceHttp` where not added to the canister response when
queried via the HTTP endpoint `/logs`. Similar to
dfinity/sol-rpc-canister#129.
node-mancer556 added a commit to node-mancer556/evm-rpc-canister that referenced this pull request Sep 27, 2025
@node-mancer556
fix: missing TraceHttp logs (#421)
ddffd9f 
Logs at level `TraceHttp` where not added to the canister response when
queried via the HTTP endpoint `/logs`. Similar to
dfinity/sol-rpc-canister#129.
AngieD101 added a commit to AngieD101/evm-rpc-canister that referenced this pull request Oct 12, 2025
@AngieD101
fix: missing TraceHttp logs (#421)
30589f2 
Logs at level `TraceHttp` where not added to the canister response when
queried via the HTTP endpoint `/logs`. Similar to
dfinity/sol-rpc-canister#129.
brightverdellmkf-61 added a commit to brightverdellmkf-61/evmrpcc that referenced this pull request Oct 27, 2025
@brightverdellmkf-61
fix: missing TraceHttp logs (#421)
eab4de7 
Logs at level `TraceHttp` where not added to the canister response when
queried via the HTTP endpoint `/logs`. Similar to
dfinity/sol-rpc-canister#129.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ninegua ninegua ninegua approved these changes

@lpahlavi lpahlavi lpahlavi approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gregorydemay @lpahlavi @ninegua