search

ccampbell / chromephp

class for logging PHP variables to Google Chrome console
http://www.chromelogger.com
1.38k stars 450 forks source link

Check HTTP header size #40

Open shashenko opened 10 years ago

okonomiyaki3000 commented 10 years ago

This looks like a good idea but where did you come up with the number 200,000? I have doubts about that number.

In fact, while it seems there is a limit on the size of the request header, it's not clear to me that there is any such restriction on response headers. Even if there is, it seems the restriction should be on the cumulative size of all headers, not just one of them.

Oxicode commented 8 years ago

cc @ccampbell

dulldusk commented 7 years ago

I´ve just posted a pull request with the solution for this problem. If the header exceeds 8k the user will reveive an error message from ChromePHP on the log, instead of an 500 Interval server error.

hopeseekr commented 5 years ago

This was addressed in the PR of my fork: phpexpertsinc/chromephp#2.