Skip to content

Adds checks for nil keys and signatures (backport #1855)#1866

Merged
lasarojc merged 2 commits intov1.xfrom
mergify/bp/v1.x/pr-1855
Dec 19, 2023
Merged

Adds checks for nil keys and signatures (backport #1855)#1866
lasarojc merged 2 commits intov1.xfrom
mergify/bp/v1.x/pr-1855

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 19, 2023

This is an automatic backport of pull request #1855 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* Check for nil keys.

* Check for nil signatures and complement test

* Check that the validator does not have an empty pub key.

* Add and move tests

* improve error message

* Add changelog

(cherry picked from commit c88a25f)
@mergify mergify bot requested a review from a team as a code owner December 19, 2023 15:02
@mergify mergify bot requested a review from a team December 19, 2023 15:02
@lasarojc lasarojc merged commit 3cd9c73 into v1.x Dec 19, 2023
@lasarojc lasarojc deleted the mergify/bp/v1.x/pr-1855 branch December 19, 2023 15:50
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.

1 participant