Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder Linux64 - Nginx (Mainline) Build #20

Results:

Build successful

SourceStamp:

Repositoryhttps://github.com/SpiderLabs/ModSecurity
Branchnginx_refactoring
Revision33b8760e87b7441142a431175d5b459245551314
Got Revision33b8760e87b7441142a431175d5b459245551314
Changes10 changes

BuildSlave:

Linux64

Reason:

The AnyBranchScheduler scheduler named 'all-any' triggered this build

Steps and Logfiles:

  1. fetch update ( 17 mins, 6 secs )
    1. stdio
  2. autogen (ModSec) autogen ( 1 mins, 52 secs )
    1. stdio
  3. configure (ModSec) configure ( 4 mins, 6 secs )
    1. stdio
  4. make (ModSec) Compilation warnings ( 8 mins, 23 secs )
    1. stdio
    2. warnings (2)
  5. download downloading to nginx.tar.gz ( 1 mins, 56 secs )
    1. - no logs -
  6. extract nginx tarball (nginx) extract - mainline ( 2 secs )
    1. stdio
  7. download_1 downloading to echo.tar.gz ( 10 secs )
    1. - no logs -
  8. extract echo tarball (nginx/echo) extract ( 1 secs )
    1. stdio
  9. configure nginx (nginx) configure ( 3 mins, 14 secs )
    1. stdio
  10. make nginx (nginx) Compilation ( 7 mins, 13 secs )
    1. stdio
  11. download_2 downloading to modsec.conf ( 3 secs )
    1. - no logs -
  12. download_3 downloading to nginx-tests.tar.gz ( 56 secs )
    1. - no logs -
  13. extract nginx regressin tarball (nginx/tests) extract ( 2 secs )
    1. stdio
  14. get_builddir property 'builddir' set ( 1 secs )
    1. stdio
    2. property changes
  15. nginx tests nginx tests failed ( 1 hrs, 1 mins, 59 secs )
    1. stdio
  16. download_4 downloading to check-coding_style.sh ( 1 secs )
    1. - no logs -
  17. coding style standalone Coding style (ModSec Standalone) warnings ( 51 secs )
    1. stdio
    2. warnings (3275)
  18. coding style nginx Coding style (ModSec nginx) ( 4 secs )
    1. stdio
  19. download_5 downloading to cppcheck.sh ( 2 secs )
    1. - no logs -
  20. static analisys standalone Static analisys (ModSec Standalone) warnings ( 11 mins, 34 secs )
    1. stdio
    2. warnings (674)
  21. coding style nginx_1 Static analisys (ModSec nginx) warnings ( 6 secs )
    1. stdio
    2. warnings (26)
  22. download_6 downloading to clean.sh ( 1 secs )
    1. - no logs -
  23. clean clean semaphores ( 1 secs )
    1. stdio
  24. unit test make test ( 13 mins, 45 secs )
    1. stdio
  25. unit test_1 unit tests ( 12 mins, 15 secs )
    1. stdio
  26. clean_1 clean semaphores ( 1 secs )
    1. stdio
  27. regression test regression [8086] ( 11 mins, 36 secs )
    1. stdio

Build Properties:

NameValueSource
branch nginx_refactoring Build
builddir /var/lib/buildbot/slaves/slave/Linux64_-_Nginx__Mainline_/build SetPropertyFromCommand Step
buildername Linux64 - Nginx (Mainline) Builder
buildnumber 20 Build
codebase Build
github_repo_name ModSecurity Builder
github_repo_owner SpiderLabs Builder
got_revision 33b8760e87b7441142a431175d5b459245551314 Git
project Build
repository https://github.com/SpiderLabs/ModSecurity Build
revision 33b8760e87b7441142a431175d5b459245551314 Build
scheduler all-any Scheduler
slavename Linux64 BuildSlave
warnings-count 3977 WarningCountingShellCommand
workdir /var/lib/buildbot/slaves/slave/Linux64_-_Nginx__Mainline_ slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Andrei Belov
  2. Felipe Zimmerle
  3. paulyang

Timing:

StartThu Jul 31 03:05:40 2014
EndThu Jul 31 09:04:29 2014
Elapsed5 hrs, 58 mins, 48 secs

All Changes:

:

  1. Change #28

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 93ab6c3f72ecf5dd79fe3ab7ca3b70b8c6bfe33e

    Comments

    nginx refactoring
    
    Refactoring on the nginx module, including:
     - Better handling larger posts;
     - Now using nginx echo module during the regression tests.
     - Better interacting with neginx chain rules
     - Separation of the request handling and content filters.
     - Better handling nginx sessions and resource counts to allow a
       more efficient garbage collector.
     - Handling both http/1.0 and 1.1, including keep-alive.
     - Tests are now capable to test nginx as a proxy or end-server.
     - Tested agains nginx 1.6 and 1.7.

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/misc/00-phases.t
    • tests/regression/misc/25-libinjection.t
    • tests/regression/nginx/conf/nginx.conf.template
  2. Change #29

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 15b5b7a2d57d90153d8c09248b96f02414b2754d

    Comments

    nginx: looking for segfaults on the regression test.
    
    If nginx segfaults it will return, warning that the test failed.

    Changed files

    • tests/run-regression-tests-nginx.pl
  3. Change #30

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 82e568429825d4341b57922d991c7a69eab47107

    Comments

    nginx: better dealing with chunked request body

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/nginx/conf/nginx.conf.template
  4. Change #31

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision d44158fde79edb05fd98dcf0233ddc77102443c0

    Comments

    nginx: copies the req body chain to be processed instead of move
    
    Add a check for the definition MOVE_REQUEST_CHAIN_TO_MODSEC, whenever it is
    set the chain will be moved into the brigade. If it was not set the chain
    will be only copied. Moving was causing segfaults on the following
    regression tests:
    
     #15 - SecRequestBodyInMemoryLimit
     #16 - SecRequestBodyInMemoryLimit (greater)
     #19 - SecRequestBodyLimitAction ProcessPartial (multipart/greater - chunked)
     (from: regression/config/10-request-directives.t)

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
  5. Change #32

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 28af5c0c9dd698ea805a81b175664d5eabb1a964

    Comments

    nginx: cosmetics: Splits lines longer than 80 characters

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
    • nginx/modsecurity/ngx_pool_context.h
  6. Change #33

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 764fa06182a8b194110a11c925dacc23d8d7ef1b

    Comments

    nginx: cosmetics: Removes trailing whitespace

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
  7. Change #34

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision e3583dea963ebdc93377c7ac34c72f21f1cbcb08

    Comments

    niginx: cosmetics: Changes CRLF to LF

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
  8. Change #35

    Category None
    Changed by paulyang <paulyang.infohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 9c1cb842ea9fb890c1e6ccab854a38756a38c7a6

    Comments

    Bugfix: add -P option in test script
    
    Otherwise nginx's installation directory could not be specified.
    
    Signed-off-by: paulyang <paulyang.inf@gmail.com>

    Changed files

    • tests/run-regression-tests-nginx.pl
  9. Change #36

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision dbc4021dfd76ed71f716152804390f0a7323596c

    Comments

    Removed unneeded and invalid initialization.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
  10. Change #37

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 33b8760e87b7441142a431175d5b459245551314

    Comments

    Obtain port from r->connection->local_sockaddr.
    
    This eliminates segfaults caused by unset (NULL) r->port_start
    and non-NULL r->port_end. In fact, r->port_start is always NULL,
    so it is useless to rely on this pointer.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c