Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

Upgrade signers to 1.0.19 #386

Merged
merged 6 commits into from
Sep 27, 2021
Merged

Upgrade signers to 1.0.19 #386

merged 6 commits into from
Sep 27, 2021

Conversation

siladu
Copy link
Contributor

@siladu siladu commented Sep 22, 2021

Includes vertx upgrade and transitive azure key vault upgrade with breaking change to SECP256K1 curve.

Allows empty password files to be read.
Fixes #372

Includes vertx upgrade and transitive azure key vault upgrade with breaking change to SECP256K1 curve.
@siladu siladu marked this pull request as ready for review September 23, 2021 13:05
@siladu siladu changed the title Upgrade signers to 1.0.18 Upgrade signers to 1.0.19 Sep 24, 2021
CHANGELOG.md Outdated
- Add validation for GoQuorum transactions with value https://github.com/ConsenSys/ethsigner/pull/377
- Upgrade web3j to latest version for fix to handle large chainids in eip155 transactions https://github.com/ConsenSys/ethsigner/pull/382
- Add publishing to docker namespace "consensys/ethsigner" and deprecate docker namespace "consensys/quorum-ethsigner" https://github.com/ConsenSys/ethsigner/issues/384
- Upgrade to signers 1.0.19 allows empty password files to be read when creating a Signer https://github.com/ConsenSys/ethsigner/issues/372
Copy link
Contributor Author

@siladu siladu Sep 24, 2021

Choose a reason for hiding this comment

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

@usmansaleem Should this be under "Bugs Fixed"?

CHANGELOG.md Outdated
- Add publishing to docker namespace "consensys/ethsigner"
- Deprecating docker namespace "consensys/quorum-ethsigner"
- Add validation for GoQuorum transactions with value https://github.com/ConsenSys/ethsigner/pull/377
- Upgrade web3j to latest version for fix to handle large chainids in eip155 transactions https://github.com/ConsenSys/ethsigner/pull/382
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@usmansaleem Should this be under "Bugs Fixed"?

@@ -1,11 +1,14 @@
# Changelog

## ...Next Version...
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@usmansaleem Any suggestions for the next version number?

Copy link
Contributor

@usmansaleem usmansaleem left a comment

Choose a reason for hiding this comment

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

LGTM

@siladu siladu merged commit 3c6882a into Consensys:master Sep 27, 2021
@siladu siladu deleted the update-signers branch September 27, 2021 13:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ethsigner fails to read empty password files and crashes
2 participants