Build fixes

- Add -I$(top_builddir) to documentation scanner CFLAGS (fixes "make
  distcheck" run from an out-of-tree build)

Use an m4/ directory for aclocal files

Add automake 1.11 silent-rules support

Use guchar instead of char where the distinction is important.

Noticed by -Wextra

Remove a redundant check

Clarify that an ignored return value is intentional, to appease fortify

http://bugzilla.gnome.org/show_bug.cgi?id=591226

Add some g_return_if_fail()s to soup-headers.c

(partly from "cache" branch)

soup-session: fix typo in documentation

Add missing soup-proxy-uri-resolver.h to soup.h

Mirror Mozilla behavior for when to follow Content-Length

Mozilla only honors content length on messages that use keep-alive to
work around servers that send broken Content-Length headers. This patch
mirrors that behavior.

An example for such a page is
http://sourceforge.net/apps/wordpress/sourceforge

Re-fix _SOUP_ATOMIC_INTERN_STRING to not cause strict-aliasing warnings

http://bugzilla.gnome.org/show_bug.cgi?id=588771

2.27.5

Don't crash when a request is redirected to an invalid/non-http URI

Make redirect_handler() reject such responses, and also make soup-uri
g_return_if_fail rather than crashing. Add a regression test.

http://bugzilla.gnome.org/show_bug.cgi?id=528882

Do not discard the request body after sending, even if !accumulate

We can't discard the request body until the message is completely
finished, because it may be necessary to send it a second time due to
redirect, authentication, or premature connection close.

In particular, if the request body did get discarded, then when
soup-message-io tried to send it the second time, it would see a non-0
content-length but no body chunks, and so would assume that the
application was intending to stream the body, and so would pause it
and wait for the first chunk. Since the application was not actually
intending to stream the request, this would result in the message
getting stuck forever.

http://bugzilla.gnome.org/show_bug.cgi?id=584645

Build fixes

- Remove acinclude.m4, which just hosted a stale copy of
  GTK_DOC_CHECK, which in turn broke "make distcheck" when used in
  combination with the gtk-doc.make installed by newer gtkdocize.

- Fix #include in soup-path-map.h.

Allow version 1 CA certificates in clients

This is done by setting the GNUTLS_VERIFY_ALLOW_X509_V1_CA_CRT
flag while setting up the credentials and parsing the CA certificate
file. This is necessary because without it, some releases of gnutls
reject certificates that are currently in use by major sites, including
Google.

Setting the flag is safe in libsoup because the file is meant to
contain only CA certificates. The main weakness around version 1
certs is that personal certificates could be mistaken for CA certs,
which shouldn't happen if libsoup is used as intended.

Servers don't have the possibility to verify their clients, so this
problem doesn't apply to them.

http://bugzilla.gnome.org/show_bug.cgi?id=589323

Fix cut+pasto in win32 gnutls code

2.27.4

distcheck fixes for sniffing-test

Make gcc happy with body handling in sniffing-test

Fix HTML or feed algorithm to not go past resource_length

Fix SSL on Windows

The connection timeout changes had broken SSL on Windows because they
used read/write rather than recv/send.

Based on a patch from Fridrich Strba.
http://bugzilla.gnome.org/show_bug.cgi?id=587910

Make SOUP_SESSION_TIMEOUT also affect async connection

http://bugzilla.gnome.org/show_bug.cgi?id=588177

Fix a content-sniffer crash with redirected URLs

SoupMessageHeaders was not properly cleaning up the cached
Content-Type when a request was redirected, which meant that if the
redirected-to URL had no Content-Type header, then
soup_message_headers_get_content_type() would return the Content-Type
of the original request, but
soup_message_headers_get_one("Content-Type") would return NULL,
causing the sniffer to crash.

http://bugzilla.gnome.org/show_bug.cgi?id=588335

Miscellaneous content-sniffing fixes

Remove the should_sniff_content flag; if there is a sniffer, we always
sniff.

Clean up soup-message-io a bit to require fewer special cases, and fix
a few potential leaks/crashes where it wasn't dealing with
IF_CANCELLED_OR_PAUSED correctly. (Although this looks like a large
patch, it actually results in a much smaller diff relative to 2.26.

Fix tests/sniffing-test to do chunked encoding correctly, make sure to
return data in chunks that are smaller than the sniffing buffer size,
and make resources/mbox larger than the sniffing buffer size so it
takes multiple reads to get enough data (when using chunked encoding).
Also add tests for empty response bodies (from Gustavo).

http://bugzilla.gnome.org/show_bug.cgi?id=587907

Actually test chunked encoding, and fix some hangs

The test was not really testing chunked encoding, because of incorrect
handling of the query string in the SoupURI, now it does. This has
exposed a couple hangs.

This change also avoids hanging on chunked encoding messages on the
sniffing test by calling soup_message_body_complete(), and by
sanitizing IO read state and read_length inconditionally when we reach
got_body, during message IO.

Fix the SOUP_METHOD_* and SOUP_URI_SCHEME_* defines to be threadsafe

Also fix/optimize a few other things in URI scheme canonicalization

Implement content sniffing

The implementation is based on the draft spec on Content-Type
Processing Model (draft-abarth-mime-sniff-01). It is a spinoff from
the HTML5 spec.

Soup now provides a SoupContentSniffer session feature, which hooks
into the message I/O, and delays emissions of the got-chunk signal to
be able to figure out the Content-Type of messages from the actual
content received, in some cases.

GIO is also used to sniff content, whenever the spec allows further
sniffing.

http://bugzilla.gnome.org/show_bug.cgi?id=572589

Require explicit --disable-ssl to build without GNUTLS support

In general, if someone is trying to build without GNUTLS installed
it's an accident, so bail out with an error.

Inspired by http://bugzilla.gnome.org/show_bug.cgi?id=584955

Fix POST 301 redirect behavior

We were doing POST->GET translation on 302 and 303, but we need to do
it on 301 as well to be compatible.

http://bugzilla.gnome.org/show_bug.cgi?id=586692

2.27.2

Remove -r flag from tests/get

The requirement that it imposed--that we "guess" the name of the index
file when fetching a URL that ended with "/"--is more annoying than
the usefulness of being able to test multiple downloads at once.

add a HACKING file with some initial notes on code style and correctness

Add "Content-Length: 0" to requests with a Content-Type but no body

Some servers get annoyed if a POST/PUT declares a Content-Type but
contains no body, so make sure to send a 0-length body (which is
different) instead.

Implement SoupProxyURIResolver, to replace SoupProxyResolver

Simplifies implementations, allows for non-http proxy resolution, and
allows authentication information to be passed.

http://bugzilla.gnome.org/show_bug.cgi?id=580051

SoupConnection/SoupSession reorg and cleanup

Move CONNECT handling from SoupConnection to SoupSession so it works
more like other requests. Generally clean up and simplify
SoupConnection, and simplify the coordination between SoupSession and
its subclasses.

Fix hostname resolution behavior

Previously we went to some effort to resolve the message URI hostname
to an IP address before figuring out proxies/connections, but this
turns out to be wrong for multiple reasons:

  1. Some hosts that send all requests via proxy don't even have a
     working DNS config.
     (http://bugzilla.gnome.org/show_bug.cgi?id=577532)

  2. Apparently no one expects hostnames in requests to be matched
     against IP addresses in proxy ignore lists anyway.

  3. The big web browsers all implement connection limits on a
     per-hostname basis, not a per-IP basis, and some web pages take
     advantage of this by using multiple aliases for the same host
     to get around the connection limit.

Also update tests/proxy-test to verify that the hostname is not
resolved when passing a request to a proxy.

More temporary SSL tweaking

Disable TLS 1.0 too, since some sites don't even support that (I'm
looking at you, PayPal). Eventually we'll want to make this be more
clever, and have it first try TLS 1.1 with extensions, and then fall
back to SSL 3.0 if the server gets confused by the new stuff. But that
will take some rewriting.

Include the (optional) algorithm param in Digest auth responses

Apple's CalDAV server apparently considers it mandatory.
http://bugzilla.gnome.org/show_bug.cgi?id=583091

Use POSIXly-correct "test a = b" rather than bash "test a == b"

Stupid bash. http://bugzilla.gnome.org/show_bug.cgi?id=583911

Update REQUIRED_AUTOMAKE_VERSION to match reality

http://bugzilla.gnome.org/show_bug.cgi?id=583942

Don't emit "authenticate" until a message is actually sent and fails.

In some cases, we're reasonably sure it's going to fail if we don't
emit authenticate, but forcing callers to deal with both the
pre-sending and post-sending cases in their authenticate handlers
makes writing them tricky. (In particular, calling
soup_message_io_pause() in the pre-sending case won't work, since the
message hasn't actually started yet.) This also solves the problem
that we were previously emitting "authenticate" both before and after
the message was sent in some cases.

Add a regression test for this case (written by Gustavo Noronha, who
figured out the circumstances where the bug was happening), and tweak
the existing regression tests to match the new rules.

http://bugzilla.gnome.org/show_bug.cgi?id=583462

Temporarily redo the TLS compatibility patch

It broke tests/ssl-test on machines that support %SSL3_RECORD_VERSION.
For now, we will just not use that flag, and instead always disable
TLS 1.1. The gnutls problem has been reported upstream.

Don't set AI_CANONNAME when calling getaddrinfo()

We weren't using the information it generated, so it had no effect
besides slowing things down with additional DNS traffic. Pointed out
by Christophe Gillette on libsoup-list.

Fix a warning introduced in fix for bug 582002

Fix cflags to work with older versions of gcc

Check that gcc supports -Wmissing-include-dirs and -Wundef rather than
using them unconditionally.

http://bugzilla.gnome.org/show_bug.cgi?id=578851

Fix the header/body boundary detection logic

The change to support LF LF in addition to CRLF CRLF broke this subtly
so that if the first network read returned a packet that ended at a
CR, then the following LF would be mistakenly interpreted as ending
the headers.

http://bugzilla.gnome.org/show_bug.cgi?id=582002

Attempt better compatibility with non-TLS1.1 https servers.

Try to use the (not-yet-widely-deployed) SSL3_RECORD_VERSION option in
gnutls to make it handshake according to the backward-compatibility
recommendations in the TLS spec. If that fails, fall back to just
disabling TLS1.1. Should hopefully help us to work better with servers
that freak out if you even suggest using TLS1.1.

Fix based on a Pidgin patch from Martin von Gagern.

http://bugzilla.gnome.org/show_bug.cgi?id=581342

Add another missing G_GNUC_NULL_TERMINATED and a G_GNUC_PRINTF

Mark soup_uri_set_query_from_fields with G_GNUC_NULL_TERMINATED

http://bugzilla.gnome.org/show_bug.cgi?id=581754

Handle PUT methods in soup_form_request_for_data

http://bugzilla.gnome.org/show_bug.cgi?id=581860

NEWS: fix a bug number that was typoed

2.27.1

Fix typo in documentation: TIME should be TYPE.

Fix a crash when cancelling a message from a "restarted" handler

SoupSessionAsync was mistakenly leaving its own "restarted" handler connected
even after "finished" was emitted. misc-test was supposed to catch that, but
it was using SoupSessionSync rather than Async. Changed it to use Async since
Async adds some signal handlers of its own, while Sync does not.

Fixes #380193, reported by Gustavo Noronha

Fix mbox to be a mailto

Verify that soup_auth_is_for_proxy() is returning the right thing

Belatedly move Changelog to Changelog.pre-git, update for that

Add a missing file to .gitignore

header-parsing.c: Add tests for new bad-header-parsing rules

Add an "expect_warning" variable to the test framework

This lets a test tell the log handler that it is expecting to cause a
warning, so it shouldn't be treated as an error

soup_message_headers_append: require @name to be non-empty

Bug 579318 – stupid servers that return garbled headers

soup_headers_parse: don't pass bad header lines to
soup_message_headers_append, since it will emit warnings in that case
now. Also, don't bail out because one line of the headers is bad.
Instead, just skip the bad line and continue with the next good line.

Mark soup_message_headers_get() deprecated

(in favor of soup_message_headers_get_one() and _get_list())

Add more tests to tests/date.c

Conversion between formats
non-UTC timestamps
floating timestamps

Fix soup_date_to_string() to deal with non-UTC times correctly

soup_date_new_from_string: fix setting of UTC vs floating for ISO8601
and RFC2822 timestamps. Allow a time of "24:00:00" (per ISO).

soup_date_to_string: coerce @date to UTC for HTTP and cookie dates,
output correct offset for RFC2822, ISO compact, and ISO full.

Bug 579055 – SOUP_DATE_RFC2822 not handled in soup_date_to_string()

Add a regression test for SOUP_SESSION_TIMEOUT

Bug 578928 – SOUP_SESSION_TIMEOUT does not work with https

soup-gnutls.c: Fix error-code logic to return G_IO_STATUS_AGAIN even
with synchronous connections, if the underlying socket operation
returns EAGAIN.

Bug 574414 – Make SOUP_SESSION_TIMEOUT work with SoupSessionAsync

soup-socket.c: when adding an IO watch to wait for an async socket to
become readable/writable, add a timeout as well, and fail if the
timeout times out.

soup-connection.c: Pass the TIMEOUT flag to async sockets too

Add .gitignore

Add libsoup.doap

2.26.1

* configure.in: 2.26.1

* NEWS: update

svn path=/trunk/; revision=1275

2.26.1

* configure.in: 2.26.1

* NEWS: update

svn path=/trunk/; revision=1274

fix use of async context here

* libsoup/soup-proxy-resolver-gnome.c (get_proxy_async): fix
use of async context here

svn path=/trunk/; revision=1273

Bug 578809 – warnings in soup_address_equal_by_ip

* libsoup/soup-session.c (get_host_for_message): don't try to look
up unresolved addresses in the hosts hash; just return NULL. The
"cancel other messages for the bad host" code in connect_result()
will loop over the whole queue, including messages with unresolved
addresses.

svn path=/trunk/; revision=1272

Bug 578746 – http_proxy env var set to ":80" in session
despite proxy mode "none"

* libsoup/soup-proxy-resolver-gnome.c (update_proxy_settings):
Don't set anything if proxy_mode is NONE, and properly ignore the
proxy host if it's empty.

svn path=/trunk/; revision=1271

Bug 578645 - crash in Rhythmbox Music Player

* libsoup/soup-proxy-resolver-gnome.c
(soup_proxy_resolver_gnome_init): unlock id.lock before freeing
it. (Fixes the case where SoupProxyResolverGNOME is first
initialized from a thread other than the main thread, while the
default main loop is running.)

svn path=/trunk/; revision=1270

2.26.0.9, aka "I can't believe it's not 2.26.1!", a pre-release to give

* configure.in: 2.26.0.9, aka "I can't believe it's not 2.26.1!",
a pre-release to give the proxy resolver fix a bit of extra
testing before next week.

* NEWS: Update

svn path=/trunk/; revision=1268

Re-fix GNOME proxy resolution (qv bug 571527)

* libsoup/soup-proxy-resolver-gnome.c: New and improved GNOME
proxy resolver; gets information out of GConf in a thread-safe
manner, and then passes it on to libproxy via environment
variables, so that libproxy won't try to access GConf itself, but
we still can use it for PAC, WPAD, and ignore_hosts.

* libsoup/soup-proxy-resolver-gconf.c:
* libsoup/soup-proxy-resolver-libproxy.c: gone now

* libsoup/soup-gnome-features.c: update for the fact that
SoupProxyResolverGNOME is actually a real type now, not
compile-time-defined alias

svn path=/trunk/; revision=1267

Fix ISO 8601 parsing to accept either "." or ","

* libsoup/soup-date.c (parse_iso8601_date): accept either "." or
"," as decimal separator, per the spec. qv glib bug 578369.

svn path=/trunk/; revision=1265

Don't allow CR/LF in header names or values

* libsoup/soup-message-headers.c (soup_message_headers_append):
Don't let the caller create a header with whitespace or ":" in its
name, or with CR or LF in its value, since that would result in us
generating syntactically invalid headers.

svn path=/trunk/; revision=1264

Bug 577386 – Fails to handle HTTPS redirect from a certain site.

* libsoup/soup-gnutls.c (soup_gnutls_read): Treat abnormal
EOFs as though they were normal, rather than as errors. Shrug.

Reported by Diego Escalante Urrelo.

svn path=/trunk/; revision=1263

Bug 577360 – handle cookies with same name but different path

* libsoup/soup-cookie-jar.c (soup_cookie_jar_add_cookie): check
both name and path when matching cookies; "foo=one; path=/bar"
should not replace "foo=two; path=/". They are separate cookies.

Reported by Alexander V. Butenko.

svn path=/trunk/; revision=1262

Fix the -p option in tests/get again

svn path=/trunk/; revision=1261

Bug 577630 - libsoup should ignore broken Content-Type headers

* libsoup/soup-message-headers.c (content_type_setter): Ignore
Content-Type if it's not at least minimally syntactically correct.
(soup_message_headers_get_content_type): Document that.

* tests/header-parsing.c (do_content_type_tests): test it.

svn path=/trunk/; revision=1260

Bug 577728 – soup_header_g_string_append_param should handle NULL values

* libsoup/soup-headers.c (soup_header_g_string_append_param):
allow @value to be %NULL.

* tests/header-parsing.c (do_append_param_tests): test
soup_header_g_string_append_param()

svn path=/trunk/; revision=1259

Bug 576760 – soup_message_headers_get_content_type returns bad headers

* libsoup/soup-message-headers.c (soup_message_headers_get_one)
(soup_message_headers_get_list): New replacements for
soup_message_headers_get(), indicating explicitly whether the
caller expects the header to be a list or not; for non-list-type
headers, if there's more than one, the second one should be
ignored rather than concatenated to the first.
(soup_message_headers_get): deprecate this.

* libsoup/*.c:
* tests/*.c: Update to use soup_message_headers_get_one() or
_get_list() as appropriate.

* tests/header-parsing.c (do_content_type_tests): Add some tests
of Content-Type parsing/setting, including making sure that
duplicate Content-Type headers are ignored.

* docs/reference/libsoup-2.4-sections.txt: update

svn path=/trunk/; revision=1258

Bug 576583 - Tests fail if "localhost" resolves to ::1

* tests/*.c: Use "127.0.0.1" in URIs rather than "localhost",
since the default SoupServer config only listen on IPv4 (qv bug
522519), and tests/httpd.conf.in only configures apache to listen
on IPv4, and we don't handle multiple IP addresses (qv bug 526321)
but the machine might be configured to return "::1" rather than
"127.0.0.1" first for "localhost". Patch from Andreas Rottmann.

svn path=/trunk/; revision=1257

Fix docs to not claim that the cookie domain can be %NULL. Add some

* libsoup/soup-cookie.c (soup_cookie_new): Fix docs to not claim
that the cookie domain can be %NULL. Add some g_return_if_fail()s.

svn path=/trunk/; revision=1256

clarify that SoupSessionAsync is not thread-safe, and that setting a

* docs/reference/client-howto.xml: clarify that SoupSessionAsync
is not thread-safe, and that setting a non-default GMainContext on
a non-threadsafe object means you can only use that object from
that GMainContext's thread.

svn path=/trunk/; revision=1255

Bug 574957 - soup-session-sync doesn't unlock mutex on proxy error

2009-03-26  Milan Crha  <mcrha@redhat.com>

Bug 574957 - soup-session-sync doesn't unlock mutex on proxy error

* libsoup/soup-session-sync.c: (wait_for_connection):
Unlock private lock also on proxy error.

svn path=/trunk/; revision=1254

Bug 571283 - Allow LF LF instead of CRLF CRLF as header/body separator

* libsoup/soup-message-io.c (read_metadata): Change how this
works; instead of taking a boundary string, just always pass "\n"
as the boundary to soup_socket_read_until(), but call it multiple
times if the caller wants us to read until a blank line.
(io_read): update to deal with the fact that the headers might be
terminated by LF LF rather than CRLF CRLF.

svn path=/trunk/; revision=1253

Bug 566530 - Handle (illegal) unencoded spaces in URIs

* libsoup/soup-uri.c (uri_normalized_copy): optionally fix up
parts with unencoded spaces in them.
(soup_uri_new_with_base): tell uri_normalized_copy() to fix up
spaces in the path and query components

* tests/uri-parsing.c (abs_tests): test parsing and unparsing a
URI with an unencoded space in it.

* tests/redirect-test.c (tests, server_callback): add a test of
redirecting to a URI with an unencoded space in it.

svn path=/trunk/; revision=1252

2.26.0

* configure.in: 2.26.0

* NEWS: Update

svn path=/trunk/; revision=1250

Add G_{BEGIN,END}_DECLS guards to public headers.

2009-03-14  Xan Lopez  <xan@gnome.org>

        Add G_{BEGIN,END}_DECLS guards to public headers.

        * libsoup/soup-auth.h:
        * libsoup/soup-cookie-jar-sqlite.h:
        * libsoup/soup-cookie-jar-text.h:
        * libsoup/soup-proxy-resolver.h:

svn path=/trunk/; revision=1249

add more detail to the doc comment

* libsoup/soup-session.c (soup_session_cancel_message): add more
detail to the doc comment

* libsoup/soup-message.c (SoupMessage): Note in the doc comment
that reason phrases are not very useful, and should not be
presented to the user.

* libsoup/soup-status.c: add a comment explaining why reason
phrases aren't localized. Also some misc doc fixes.
(soup_status_get_phrase): Note in the doc comment that you
shouldn't present reason phrases to the user.

svn path=/trunk/; revision=1248

Bug 571527 – gvfsd-http crashed with SIGSEGV in
g_hash_table_lookup().

As currently written, libproxy's gnome plugin causes intermittent
gvfsd-http crashes, and I didn't have time to come up with either
a fix or a workaround. So for 2.26.0 we will fall back to using
the GConf-based proxy resolver. Hopefully to be fixed for 2.26.1.

* configure.in: require both gconf and libproxy if building
--with-gnome

* libsoup/soup-proxy-resolver-gconf.c: bring this back.

* libsoup/soup-gnome-features.c
(soup_proxy_resolver_gnome_get_type): use gconf instead of
libproxy

* libsoup/Makefile.am: updates

svn path=/trunk/; revision=1247

gtk-doc updates. In particular, document a bunch of convenience #defines,

* libsoup/*.c: gtk-doc updates. In particular, document a bunch of
convenience #defines, and add "Since" tags where appropriate.

svn path=/trunk/; revision=1246

Process messages in the CONNECTING iostate before messages in the QUEUED

* libsoup/soup-session-async.c (run_queue): Process messages in
the CONNECTING iostate before messages in the QUEUED iostate.
Otherwise, if a bunch of messages are queued all at once, and the
server doesn't support persistent connections, some messages can
get stranded in the queue. Fixes #574365 (reported by Xan Lopez)
and removes a five-year-old FIXME...

svn path=/trunk/; revision=1245