Missing SSL_CONNECTION_VERSION_TLS1_3 (3729 branch)

[ndesktop]

I'm looking in cef_types and I noticed that SSL_CONNECTION_VERSION_TLS1_3 is missing.

Code: Select all

// Supported SSL version values. See net/ssl/ssl_connection_status_flags.h
// for more information.
typedef enum {
  SSL_CONNECTION_VERSION_UNKNOWN = 0,  // Unknown SSL version.
  SSL_CONNECTION_VERSION_SSL2 = 1,
  SSL_CONNECTION_VERSION_SSL3 = 2,
  SSL_CONNECTION_VERSION_TLS1 = 3,
  SSL_CONNECTION_VERSION_TLS1_1 = 4,
  SSL_CONNECTION_VERSION_TLS1_2 = 5,
  // Reserve 6 for TLS 1.3.
  SSL_CONNECTION_VERSION_QUIC = 7,
} cef_ssl_version_t;


Inside net/\ssl/ssl_connection_status_flags.h, though, it is defined:

Code: Select all

enum SSLVersion {
  SSL_CONNECTION_VERSION_UNKNOWN = 0,  // Unknown SSL version.
  SSL_CONNECTION_VERSION_SSL2 = 1,
  SSL_CONNECTION_VERSION_SSL3 = 2,
  SSL_CONNECTION_VERSION_TLS1 = 3,
  SSL_CONNECTION_VERSION_TLS1_1 = 4,
  SSL_CONNECTION_VERSION_TLS1_2 = 5,
  SSL_CONNECTION_VERSION_TLS1_3 = 6,
  SSL_CONNECTION_VERSION_QUIC = 7,
  SSL_CONNECTION_VERSION_MAX,
};


Is there a reason for this?

Edit: SSL constants were added in 10c1fd6b (9/2/2016), when TLS 1.3 was not defined (RFC 8446 become available in August 2018).
Now I think it can be added.

[magreenblatt]

The enumeration should be updated. Please add a bug, and a PR would be welcome.

[ndesktop]

Sorry about the delay.
Issue 2669 here, PR here.