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

5098 branch 3 update invalid address hash params #7403

Conversation

Matilda-Clerke
Copy link
Contributor

PR description

Update RpcErrorType.INVALID_ADDRESS_HASH_PARAMS

Fixed Issue(s)

#5098

Signed-off-by: Matilda Clerke <[email protected]>
Signed-off-by: Matilda Clerke <[email protected]>
…' into 5098-branch-3-update-invalid-address-hash-params
Signed-off-by: Matilda Clerke <[email protected]>
…' into 5098-branch-3-update-invalid-address-hash-params
…' into 5098-branch-3-update-invalid-address-hash-params
Signed-off-by: Matilda Clerke <[email protected]>
…ounts-params' into 5098-branch-2-update-invalid-accounts-params
…' into 5098-branch-3-update-invalid-address-hash-params
Signed-off-by: Matilda Clerke <[email protected]>
…' into 5098-branch-3-update-invalid-address-hash-params
Copy link
Contributor

@jframe jframe left a comment

Choose a reason for hiding this comment

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

LGTM

@Matilda-Clerke Matilda-Clerke enabled auto-merge (squash) July 31, 2024 06:36
@siladu
Copy link
Contributor

siladu commented Jul 31, 2024

Added some comments to branch 4 which might be relevant here as well #7405 (review)

@Matilda-Clerke Matilda-Clerke merged commit a844684 into hyperledger:main Jul 31, 2024
40 checks passed
gconnect pushed a commit to gconnect/besu that referenced this pull request Aug 26, 2024
* 5098: Add RpcErrorTypes

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Modify InvalidJsonRpcParameters and InvalidJsonRpcRequestException, apply spotless

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Modify InvalidJsonRpcParameters and InvalidJsonRpcRequestException, apply spotless

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Add JsonRpcParameterException for later use

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Update locations for RpcErrorType.INVALID_ACCOUNTS_PARAMS

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Address review comments, apply spotless

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Update with changes from branch 1

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Update code to use RpcErrorType.INVALID_ADDRESS_HASH_PARAMS

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: apply spotless

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: apply spotless

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Update plugin-api gradle hash

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Add comment on INVALID_PARAMS_ERROR_CODE

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Apply spotless on latest changes

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Fix broken unit test

Signed-off-by: Matilda Clerke <[email protected]>

* 5098: Fix broken unit test

Signed-off-by: Matilda Clerke <[email protected]>

---------

Signed-off-by: Matilda Clerke <[email protected]>
Signed-off-by: gconnect <[email protected]>
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.

3 participants