Skip to content

feat: enable external signer mode for production - #2083

Merged
DariusParvin merged 1 commit into
masterfrom
BG-43925-remove-signer-prod-restriction
Mar 8, 2022
Merged

feat: enable external signer mode for production#2083
DariusParvin merged 1 commit into
masterfrom
BG-43925-remove-signer-prod-restriction

Conversation

@DariusParvin

Copy link
Copy Markdown
Contributor

remove the restriction that allowed the external signer to be used only in a test env

Note: This should only be merged after the following PRs have been merged

Ticket: BG-43925

remove the restriction that allowed the external signer to be used only in a test env

Ticket: BG-43925
@DariusParvin
DariusParvin force-pushed the BG-43925-remove-signer-prod-restriction branch from 5104574 to 077d2de Compare March 8, 2022 00:28
@DariusParvin
DariusParvin marked this pull request as ready for review March 8, 2022 00:28
@codecov

codecov Bot commented Mar 8, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2083 (97b4424) into master (7f08e16) will increase coverage by 52.50%.
The diff coverage is n/a.

❗ Current head 97b4424 differs from pull request most recent head 077d2de. Consider uploading reports for the commit 077d2de to get more accurate results

@@             Coverage Diff             @@
##           master    #2083       +/-   ##
===========================================
+ Coverage        0   52.50%   +52.50%     
===========================================
  Files           0        6        +6     
  Lines           0      558      +558     
  Branches        0       85       +85     
===========================================
+ Hits            0      293      +293     
- Misses          0      259      +259     
- Partials        0        6        +6     
Flag Coverage Δ
unit 52.50% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
modules/express/src/expressApp.ts 89.51% <ø> (ø)
modules/express/src/args.ts 100.00% <0.00%> (ø)
modules/express/src/clientRoutes.ts 33.52% <0.00%> (ø)
modules/express/src/retryPromise.ts 11.76% <0.00%> (ø)
modules/express/src/errors.ts 82.35% <0.00%> (ø)
modules/express/src/config.ts 100.00% <0.00%> (ø)

@DariusParvin
DariusParvin merged commit 35581de into master Mar 8, 2022
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.

2 participants