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

fix: removed name field in JsonRpcError class #2590

Conversation

quiet-node
Copy link
Member

Description:
This PR removes the name field in JsonRpcError class to conform to the JSON-RPC 2.0 Specification. Many tests were updated to apply this new changes.

Related issue(s):

Fixes #2589

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@quiet-node quiet-node added the bug Something isn't working label Jun 11, 2024
@quiet-node quiet-node added this to the 0.50.0 milestone Jun 11, 2024
@quiet-node quiet-node self-assigned this Jun 11, 2024
@quiet-node quiet-node requested review from a team, Nana-EC and lukelee-sl as code owners June 11, 2024 21:05
Copy link

github-actions bot commented Jun 11, 2024

Acceptance Tests

  18 files  243 suites   30m 28s ⏱️
603 tests 598 ✔️ 3 💤 2
762 runs  755 ✔️ 5 💤 2

Results for commit f37d253.

♻️ This comment has been updated with latest results.

@quiet-node quiet-node force-pushed the 2589-relay-error-response-is-not-conforming-to-the-json-rpc-20-specification branch 4 times, most recently from 7c0b1e8 to 1f1a4e0 Compare June 12, 2024 14:26
@quiet-node quiet-node requested review from a team as code owners June 12, 2024 14:45
@quiet-node quiet-node force-pushed the 2589-relay-error-response-is-not-conforming-to-the-json-rpc-20-specification branch from 6092489 to 9f5296c Compare June 12, 2024 15:50
Copy link

github-actions bot commented Jun 12, 2024

Tests

    2 files  151 suites   13s ⏱️
840 tests 839 ✔️ 1 💤 0
852 runs  851 ✔️ 1 💤 0

Results for commit f37d253.

♻️ This comment has been updated with latest results.

@quiet-node quiet-node force-pushed the 2589-relay-error-response-is-not-conforming-to-the-json-rpc-20-specification branch 2 times, most recently from a0f143f to c8024e3 Compare June 12, 2024 17:51
Copy link
Collaborator

@AlfredoG87 AlfredoG87 left a comment

Choose a reason for hiding this comment

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

PR Changes are empty 😅

Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>

Revert "fix: removed name field in JsonRpcError class"

This reverts commit c8024e3.
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>

Reapply "fix: removed name field in JsonRpcError class"

This reverts commit e64d1fc.
@quiet-node quiet-node force-pushed the 2589-relay-error-response-is-not-conforming-to-the-json-rpc-20-specification branch from e64d1fc to f37d253 Compare June 13, 2024 13:57
Copy link

sonarcloud bot commented Jun 13, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@quiet-node
Copy link
Member Author

PR Changes are empty 😅

😓 yes I reverted the changes to test the Tests/Tests CI and it still failed for some reasons.

Copy link
Collaborator

@AlfredoG87 AlfredoG87 left a comment

Choose a reason for hiding this comment

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

LGTM

@quiet-node quiet-node merged commit ef4c91e into main Jun 13, 2024
35 checks passed
@quiet-node quiet-node deleted the 2589-relay-error-response-is-not-conforming-to-the-json-rpc-20-specification branch June 13, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Relay error response is not conforming to the JSON-RPC 2.0 Specification
2 participants