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

Adapt testnet debug/tracers tests so they use Solo instead except for the call ones #1357

Closed
freemanzMrojo opened this issue Sep 27, 2024 · 0 comments · Fixed by #1374
Closed

Comments

@freemanzMrojo
Copy link
Member

Summary

https://github.com/vechain/thor/releases/tag/v2.1.4-rc.3 disables some of the debug/tracers methods (look for the GitHub issue ID in the codebase) so some of the testnet tests are disabled.

The purpose of this ticket is to adapt them so they point to solo instead, while remaining the call ones for testnet.

Basic Example

The error we are getting is as follows:

2024-09-27T11:22:11.2045814Z   ● ThorClient - Debug Module › traceTransactionClause › traceTransactionClause - opcount
2024-09-27T11:22:11.2045821Z 
2024-09-27T11:22:11.2045987Z     Method 'HttpClient.http()' failed.
2024-09-27T11:22:11.2046425Z     -Reason: 'Request failed with status 403 and message creating tracer is not allowed: opcount'
2024-09-27T11:22:11.2046541Z     -Parameters: 
2024-09-27T11:22:11.2046619Z     	{
2024-09-27T11:22:11.2046724Z       "method": "POST",
2024-09-27T11:22:11.2046917Z       "url": "https://testnet.vechain.org/debug/tracers"
@freemanzMrojo freemanzMrojo changed the title Adapt testnet debug/tracers tests so they use Solo instead except for call ones Adapt testnet debug/tracers tests so they use Solo instead except for the call ones Sep 27, 2024
@victhorbi victhorbi added this to the Audit milestone Sep 30, 2024
@rodolfopietro97 rodolfopietro97 self-assigned this Oct 1, 2024
@rodolfopietro97 rodolfopietro97 linked a pull request Oct 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants