Opened 12 years ago

Closed 12 years ago

#1752 defect closed wontfix (wontfix)

t.w2's header tests that use runRoundtripTest and requiredGeneratedElements are invalid.

Reported by: David Reid Owned by: David Reid
Priority: normal Milestone:
Component: web2 Keywords:
Cc: Branch:


twisted.web2.test.test_http_headers.HeaderParsingTestBase.runRoundtripTest's requiredGeneratedElements functionality is invalid for any generator that produces a condensed single line comma seperated list of headers.

Change History (4)

comment:1 Changed 12 years ago by David Reid

Priority: normalhigh
Status: newassigned

comment:2 Changed 12 years ago by David Reid

Minddump follows.

This issue applies to multiple generated headers that could have elements repeated between several headers. such as WWW-Authenticate with multiple schemes for a single realm. I do not believe any other headers display similar behavior, and therefor no other header testcases are invalid. Since WWW-Authenticate's tests do not use runRoundtripTest instead a custom variation of runRoundtripTest it is not invalid in current trunk. We can't make WWW-Authenticate generate a single header because no browsers can parse it, and it wouldn't fix the problem anyway. While not entirely invalid all the unittests that use the 3rd table element (required generated elements) are wrong because they are not testing the header in it's entirety. This is possibly unrectifiable without using an ordered dictionary as many headers using that behavior are generated from dictionaries. We could require that a header with an odict be passed in for the generator and compare that string in it's entirety. Since it is probably wrong to rely on the output of the parser for anything other than making sure we can generate headers we parse.

comment:3 Changed 12 years ago by David Reid

Priority: highnormal

I need to reevaluate the applicability of this issue to the majority of test cases.

comment:4 Changed 12 years ago by David Reid

Resolution: wontfix
Status: assignedclosed

I haven't found this to affect any headers other than www-authenticate.

Note: See TracTickets for help on using tickets.