[BUG] Problem importing a certificate

[Courela]

Library name and version

Azure.Security.KeyVault.Certificates 4.6.0

Describe the bug

Hello,
I'm using Azure SDK to import a certificate to KeyVault. I'm using a method called
Azure.Security.KeyVault.Certificates.ImportCertificate

In here I see a call to SendRequest that takes CertificatesPath and a slash (/) as parameters, but CertificatesPath already has a slash at the end

azure-sdk-for-net/sdk/keyvault/Azure.Security.KeyVault.Certificates/src/CertificateClient.cs

Line 25 in 5de735f

internal const string CertificatesPath = "/certificates/";

making the endpoint being called like
/certificates//someName/import
with two slashes between /certificates and /import

Expected behavior

I would expect that the called endpoint be
/certificates/someName/import

Actual behavior

Endpoint called
/certificates//someName/import

Reproduction Steps

Create a CertificateClient and call ImportCertificate

this.certificateClient.ImportCertificate(importOptions);

Environment

OS: Windows 10
IDE: Visual Studio 17.9.6

Using mcr.microsoft.com/dotnet/sdk:6.0-alpine docker image to compile and mcr.microsoft.com/dotnet/aspnet:6.0-alpine docker image to run the app.

[github-actions]

Thank you for your feedback. Tagging and routing to the team member best able to assist.

[Courela]

I managed to compile the code and the CertificatesPath needs the ending slash for other methods, adding an extra slash in here

azure-sdk-for-net/sdk/keyvault/Azure.Security.KeyVault.Certificates/src/CertificateClient.cs

Line 896 in 5de735f

return _pipeline.SendRequest(RequestMethod.Post, importCertificateOptions, () => new KeyVaultCertificateWithPolicy(), cancellationToken, CertificatesPath, "/", importCertificateOptions.Name, "/import");

seems to be the problem.

It also happens here

azure-sdk-for-net/sdk/keyvault/Azure.Security.KeyVault.Certificates/src/CertificateClient.cs

Line 926 in 5de735f

return await _pipeline.SendRequestAsync(RequestMethod.Post, importCertificateOptions, () => new KeyVaultCertificateWithPolicy(), cancellationToken, CertificatesPath, "/", importCertificateOptions.Name, "/import").ConfigureAwait(false);