Azure / azure-sdk-for-cpp

This repository is for active development of the Azure SDK for C++. For consumers of the SDK we recommend visiting our versioned developer docs at https://azure.github.io/azure-sdk-for-cpp.
MIT License
181 stars 126 forks source link

Reduce duplication in opening and reading the pem file for cert content in ClientCertificateCredential #5990

Open ahsonkhan opened 1 month ago

ahsonkhan commented 1 month ago
          `ReadPemCertificate()` already does open the file and read its contents. Can you make it so that x5c gets read as part of that process, instead of opening the file for the second time and reading its contents again?

_Originally posted by @antkmsft in https://github.com/Azure/azure-sdk-for-cpp/pull/5988#discussion_r1759635775_

github-actions[bot] commented 1 month ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Jinming-Hu @microzchang @vinjiang.