mozilla-vpn-client icon indicating copy to clipboard operation
mozilla-vpn-client copied to clipboard

VPN-6012 basti/web ext refactor

Open strseb opened this issue 1 year ago • 0 comments

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.

This does not actually change the API

strseb avatar Feb 07 '24 17:02 strseb