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