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.

Sign up for GitHub

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

Jump to bottom

''' /tmp/ngx_brotli/filter/ngx_http_brotli_filter_module.c: In function 'ngx_http_brotli_header_filter': 136.9 /tmp/ngx_brotli/filter/ngx_http_brotli_filter_module.c:322:4: error: 'ngx_table_elt_t' has no member named 'next' 136.9 322 | h->next = NULL; 136.9 | ^~ #156

Closed
zorka007 opened this issue Sep 26, 2023 · 9 comments · Fixed by #157
Closed

''' /tmp/ngx_brotli/filter/ngx_http_brotli_filter_module.c: In function 'ngx_http_brotli_header_filter': 136.9 /tmp/ngx_brotli/filter/ngx_http_brotli_filter_module.c:322:4: error: 'ngx_table_elt_t' has no member named 'next' 136.9 322 | h->next = NULL; 136.9 | ^~ #156

zorka007 opened this issue Sep 26, 2023 · 9 comments · Fixed by #157

Comments

@zorka007
Copy link

No description provided.

@zorka007
Copy link
Author

Hello! Help)

@nadz-goldman
Copy link

same problem

@Maxistr1
Copy link

Have the same problem after last changes. Please help

@Cristobal-M
Copy link

Cristobal-M commented Sep 27, 2023 •
edited
Loading

Same problem, for the time being I'm using a previous commit 63ca02a

Building for nginx-1.18.0 in a docker container of Ubuntu 22.04

@jannis-t69
Copy link

Same problem for nginx-1.22.1

@wyattoday
Copy link
Contributor

Use the latest NGINX it has security fixes.

See the README for how to build NGINX and ngx_brotli.

@TechAdkey
Copy link

I have the same problem, when are you planning to fix this issue or give advice how to fix it

nedvna added a commit to wallarm/ingress that referenced this issue Sep 27, 2023
@nedvna
Temp workaround due to latest brotli changes
47222d4 
google/ngx_brotli#156
This was referenced Sep 28, 2023
Merged
Merged
xDmitriev pushed a commit to wallarm/ingress that referenced this issue Sep 28, 2023
DEVOPS-2206 Temp workaround due to latest brotli changes google/ngx_b…
2749008 
…rotli#156
@gbtec-ext-fernandoquiroga
Copy link

Same issue nginx 1.21.5

@davidkel davidkel mentioned this issue Sep 29, 2023
Closed
@strongjz strongjz mentioned this issue Oct 1, 2023
Merged
@marcelmfs
Copy link

I can confirm that building with nginx latest 1.25.2 the issue wasn't reproduced.
From 1.21.2 when it broke our CI this weekend.

@u5surf u5surf mentioned this issue Oct 4, 2023
Merged
@u5surf u5surf mentioned this issue Oct 9, 2023
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix build failed for nginx version branch u5surf/ngx_brotli
9 participants
@nadz-goldman @marcelmfs @Cristobal-M @wyattoday @Maxistr1 @gbtec-ext-fernandoquiroga @jannis-t69 @zorka007 @TechAdkey