Actions
Story #14242
closed
[controller] Verify hash on manifest text when querying by PDH
Start date:
09/19/2018
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
Story points:
0.5
Release:
Release relationship:
Auto
Description
As came up in the sprint review, when sending out a query for a collection by PDH, controller should verify that the manifest text that is returned matches the PDH.
Updated by Peter Amstutz over 6 years ago
- Status changed from New to In Progress
Updated by Peter Amstutz over 6 years ago
- Description updated (diff)
- Story points set to 0.5
Updated by Lucas Di Pentima over 6 years ago
Reviewing 720e76bb1d82d5a5448ce395df634310ceee473e
This lgtm, but I think adding test cases would be convenient.
Updated by Peter Amstutz over 6 years ago
Lucas Di Pentima wrote:
Reviewing 720e76bb1d82d5a5448ce395df634310ceee473e
This lgtm, but I think adding test cases would be convenient.
Test added, also added check that the portable_data_hash record field is consistent with manifest_text hash and requested PDH.
Now @ 494cc8312cff496bab3653d5b2ab1f00ac43a40c
https://ci.curoverse.com/view/Developer/job/developer-run-tests/900/
Updated by Peter Amstutz over 6 years ago
- Status changed from In Progress to Resolved
Updated by Tom Morris
Actions