commit c111387efe8dcfe642e2e14915ca676443f9500b
author Maxim Dounin <mdounin@mdounin.ru> Thu Jul 11 23:20:08 2019 +0300
committer Maxim Dounin <mdounin@mdounin.ru> Thu Jul 11 23:20:08 2019 +0300
tree badde98c15cbbd7f6455f9edd513e21ef866b1d2
parent 15560130bb6f1dee2d7bfedb818b47dbfd6d74e2

Perl: removed unneeded NGX_DONE test.

The NGX_DONE test in ngx_http_perl_handle_request() was introduced
in 1702:86bb52e28ce0, which also modified ngx_http_perl_call_handler()
to return NGX_DONE with c->destroyed.  The latter part was then
removed in 3050:f54b02dbb12b, so NGX_DONE test is no longer needed.

src/http/modules/perl/ngx_http_perl_module.c

diff --git a/src/http/modules/perl/ngx_http_perl_module.c b/src/http/modules/perl/ngx_http_perl_module.c
index 6d3be91..d06b43c 100644
--- a/src/http/modules/perl/ngx_http_perl_module.c
+++ b/src/http/modules/perl/ngx_http_perl_module.c
@@ -228,11 +228,6 @@
     ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
                    "perl handler done: %i", rc);
 
-    if (rc == NGX_DONE) {
-        ngx_http_finalize_request(r, rc);
-        return;
-    }
-
     if (rc > 600) {
         rc = NGX_OK;
     }