Merge pull request #104 from p-alik/http-plugin

gearmand should release http client if request contains 'X-Gearman-Background: true' header Reviewed-by: https://github.com/bonnyci[bot]
gearman · May 18, 2017 · e3dbdda · e3dbdda
2 parents 4ee46f8 + c906a12
commit e3dbdda
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/libgearman-server/plugins/protocol/http/protocol.cc b/libgearman-server/plugins/protocol/http/protocol.cc
@@ -164,11 +164,15 @@ class HTTPtext : public gearmand::protocol::Context
 
     case GEARMAN_COMMAND_JOB_CREATED:
       {
-        gearmand_log_debug(GEARMAN_DEFAULT_LOG_PARAM,
-                           "Sending HTTP told to ignore packet: gearmand_command_t:%s", 
-                           gearman_strcommand(packet->command));
-        ret_ptr= GEARMAND_IGNORE_PACKET;
-        return 0;
+        if(this->_background) {
+          ret_ptr = GEARMAND_SUCCESS;
+        } else {
+          gearmand_log_debug(GEARMAN_DEFAULT_LOG_PARAM,
+                             "Sending HTTP told to ignore packet: gearmand_command_t:%s",
+                             gearman_strcommand(packet->command));
+          ret_ptr= GEARMAND_IGNORE_PACKET;
+          return 0;
+        }
       }
     }