[dovecot/core] 4102d9: lib-http: test-http-client-errors: Always use the ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/dovecot/core
  Commit: 4102d9f269837affa671a46fe047290366768ad1
      https://github.com/dovecot/core/commit/4102d9f269837affa671a46fe047290366768ad1
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/test-http-client-errors.c

  Log Message:
  -----------
  lib-http: test-http-client-errors: Always use the proper error status code definitions rather than an integer literal.


  Commit: a8741f46cb3cf40e58e3d47b263f238918243380
      https://github.com/dovecot/core/commit/a8741f46cb3cf40e58e3d47b263f238918243380
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-client.h
    M src/lib-http/http-response.h

  Log Message:
  -----------
  lib-http: Explicitly define the start of the range of internal response status codes.


  Commit: 17908b11e2e0cece58f0b653876385da5d62f3c0
      https://github.com/dovecot/core/commit/17908b11e2e0cece58f0b653876385da5d62f3c0
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-response.h

  Log Message:
  -----------
  lib-http: Added a function to obtain a message suitable for logging from a response object.

This prevents logging internal error codes (>= 9000).


  Commit: ce3fc9190e82d8d9e9604afd4ebbee1d61957764
      https://github.com/dovecot/core/commit/ce3fc9190e82d8d9e9604afd4ebbee1d61957764
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/auth/auth-policy.c
    M src/lib-http/http-client-connection.c
    M src/plugins/fts-solr/solr-connection.c
    M src/plugins/fts/fts-parser-tika.c
    M src/plugins/push-notification/push-notification-driver-ox.c

  Log Message:
  -----------
  global: Use http_response_get_message() to log HTTP responses.


  Commit: 905ada5e56cbad1a3ccf827332dabd4898ea8fc5
      https://github.com/dovecot/core/commit/905ada5e56cbad1a3ccf827332dabd4898ea8fc5
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/test-http-client-errors.c

  Log Message:
  -----------
  lib-http: test-http-client-errors: Added test for using https:// while SSL is not configured.


  Commit: 15fe3500f08cfb1d4bde1f97afd7d287486dbd21
      https://github.com/dovecot/core/commit/15fe3500f08cfb1d4bde1f97afd7d287486dbd21
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-client-request.c

  Log Message:
  -----------
  lib-http: client: Clear request pointer immediately in http_client_request_error().


  Commit: 8d845733408c0cb06a8884d12101beb0d40e6869
      https://github.com/dovecot/core/commit/8d845733408c0cb06a8884d12101beb0d40e6869
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-client-request.c
    M src/lib-http/http-client.c

  Log Message:
  -----------
  lib-http: client: Restructured the handling of delayed errors.

The req->delayed_error* fields are cleared before the callback is called and before the request is dereferenced.
This way, freeing the request never removes it from the delayed request list when the list is being processed.
It also makes it possible to resubmit the request from the callback (mainly for future use).


  Commit: 7a358ecd8918d1280a0b68cdf348c23f6721c799
      https://github.com/dovecot/core/commit/7a358ecd8918d1280a0b68cdf348c23f6721c799
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-client-request.c

  Log Message:
  -----------
  lib-http: client: Make sure a request's log label never gets a duplicate request target part.

Make sure the URL part of the label always only consists of a http://host:port part and nothing more.


  Commit: e46130f48a6962734bfde7e71906d3894bedf45b
      https://github.com/dovecot/core/commit/e46130f48a6962734bfde7e71906d3894bedf45b
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2017-03-22 (Wed, 22 Mar 2017)

  Changed paths:
    M src/lib-http/http-client-request.c
    M src/lib-http/http-client.h
    M src/lib-http/test-http-client-errors.c

  Log Message:
  -----------
  lib-http: client: Implemented http_client_request_url_str() function that accepts an URL string rather than a pre-parsed URL object.

If the provided HTTP URL is invalid, the callback with the error is called some time later from the ioloop.
This change also amends the test-http-client-errors test suite with a new test for this new feature.


Compare: https://github.com/dovecot/core/compare/89387c6f098b...e46130f48a69