-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does not compile with Nginx 1.21.1 #51
Comments
Hello, Is there any update on this? The compilation is failing with the following error:
The error should be related to the changes performed in the latest NGINX version `1.21.1: |
This one is outdated, see https://github.com/dvershinin/nginx_ajp_module |
Hi, |
In fact this behavior is inconsistent. You can just delete it. - if (r->quoted_uri || r->space_in_uri || r->internal) {
+ if (r->quoted_uri || r->internal) {
|
Can you issue a pull request? Thanks.
Jintao Zhang ***@***.***> 于2022年10月26日周三 05:01写道:
… Hi, same error and my solution with updated project: in file
ngx_http_ajp.c, line 139: *if (r->quoted_uri || r->space_in_uri ||
r->internal) {* replace with: *if (r->quoted_uri || r->plus_in_uri ||
r->internal) {* and rerun ./configure, make and make install working
correctly
In fact this behavior is inconsistent. You can just delete it.
- if (r->quoted_uri || r->space_in_uri || r->internal) {+ if (r->quoted_uri || r->internal) {
space_in_uri is to handle some bad clients, but it's not necessary now
—
Reply to this email directly, view it on GitHub
<#51 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABFRGNLVPOTLD27FN7QJNTWFBDDRANCNFSM5AAF6DPQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Sure! |
Compiler complains with:
The text was updated successfully, but these errors were encountered: