Skip to content

Passthrough copies verify md5 sums#359

Merged
xbrianh merged 1 commit intomasterfrom
xbrianh-passthrough-md5
Sep 24, 2021
Merged

Passthrough copies verify md5 sums#359
xbrianh merged 1 commit intomasterfrom
xbrianh-passthrough-md5

Conversation

@xbrianh
Copy link
Copy Markdown
Member

@xbrianh xbrianh commented Sep 24, 2021

Use md5 checksum verification for passthrough copies.

In the current context of TNU, these are copies from URLBlob to GSBlob. This will soon also cover URLBlob -> AzureBlob, see #356

depends on #357 #358

@xbrianh xbrianh self-assigned this Sep 24, 2021
Copy link
Copy Markdown
Member

@DailyDreaming DailyDreaming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@xbrianh xbrianh force-pushed the xbrianh-passthrough-md5 branch from 52d1ca9 to 3791e2d Compare September 24, 2021 21:20
Base automatically changed from xbrianh-url-md5 to master September 24, 2021 21:42
@xbrianh xbrianh force-pushed the xbrianh-passthrough-md5 branch from 3791e2d to 34e6990 Compare September 24, 2021 21:43
Passthrough copies occur for URLBlob -> GSBlob, and in the future
for URLBlob -> AzureBlob. Since md5 sums are expected to be
available for all DRS responses containing Signed URLs.
@xbrianh xbrianh force-pushed the xbrianh-passthrough-md5 branch from 34e6990 to b016aac Compare September 24, 2021 22:33
@xbrianh xbrianh merged commit f326afb into master Sep 24, 2021
@xbrianh xbrianh deleted the xbrianh-passthrough-md5 branch September 24, 2021 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants