Split ProxyConnectionSettings into its own PR

Author: michel-laterman

proto/opamp.proto

@@ -327,6 +327,10 @@ message OpAMPConnectionSettings {
     // Optional connection specific TLS settings.
     // Status: [Development]
     TLSConnectionSettings tls = 5;
+
+    // Optional connection specific proxy settings.
+    // Status: [Development]
+    ProxyConnectionSettings proxy = 6;
 }
 
 // The TelemetryConnectionSettings message is a collection of fields which comprise an
@@ -357,6 +361,10 @@ message TelemetryConnectionSettings {
     // Optional connection specific TLS settings.
     // Status: [Development]
     TLSConnectionSettings tls = 4;
+
+    // Optional connection specific proxy settings.
+    // Status: [Development]
+    ProxyConnectionSettings proxy = 5;
 }
 
 // The OtherConnectionSettings message is a collection of fields which comprise an
@@ -406,6 +414,10 @@ message OtherConnectionSettings {
     // Optional connection specific TLS settings.
     // Status: [Development]
     TLSConnectionSettings tls = 5;
+
+    // Optional connection specific proxy settings.
+    // Status: [Development]
+    ProxyConnectionSettings proxy = 6;
 }
 
 
@@ -432,6 +444,30 @@ message TLSConnectionSettings {
   repeated string cipher_suites = 6;
 }
 
+// Status: [Development]
+message ProxyConnectionSettings {
+    // A URL, host:port or some other destination specifier.
+    string destination_endpoint = 1;
+
+    // Optional headers to send to proxies during CONNECT requests.
+    // These headers can be ignored for non-HTTP based proxies.
+    // For example:
+    // key="Authorization", Value="Basic YWxhZGRpbjpvcGVuc2VzYW1l".
+    Headers connect_headers = 2;
+
+    // The Agent should use the offered certificate to connect to the destination
+    // from now on. If the Agent is able to validate and connect using the offered
+    // certificate the Agent SHOULD forget any previous client certificates
+    // for this connection.
+    // This field is optional: if omitted the client SHOULD NOT use a client-side certificate.
+    // This field can be used to perform a client certificate revocation/rotation.
+    TLSCertificate certificate = 3;
+
+    // Optional connection specific TLS settings.
+    // Status: [Development]
+    TLSConnectionSettings tls = 4;
+}
+
 // Status: [Beta]
 message Headers {
     repeated Header headers = 1;

specification.md

@@ -115,18 +115,22 @@ Status: [Beta]
       - [OpAMPConnectionSettings.certificate](#opampconnectionsettingscertificate)
       - [OpAMPConnectionSettings.heartbeat_interval_seconds](#opampconnectionsettingsheartbeat_interval_seconds)
       - [OpAMPConnectionSettings.tls](#opampconnectionsettingstls)
+      - [OpAMPConnectionSettings.proxy](#opampconnectionsettingsproxy)
     + [TelemetryConnectionSettings](#telemetryconnectionsettings)
       - [TelemetryConnectionSettings.destination_endpoint](#telemetryconnectionsettingsdestination_endpoint)
       - [TelemetryConnectionSettings.headers](#telemetryconnectionsettingsheaders)
       - [TelemetryConnectionSettings.certificate](#telemetryconnectionsettingscertificate)
       - [TelemetryConnectionSettings.tls](#telemetryconnectionsettingstls)
+      - [TelemetryConnectionSettings.proxy](#telemetryconnectionsettingsproxy)
     + [OtherConnectionSettings](#otherconnectionsettings)
       - [OtherConnectionSettings.destination_endpoint](#otherconnectionsettingsdestination_endpoint)
       - [OtherConnectionSettings.headers](#otherconnectionsettingsheaders)
       - [OtherConnectionSettings.certificate](#otherconnectionsettingscertificate)
       - [OtherConnectionSettings.other_settings](#otherconnectionsettingsother_settings)
       - [OtherConnectionSettings.tls](#otherconnectionsettingstls)
+      - [OtherConnectionSettings.proxy](#otherconnectionsettingsproxy)
     + [TLSConnectionSettings Message](#tlsconnectionsettings-message)
+    + [ProxyConnectionSettings Message](#proxyconnectionsettings-message)
     + [Headers Message](#headers-message)
     + [TLSCertificate Message](#tlscertificate-message)
       - [TLSCertificate.cert](#tlscertificatecert)
@@ -1909,6 +1913,7 @@ message OpAMPConnectionSettings {
     TLSCertificate certificate = 3;
     uint64 heartbeat_interval_seconds = 4;
     TLSConnectionSettings tls = 5;
+    ProxyConnectionSettings proxy = 6;
 }
 ```
 
@@ -1990,6 +1995,12 @@ Status: [Development]
 
 Optional OpAMP specific TLS settings.
 
+##### OpAMPConnectionSettings.proxy
+
+Status: [Development]
+
+Optional OpAMP specific proxy settings.
+
 #### TelemetryConnectionSettings
 
 The TelemetryConnectionSettings message is a collection of fields which comprise an
@@ -2002,6 +2013,7 @@ message TelemetryConnectionSettings {
     Headers headers = 2;
     TLSCertificate certificate = 3;
     TLSConnectionSettings tls = 4;
+    ProxyConnectionSettings proxy = 5;
 }
 ```
 
@@ -2034,6 +2046,12 @@ Status: [Development]
 
 Optional telemetry specific TLS settings.
 
+##### TelemetryConnectionSettings.proxy
+
+Status: [Development]
+
+Optional telemetry specific proxy settings.
+
 #### OtherConnectionSettings
 
 The OtherConnectionSettings message is a collection of fields which comprise an
@@ -2064,6 +2082,7 @@ message OtherConnectionSettings {
     TLSCertificate certificate = 3;
     map<string, string> other_settings = 4;
     TLSConnectionSettings tls = 5;
+    ProxyConnectionSettings proxy = 6;
 }
 ```
 
@@ -2099,6 +2118,12 @@ Status: [Development]
 
 Optional connection specific TLS settings.
 
+##### OtherConnectionSettings.proxy
+
+Status: [Development]
+
+Optional connection specific proxy settings.
+
 #### TLSConnectionSettings Message
 
 Status: [Development]
@@ -2119,6 +2144,25 @@ message TLSConnectionSettings {
 }
 ```
 
+#### ProxyConnectionSettings Message
+
+Status: [Development]
+
+The message carries optional proxy settings that are used to configure a
+client's connection. If the Agent is able to validate the connection settings,
+the Agent SHOULD forget any previous proxy settings. If this message is not
+included, the client SHOULD use the agent's default proxy settings for the
+connection.
+
+```protobuf
+message ProxyConnectionSettings {
+    string destination_endpoint = 1;
+    Headers connect_headers = 2;
+    TLSCertificate certificate = 3;
+    TLSConnectionSettings tls = 4;
+}
+```
+
 #### Headers Message
 
 ```protobuf