feat(jsonrpc): add support for allow-insecure-unlock

[timwhite2]

Allow insecure account unlocking when account-related RPCs are exposed by http

Description

Closes: #1657

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• tackled an existing issue or discussed with a team member
• left instructions on how to review the changes
• targeted the correct branch (see PR Targeting)

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

• added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• confirmed all author checklist items have been addressed
• confirmed that this PR does not change production code
• reviewed content
• tested instructions (if applicable)
• confirmed all CI checks have passed

[0xstepit]

Hey @timwhite2, this is cool! Thanks for the implementation of this feature. Will look into it ASAP

[codecov]

Codecov Report

Attention: Patch coverage is 25.00000% with 12 lines in your changes are missing coverage. Please review.

Project coverage is 59.55%. Comparing base (f097735) to head (738e2f6).
Report is 145 commits behind head on main.

❗ Current head 738e2f6 differs from pull request most recent head d642c59. Consider uploading reports for the commit d642c59 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2375       +/-   ##
===========================================
- Coverage   70.45%   59.55%   -10.90%     
===========================================
  Files         293      341       +48     
  Lines       22559    21598      -961     
===========================================
- Hits        15893    12862     -3031     
- Misses       5800     7689     +1889     
- Partials      866     1047      +181     

Files	Coverage Δ
server/config/config.go	47.05% <100.00%> (+10.24%)	⬆️
server/flags/flags.go	71.42% <ø> (ø)
server/start.go	19.89% <100.00%> (ø)
rpc/backend/sign_tx.go	57.14% <0.00%> (-3.81%)	⬇️
rpc/backend/node_info.go	43.42% <0.00%> (-3.09%)	⬇️

... and 341 files with indirect coverage changes

[GAtom22]

Thanks for the contribution @timwhite2!!

Left one comment.
Also, please add a changelog entry

[GAtom22]

LGTM!! Thanks for this great contribution @timwhite2!

[CLAassistant]

All committers have signed the CLA.

[ramacarlucho]

LGTM!
Thank you for the contribution

Test not passing

[ramacarlucho]

Can you fix the tests that are not passing?

[timwhite2]

Can you fix the tests that are not passing?

Sure

[timwhite2]

The other errors seem to have nothing to do with this branch

[timwhite2]

@ramacarlucho It seems that all required checks have passed

addressed

[MalteHerrmann]

Thanks for the contribution, however I see it's being enabled in a lot of places but it's usage is actually not tested. There should be dedicated tests added to showcase the feature and that it is working, if we're going to enable it.

addressed