FIxes gasLimit parse bug & updates vault and go

wistefan commented 2 years ago

Description

The PR updates the go-lang version and the vault-dependencies. Beside that, a fix for parsing the gas_limit from request is provided. Previous implementation ignored the provided gas_limit and took the default value in all cases. The fix does only take the default, if not gas_limit is provided.

Motivation and Context

Plugin should run with latest vault
I want to use the plugin for signing transactions. This requires proper handling of the gas_limit field.

Types of changes

[ ] Documentation enhancement
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

wistefan commented 2 years ago

@cypherhat is this repo still active? Or is there a new place to apply PRs? Best, Stefan

cypherhat commented 2 years ago

sorry - lot of life things :)

wistefan commented 2 years ago

@cypherhat No problem, thank you for merging. Very nice plugin, btw.

immutability-io / vault-ethereum