validateManifest() should use HEAD for blobs.

bouncestorage / swiftproxy

Access other storage backends via the Swift API

Apache License 2.0

31 stars 2 forks source link

validateManifest() should use HEAD for blobs. #51

Closed timuralp closed 9 years ago

timuralp commented 9 years ago

validateManifest() uses GET on all objects, but only consumes the payload if it is a manifest. This could be problematic, as the server may hold resources expecting the client to consume the payload.

The commit changes the code to use GET only for manifest blobs and otherwise use HEAD.

kahing commented 9 years ago

Would e65ea9a31c973a4fb96354936755418343a1296a and 2938f400123bcf6b9dbbfa27b7eb2b5883271e1c solve this?

timuralp commented 9 years ago

@kahing yep, that works. Closing this PR.