azure-sdk-for-python icon indicating copy to clipboard operation
azure-sdk-for-python copied to clipboard

Replace get_object_hash with get_content_hash for all uploads

Open diondrapeck opened this issue 2 years ago • 1 comments

Description

Removes md5 get_object_hash that was being used for non-snapshot uploads and transitions all snapshots to use shas56 get_content_hash. This will allow increased security against tampering and reduce complexity by allowing all artifacts to follow the same upload process.

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • [x] The pull request does not introduce [breaking changes]
  • [x] CHANGELOG is updated for new features, bug fixes or other significant changes.
  • [x] I have read the contribution guidelines.

General Guidelines and Best Practices

  • [x] Title of the pull request is clear and informative.
  • [x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • [x] Pull request includes test coverage for the included changes.

diondrapeck avatar Nov 10 '22 17:11 diondrapeck

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-ai-ml

azure-sdk avatar Nov 10 '22 17:11 azure-sdk