Skip to content
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

Hi, I can complie It on apache 2.4.39 #1

Open
jhzao82 opened this issue Apr 14, 2019 · 9 comments
Open

Hi, I can complie It on apache 2.4.39 #1

jhzao82 opened this issue Apr 14, 2019 · 9 comments

Comments

@jhzao82
Copy link

jhzao82 commented Apr 14, 2019

Here is the error, How can fix that?

apxs -i -a -c mod_bw.c
/usr/lib64/apr15u-1/build/libtool --silent --mode=compile gcc -std=gnu99 -prefer-pic -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic -DLINUX -D_REENTRANT -D_GNU_SOURCE -pthread -I/usr/include/httpd -I/usr/include/apr15u-1 -I/usr/include/apr15u-1 -c -o mod_bw.lo mod_bw.c && touch mod_bw.slo
mod_bw.c: In function 'get_bw_rate':
mod_bw.c:567:59: error: 'conn_rec' has no member named 'remote_addr'
if (apr_ipsubnet_test(e[i].x.ip, r->connection->remote_addr)) {
^
mod_bw.c: In function 'get_maxconn':
mod_bw.c:658:59: error: 'conn_rec' has no member named 'remote_addr'
if (apr_ipsubnet_test(e[i].x.ip, r->connection->remote_addr)) {
^
mod_bw.c: In function 'get_sid':
mod_bw.c:709:59: error: 'conn_rec' has no member named 'remote_addr'
if (apr_ipsubnet_test(e[i].x.ip, r->connection->remote_addr)) {
^
apxs:Error: Command failed with rc=65536

@IvnSoft
Copy link
Owner

IvnSoft commented Apr 14, 2019 via email

@IvnSoft
Copy link
Owner

IvnSoft commented Apr 15, 2019 via email

@jhzao82
Copy link
Author

jhzao82 commented Apr 16, 2019

well done! but new patch looks not commited?

@IvnSoft
Copy link
Owner

IvnSoft commented Apr 16, 2019 via email

@jsainzvela
Copy link

Hola Ivan!

One question, did you ever commit this change?

I am facing the same problem as @jhzao82 was having with the latest master branch.

Thanks in advance!

Jorge

@sosherof
Copy link

A bit late for this reply but you can replace 'remove_addr' with 'client_addr' in the code and it should compile.

@IvnSoft
Copy link
Owner

IvnSoft commented Oct 25, 2021

Sorry guys for dissapearing.
I'll be testing and updating the mod with the latest versions this weekend.

@sosherof
Copy link

sosherof commented Feb 9, 2022

@IvnSoft Any luck getting the new version out?

@sosherof
Copy link

sosherof commented May 5, 2022

Bump. Still looking forward to the next version (maybe with some of my Windows enhancements included).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants