mozilla-vpn-client
mozilla-vpn-client copied to clipboard
VPN-6012 basti/web ext refactor
Description
Before tacking new stuff into the Extension API, i think this is a good option to cleanup some stuff.
Currently it's named /server
and we have one class plaing a "simple tcp" server and another class, both handling the protocol definition and the rpc part based of the protocol.
My goal is to break this into 3 parts: -> One class only handling "what rpc'calls are available" -> One class "how is this encoded on connection" -> One class creating a Service to connect to. I've tried to make all things loosely coupled, with exception of the VPNAdapter that handles the VPN specific actions, so i.e we can switch out the underlying QIODevice whenever we want to.
A rare thing, reading commit by commit actually make more sense.
- 01: Just Rename things
- 02: Loosely couple the Components
- 03: Move Non VPN dependent code into it's own target
- 04: Make it a module and add tests
- 05: How about... documentation?!
This does not actually change the API