Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(199)

Unified Diff: webrtc/base/opensslidentity.h

Issue 1683193003: Implement certificate lifetime parameter as required by WebRTC RFC. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Address feedback Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webrtc/base/opensslidentity.cc » ('j') | webrtc/base/opensslidentity.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/opensslidentity.h
diff --git a/webrtc/base/opensslidentity.h b/webrtc/base/opensslidentity.h
index c8aa69a76eee5551b6ddd748bc23c0fb2d630c82..dda50be775a635883a4580522cb5a1898222145a 100644
--- a/webrtc/base/opensslidentity.h
+++ b/webrtc/base/opensslidentity.h
@@ -102,7 +102,8 @@ class OpenSSLCertificate : public SSLCertificate {
class OpenSSLIdentity : public SSLIdentity {
public:
static OpenSSLIdentity* Generate(const std::string& common_name,
- const KeyParams& key_params);
+ const KeyParams& key_params,
+ time_t certificate_lifetime);
Ryan Sleevi 2016/03/08 17:04:43 I would strongly advise against using time_t in a
torbjorng (webrtc) 2016/03/30 14:00:29 Note that Open/BoringSSL uses time_t, and this is
Ryan Sleevi 2016/03/31 02:07:53 OpenSSL's API is actually designed around the ASN1
torbjorng (webrtc) 2016/03/31 13:18:34 Right, I wasn't aware of that one set of interface
static OpenSSLIdentity* GenerateForTest(const SSLIdentityParams& params);
static SSLIdentity* FromPEMStrings(const std::string& private_key,
const std::string& certificate);
« no previous file with comments | « no previous file | webrtc/base/opensslidentity.cc » ('j') | webrtc/base/opensslidentity.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698