Skip to content
This repository has been archived by the owner on Nov 8, 2023. It is now read-only.

config in main and server #562

Closed
wants to merge 5 commits into from
Closed

config in main and server #562

wants to merge 5 commits into from

Conversation

RekGRpth
Copy link
Contributor

No description provided.

@RekGRpth RekGRpth mentioned this pull request Jun 30, 2021
Closed
wargio
wargio approved these changes Jun 30, 2021
View reviewed changes
Copy link
Contributor

@wargio wargio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wargio
Copy link
Contributor

wargio commented Jun 30, 2021

@rsp-eipi

@ffontaine
naxsi_src/naxsi_runtime.c: fix build without x_forwarded_for (nbs-sys…
71240db 
…tem#568)

x_forwarded_for is not available if realip, geo, geoip or proxy modules
aren't enabled resulting in the following build failure since version
1.1a and
nbs-system@07a056c:

/home/buildroot/autobuild/instance-3/output-1/build/nginx-naxsi-1.3/naxsi_src/naxsi_runtime.c: In function 'ngx_http_naxsi_data_parse':
/home/buildroot/autobuild/instance-3/output-1/build/nginx-naxsi-1.3/naxsi_src/naxsi_runtime.c:2846:20: error: 'ngx_http_headers_in_t' has no member named 'x_forwarded_for'
   if (r->headers_in.x_forwarded_for.nelts >= 1) {
                    ^

Fixes:
 - http://autobuild.buildroot.org/results/cdbc1536f6b5de3d4c836efa2f0dcaf0cdbb1462

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@wargio wargio mentioned this pull request Oct 13, 2021
Open
@mremande mremande closed this Nov 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@wargio wargio wargio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@RekGRpth @wargio @mremande @rapenne-s @ffontaine