Skip to content

Conversation

@janbarasek
Copy link
Contributor

  • new feature
  • BC break? yes

If request was called as ajax json payload with some data but without 'Content-Type': 'application/x-www-form-urlencoded' header, Tracy can not dump sent values.

This change add automatic detection of PHP Input and render to HTTP request section.

Example:

Snímek obrazovky 2019-11-01 v 12 12 52

If entered data was short json, Tracy will detect it automatically and dump as array for bette readability.

More information (Czech language): https://php.baraja.cz/ajax-post

Thanks.

@lukaspijak
Copy link

Hi it's nice feature but this "JSON detection" is very dumb. I think it would be better to detect JSON using content type (application/json).

@JanTvrdik
Copy link
Contributor

You cannot simple use file_get_contents('php://input'), because the output may be too large.

@janbarasek
Copy link
Contributor Author

Thanks to everybody.

@lukaspijak Added detection by HTTP header Content-Type which must starts with application/json. For example: application/json;charset=UTF-8

@JanTvrdik Added limitation for 10000 input characters.

@dg dg force-pushed the master branch 6 times, most recently from de3ad52 to 191c0d2 Compare December 15, 2019 22:49
@dg dg force-pushed the master branch 2 times, most recently from 2b958bb to f36b649 Compare January 2, 2020 19:40
@dg dg force-pushed the master branch 2 times, most recently from 68b0ec8 to 344c772 Compare February 23, 2020 18:40
@dg dg force-pushed the master branch 15 times, most recently from 8ec7417 to c8f135c Compare April 14, 2020 06:14
@dg dg force-pushed the master branch 9 times, most recently from c3c8bf0 to 6f7e1c3 Compare January 22, 2021 23:21
@dg dg force-pushed the master branch 2 times, most recently from 84dee11 to bd73e33 Compare March 4, 2021 20:09
@dg dg force-pushed the master branch 8 times, most recently from 7799297 to a386d38 Compare April 29, 2021 11:09
@dg dg force-pushed the master branch 2 times, most recently from f3f04e0 to 0a90e93 Compare May 10, 2021 11:10
@dg dg force-pushed the master branch 6 times, most recently from 5ee33d7 to 8e708de Compare August 24, 2021 16:26
@KminekMatej
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants