ArcticDB icon indicating copy to clipboard operation
ArcticDB copied to clipboard

Add support of providing CA path directory for Azure SDK's libcurl backend

Open phoebusm opened this issue 1 year ago • 5 comments

Reference Issues/PRs

https://github.com/man-group/ArcticDB/issues/514

What does this implement/fix? How does it work (high level)? Highlight notable design decisions.

Add support of providing CA path directory for Azure SDK's underlying libcurl. This PR includes two parts:

  1. Upgrade azure-cpp-sdk
  2. Corresponding changes and test for ArcticDB to allow user to update the above option

This PR will be depended by the feature of auto supplying Linux OS-default CA cert to Azure's libcurl backend.

Any other comments?

Checklist

Checklist for code changes...
  • [x] Have you updated the relevant docstrings and documentation?
  • [x] Is this contribution tested against all ArcticDB's features?
  • [x] Do all exceptions introduced raise appropriate error messages?
  • [x] Are API changes highlighted in the PR description?
  • [x] Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

phoebusm avatar Sep 18 '23 01:09 phoebusm

Pending corresponding patch for conda environment. Conda does not support patching sources locally

phoebusm avatar Sep 18 '23 14:09 phoebusm

https://github.com/Azure/azure-sdk-for-cpp/pull/4982 Awaiting above PR so that the pin of Azure C++ SDK on Conda can be updated

phoebusm avatar Sep 28 '23 16:09 phoebusm

Azure/azure-sdk-for-cpp#4982 Awaiting above PR so that the pin of Azure C++ SDK on Conda can be updated

PR merged. Awaiting Azure SDK pin being updated on conda feedstock

phoebusm avatar Sep 29 '23 16:09 phoebusm

https://github.com/Azure/azure-sdk-for-cpp/pull/4982#issuecomment-1741187849

...this change should be available on vcpkg at some time next week with the October release of the Azure SDK.

phoebusm avatar Sep 29 '23 16:09 phoebusm

This PR will be put on-hold as https://github.com/Azure/azure-sdk-for-cpp/pull/4982#issuecomment-1743808598 stated that the PR on Azure SDK won't be officially release until 10th Nov.

phoebusm avatar Oct 03 '23 09:10 phoebusm