diff --git a/sdk/keyvault/azure-security-keyvault-certificates/inc/azure/keyvault/certificates/certificate_client.hpp b/sdk/keyvault/azure-security-keyvault-certificates/inc/azure/keyvault/certificates/certificate_client.hpp index c7f28e893..a1d8257a6 100644 --- a/sdk/keyvault/azure-security-keyvault-certificates/inc/azure/keyvault/certificates/certificate_client.hpp +++ b/sdk/keyvault/azure-security-keyvault-certificates/inc/azure/keyvault/certificates/certificate_client.hpp @@ -165,11 +165,13 @@ namespace Azure { namespace Security { namespace KeyVault { namespace Certificat * * @remark This operation requires the certificates/setissuers permission. * + * @param issuerName The name of the issuer. * @param certificateIssuer The certificate issuer. * @param context The context for the operation can be used for request cancellation. * @return CertificateIssuer instance. */ Azure::Response UpdateIssuer( + std::string const& issuerName, CertificateIssuer const& certificateIssuer, Azure::Core::Context const& context = Azure::Core::Context()) const; diff --git a/sdk/keyvault/azure-security-keyvault-certificates/src/certificate_client.cpp b/sdk/keyvault/azure-security-keyvault-certificates/src/certificate_client.cpp index 641955d07..a360451e6 100644 --- a/sdk/keyvault/azure-security-keyvault-certificates/src/certificate_client.cpp +++ b/sdk/keyvault/azure-security-keyvault-certificates/src/certificate_client.cpp @@ -188,19 +188,19 @@ Azure::Response CertificateClient::CreateIssuer( } Azure::Response CertificateClient::UpdateIssuer( + std::string const& issuerName, CertificateIssuer const& certificateIssuer, Azure::Core::Context const& context) const { - std::string name = certificateIssuer.Name; auto payload = CertificateIssuerSerializer::Serialize(certificateIssuer); Azure::Core::IO::MemoryBodyStream payloadStream( reinterpret_cast(payload.data()), payload.size()); - auto request - = CreateRequest(HttpMethod::Patch, {CertificatesPath, IssuersPath, name}, &payloadStream); + auto request = CreateRequest( + HttpMethod::Patch, {CertificatesPath, IssuersPath, issuerName}, &payloadStream); auto rawResponse = SendRequest(request, context); - auto value = CertificateIssuerSerializer::Deserialize(name, *rawResponse); + auto value = CertificateIssuerSerializer::Deserialize(issuerName, *rawResponse); return Azure::Response(std::move(value), std::move(rawResponse)); } diff --git a/sdk/keyvault/azure-security-keyvault-certificates/test/ut/certificate_client_test.cpp b/sdk/keyvault/azure-security-keyvault-certificates/test/ut/certificate_client_test.cpp index b72df71c8..6ebd543bb 100644 --- a/sdk/keyvault/azure-security-keyvault-certificates/test/ut/certificate_client_test.cpp +++ b/sdk/keyvault/azure-security-keyvault-certificates/test/ut/certificate_client_test.cpp @@ -294,7 +294,7 @@ TEST_F(KeyVaultCertificateClientTest, UpdateIssuer) { issuer.Credentials.Password = "password2"; - auto result = client.UpdateIssuer(issuer); + auto result = client.UpdateIssuer(issuer.Name, issuer); CheckIssuers(result.Value, issuer); }