Closed Bug 1488511 Opened 2 years ago Closed 2 years ago

Final verification on ESR 60.2.0 failed due to case mismatch in parsing http headers.

Categories

(Release Engineering :: Release Automation: Updates, enhancement)

enhancement
Not set
normal

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.
https://hg.mozilla.org/build/tools/rev/d76acbc873de
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
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+
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)

That might make sense, but we are rewriting this in python, so probably not worth it at this point.

Flags: needinfo?(mozilla)
You need to log in before you can comment on or make changes to this bug.