Ticket #449 (defect)
Opened 3 years ago
Last modified 3 years ago
HeaderMap elements method should always return a list
Status: closed (fixed)
| Reported by: | lawouach | Assigned to: | lawouach |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.2-final |
| Component: | CherryPy code | Keywords: | |
| Cc: |
Currently the elements method of the HeaderMap? class returns None when the header is not found. It would be easier and simpler to return an empty list instead of None.
Is there anyone using that code that would be annoyed by doing so?
Change History
01/25/06 16:12:44: Modified by fumanchu
01/26/06 08:51:34: Modified by lawouach
Fine. That's why I opened this ticket before doing it. I wanted to check no one would come and say "wait i am using that very feature".
I don't think that many people use it so I'll change that to an empty list for consitancy.
01/26/06 09:15:01: Modified by lawouach
- status changed from new to closed.
- resolution set to fixed.
Fixed in [945]
That's OK with me as long as nobody complains. I think my intent in the beginning was some vague idea to distinguish between an emtpy header and a missing header. Both RFC 2616 (HTTP/1.1) and RFC 822 specify that the header value can be completely blank, but that's rarely seen in practice.