Theo Van Dinter wrote (a while ago -- my VPS provider didn't like spamd running on my host and shut my server down without notice):
I'm pretty close to -1 on this change.

I am -1 on it. Looking at it again, I'm not sure WTH I was thinking sticking it there, it's hackish as hell.


 I don't want to see special
hacks put in for certain headers on a function which is supposed to return all
of the headers.  It'd be better to have this functionality in the caller,
either use get_all_headers() and parse out what you want,

Since there's more than a single use for it, I'd rather have a function that does it once and does it right, instead of having the similar slightly broken code all over the place. I'll clean this up tomorrow or Saturday.


or just request the
appropriate header array and use that.

That doesn't work, and is the reason for adding the code, since you need to have headers you know were added by internal hosts, not just the first header instance of a particular header.


Daryl

Reply via email to