<div class="__aliyun_email_body_block"><div  style="line-height:1.7;font-family:Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;color:#000000;"><div  style="clear:both;">OK,  I <span >I have another question about this.  '<span  style="color:#000000;font-family:Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:.0px;text-transform:none;white-space:normal;word-spacing:.0px;background-color:#ffffff;text-decoration-thickness:initial;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline;">NGX_MODULE_SIGNATURE_27</span>'<span > should be with NGX_HTTP_GZIP <span >as the following patch ?</span></span></span></div><div  style="clear:both;"><br ></div><div  style="clear:both;"><span ># HG changeset patch</span><div  style="clear:both;"># User BullerDu <yefei.dyf@alibaba-inc.com></div><div  style="clear:both;"># Date 1670204057 -28800</div><div  style="clear:both;">#      Mon Dec 05 09:34:17 2022 +0800</div><div  style="clear:both;"># Branch bugfix_dynamic_module</div><div  style="clear:both;"># Node ID 5e63c3cf514e64a5f6499b66613d4935bd026d5f</div><div  style="clear:both;"># Parent  0b360747c74e3fa7e439e0684a8cf1da2d14d8f6</div><div  style="clear:both;">Modules compatibility: degradation signature must be with NGX_HTTP_GZIP(Introduced by 3b522d7a5b34).</div><div  style="clear:both;">diff -r 0b360747c74e -r 5e63c3cf514e src/core/ngx_module.h</div><div  style="clear:both;">--- a/src/core/ngx_module.h     Thu Nov 24 23:08:30 2022 +0400</div><div  style="clear:both;">+++ b/src/core/ngx_module.h     Mon Dec 05 09:34:17 2022 +0800</div><div  style="clear:both;">@@ -149,12 +149,12 @@</div><div  style="clear:both;"> #if (NGX_HTTP_GZIP)</div><div  style="clear:both;"> #define NGX_MODULE_SIGNATURE_26  "1"</div><div  style="clear:both;">+#define NGX_MODULE_SIGNATURE_27  "1"</div><div  style="clear:both;"> #else</div><div  style="clear:both;"> #define NGX_MODULE_SIGNATURE_26  "0"</div><div  style="clear:both;">+#define NGX_MODULE_SIGNATURE_27  "0"</div><div  style="clear:both;"> #endif</div><div  style="clear:both;">-#define NGX_MODULE_SIGNATURE_27  "1"</div><div  style="clear:both;">-</div><div  style="clear:both;"> #if (NGX_HTTP_X_FORWARDED_FOR)</div><div  style="clear:both;"> #define NGX_MODULE_SIGNATURE_28  "1"</div><span > #else</span></div><div  style="clear:both;"><br ></div><div  style="clear:both;"><br ></div><div  style="clear:both;"><br /></div><blockquote  style="margin-right:0;margin-top:0;margin-bottom:0;font-family:Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;color:#000000;"><div  style="clear:both;">------------------------------------------------------------------</div><div  style="clear:both;">发件人：Maxim Dounin <mdounin@mdounin.ru></div><div  style="clear:both;">发送时间：2022年12月5日(星期一) 04:13</div><div  style="clear:both;">收件人：nginx-devel <nginx-devel@nginx.org></div><div  style="clear:both;">主　题：Re: Fixed gzip_disable_degradation defined without NGX_HTTP_DEGRADATION (broken by 3b522d7a5b34).</div><div  style="clear:both;"><br /></div>Hello!<br ><br >On Sat, Dec 03, 2022 at 11:16:44PM +0800, 杜叶飞(淮叶) via nginx-devel wrote:<br ><br >> I think gzip_disable_degradation needs NGX_HTTP_DEGRADATION in order to be consistent with where used. <br >> details: https://hg.nginx.org/nginx/rev/3b522d7a5b34 <https://hg.nginx.org/nginx/rev/3b522d7a5b34 ><br ><br >The revision you've linked explains why this "#if" is not really <br >needed even if we are concerned about saving these two bits in the <br >location configuration structure (and we aren't really concerned <br >anyway).<br ><br >Further, the patch you've suggested breaks binary compatibility <br >between nginx builds with and without the degradation module <br >without restoring appropriate flag in the binary signature.  This <br >is clearly incorrect behaviour which can result in segmentation <br >faults or other unexpected behaviour if modules compiled with <br >different assumptions are loaded into nginx.<br ><br >-- <br >Maxim Dounin<br >http://mdounin.ru/<br >_______________________________________________<br >nginx-devel mailing list -- nginx-devel@nginx.org<br >To unsubscribe send an email to nginx-devel-leave@nginx.org<br ></blockquote></div></div>