Page MenuHomePhabricator

Remove config managed RPC user/pass
ClosedPublic

Authored by deadalnix on Apr 30 2020, 22:42.

Details

Summary

The user/pass is read from garg and stored into the config, to be fetched from there again by the exact same module. Even if we can move away from having so much global state, then a member variable for the http thing would make sense, not the config.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Branch
ConfigRPCUserAndPassword
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 10532
Build 18877: Default Diff Build & Tests
Build 18876: arc lint + arc unit

Event Timeline

Also do it for CORS config

This revision is now accepted and ready to land.May 1 2020, 16:07
This revision was automatically updated to reflect the committed changes.