Home
last modified time | relevance | path

Searched hist:144 (Results 1 – 3 of 3) sorted by relevance

/unit/src/
H A Dnxt_http_route.cdiff 1785:144a42933ea7 Mon Feb 01 15:55:00 UTC 2021 Valentin Bartenev <vbart@nginx.com> Fixed building by GCC 10 with -flto and -O2.

This closes #467 issue on GitHub.
H A Dnxt_php_sapi.cdiff 714:615827e60462 Thu Jul 05 13:43:00 UTC 2018 Valentin Bartenev <vbart@nginx.com> PHP: fixed request body processing.

The implementation of module was based on the assumption that PHP reads request
body and headers in the particular order. For the POST request the body goes
before headers and vice versa for all other requests.

But as it appeared later, this order is unspecified and depends on many factors,
including the particular code of PHP application. Among other factors those
can affect ordering:

- presence of "Content-Type" header;
- "variables_order" php.ini setting;
- "enable_post_data_reading" php.ini setting;
- reading php://input by application;

and this list can be incomplete.

As a temporary workaround, request body now is always put before headers and it
is gracefully skipped whenever PHP wants to get headers.

This closes #144 issue on GitHub.
H A Dnxt_router.cdiff 714:615827e60462 Thu Jul 05 13:43:00 UTC 2018 Valentin Bartenev <vbart@nginx.com> PHP: fixed request body processing.

The implementation of module was based on the assumption that PHP reads request
body and headers in the particular order. For the POST request the body goes
before headers and vice versa for all other requests.

But as it appeared later, this order is unspecified and depends on many factors,
including the particular code of PHP application. Among other factors those
can affect ordering:

- presence of "Content-Type" header;
- "variables_order" php.ini setting;
- "enable_post_data_reading" php.ini setting;
- reading php://input by application;

and this list can be incomplete.

As a temporary workaround, request body now is always put before headers and it
is gracefully skipped whenever PHP wants to get headers.

This closes #144 issue on GitHub.
diff 144:ed2d8228a62a Wed Jul 12 17:32:00 UTC 2017 Max Romanov <max.romanov@nginx.com> Single allocation for app conf and name.