Closed
Bug 1488511
Opened 7 years ago
Closed 7 years ago
Final verification on ESR 60.2.0 failed due to case mismatch in parsing http headers.
Categories
(Release Engineering :: Release Automation, enhancement)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: tomprince, Unassigned)
Details
Attachments
(1 file)
https://hg.mozilla.org/build/tools/file/8b3d4005593e/release/test-mar-url.sh#l23 checks headers case-insensitively, but it appears that cloud front is now returning that header lowercase.
|
Reporter | |
Comment 1•7 years ago
|
||
|
|
Reporter | |
Comment 2•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
|
||
Comment 3•7 years ago
|
||
Comment on attachment 9006311 [details]
Bug 1488511: [final-verification] Check content-length case insensitively; r=jlorenzo
I gave r+ on IRC before the patch landed
Attachment #9006311 -
Flags: review+
|
|
||
Comment 4•7 years ago
|
||
As a follow up, we should also make [1] case insensitive. What do you think, Tom?
[1] https://hg.mozilla.org/build/tools/file/8b3d4005593e/release/test-mar-url.sh#l24
Flags: needinfo?(mozilla)
|
|
Reporter | |
Comment 5•5 years ago
|
||
That might make sense, but we are rewriting this in python, so probably not worth it at this point.
Flags: needinfo?(mozilla)
|
||
Updated•1 year ago
|
Component: Release Automation: Updates → Release Automation
You need to log in
before you can comment on or make changes to this bug.
Description
•