Skip to content

Commit

Permalink
Refactor uses of setClientCertificate to setClientCertManager
Browse files Browse the repository at this point in the history
Summary: use the new setClientCertManager method instead of the deprecated setClientCertificate method and remove it from FizzClientContext.h

Reviewed By: zxjtan

Differential Revision: D62404702

fbshipit-source-id: 936c19c31499558043e50027875143edebb4539c
  • Loading branch information
Abdulkadir Fiqi authored and facebook-github-bot committed Sep 16, 2024
1 parent 64334f9 commit 9beaf18
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion mcrouter/lib/network/FizzContextProvider.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,9 @@ FizzContextAndVerifier createClientFizzContextAndVerifier(
if (!certData.empty() && !keyData.empty()) {
auto cert = fizz::openssl::CertUtils::makeSelfCert(
std::move(certData), std::move(keyData));
ctx->setClientCertificate(std::move(cert));
auto certMgr = std::make_shared<fizz::client::CertManager>();
certMgr->addCert(std::move(cert));
ctx->setClientCertManager(std::move(certMgr));
}
std::shared_ptr<fizz::DefaultCertificateVerifier> verifier;
if (!pemCaPath.empty()) {
Expand Down

0 comments on commit 9beaf18

Please sign in to comment.