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

now returns NULL if there is no Content-Type header; fix documentation to

2009-02-20  Gustavo Noronha Silva  <gns@gnome.org>

* libsoup/soup-message-headers.c
(soup_message_headers_get_content_type): now returns NULL if there
is no Content-Type header; fix documentation to account for this
change and to actually be correct about how the Content-Type is
returned.

svn path=/trunk/; revision=1244

Bug 572153 – SoupServer doesn't support SOUP_ENCODING_EOF

* libsoup/soup-message-io.c (io_write): Various fixes to make
SOUP_ENCODING_EOF work correctly when sending response
bodies. (Previously, the code assumed that SoupServer responses
would always be chunked or Content-Length-encoded.)

* libsoup/soup-message-client-io.c (get_request_headers): when
changing a request body from SOUP_ENCODING_NONE to
SOUP_ENCODING_CONTENT_LENGTH, return the new encoding value to
soup-message-io, not the old one.

* libsoup/soup-message.c (set_property): when setting
priv->server_side to TRUE, set the default encoding on the
response headers to CONTENT_LENGTH. (Moved from SoupServer.)
(soup_message_cleanup_response): If priv->server_side is TRUE,
re-fix the response header encoding after clearing the headers.
Otherwise the response headers revert to SOUP_ENCODING_EOF after
sending a "100 Continue".
(soup_message_is_keepalive): reorganize a little, fix a bug in the
HTTP/1.0 case.

* libsoup/soup-server.c (start_request): remove request encoding
override from here.

* tests/streaming-test.c: new test of SoupServer response
streaming, testing chunked, content-length, and eof-terminated
responses

svn path=/trunk/; revision=1243

explain why @len normally needs to be at least @boundary_len+1. Suggested

* libsoup/soup-socket.c (soup_socket_read_until): explain why @len
normally needs to be at least @boundary_len+1. Suggested by
Benjamin Otte.
(soup_socket_class_init): add longer gtk-docs to the non-blocking
property, to explain the way non-blocking I/O works in SoupSocket.

svn path=/trunk/; revision=1242

Fix a series of issues that prevented SoupCookieJarSQLite from working:

2009-02-19  Xan Lopez  <xan@gnome.org>

        * libsoup/soup-cookie-jar-sqlite.c:
        (callback):
        (try_create_table):
        (exec_query_with_try_create_table):
        (load):
        (changed):

        Fix a series of issues that prevented SoupCookieJarSQLite from working:

        - Try to create cookies table on errors. The table was never created before.
        - Store max_age in a gulong, not int.
        - Parse correctly boolean values from query. It's 0/1, not FALSE/TRUE.
- The host is stored as host in the table, not as domain.

        #572409

svn path=/trunk/; revision=1241

2.25.91

* configure.in: 2.25.91

* NEWS: update

svn path=/trunk/; revision=1239

Updates: Mention SoupSessionFeature (and link to SoupLogger,

* docs/reference/client-howto.xml: Updates: Mention
SoupSessionFeature (and link to SoupLogger, SoupCookieJar, and
SoupProxyResolverGNOME specifically). Mention forms and XML-RPC
support. Mention header-parsing methods. Give a concrete example
of connecting to SoupMessage signals. Document the (minimal)
thread-safety guarantees

* docs/reference/build-howto.xml: basic notes on pkg-config and
#include usage.

svn path=/trunk/; revision=1238

Fix crash when getting a 407 when using SoupProxyResolverGNOME (or when

* libsoup/soup-auth-manager.c (authenticate_auth): Fix crash when
getting a 407 when using SoupProxyResolverGNOME (or when not using
any proxy). Launchpad bug #326099. (Note that this doesn't
actually make proxy-auth-with-SoupProxyResolverGNOME *work*, it
just makes it not crash.)

svn path=/trunk/; revision=1237

2.25.5

* configure.in: 2.25.5

* NEWS: update

svn path=/trunk/; revision=1235

fix docs build

svn path=/trunk/; revision=1234

Remove this, as it was incomplete, and libproxy is now officially a

* libsoup/soup-proxy-resolver-gconf.h:
* libsoup/soup-proxy-resolver-gconf.c: Remove this, as it was
incomplete, and libproxy is now officially a dependency of GNOME.

* libsoup/Makefile.am:
* libsoup/soup-gnome-features.c: remove gconf-vs-libproxy ifdefs

* configure.in: Remove GConf checks, require libproxy if building
libsoup-gnome.

svn path=/trunk/; revision=1233

Fix this; previously it would discard the entire message body after

* libsoup/soup-message-body.c (soup_message_body_wrote_chunk): Fix
this; previously it would discard the entire message body after
writing a SOUP_MEMORY_TEMPORARY chunk. Part of WebKit bug 18343.

* libsoup/soup-message-io.c (io_write): use
io->write_chunk->length *before* freeing io->write_chunk.

* tests/chunk-test.c (do_temporary_test): new test to make sure
that TEMPORARY buffers are handled properly.

svn path=/trunk/; revision=1232

remove the dummy typedef for struct SoupSessionFeature that was needed

* libsoup/soup-session-feature.h: remove the dummy typedef for
struct SoupSessionFeature that was needed because we weren't
scanning soup-types.h. Fixes the build with non-GNU compilers.

* libsoup/soup-session-feature.c: document
SoupSessionFeatureInterface

* docs/reference/Makefile.am (IGNORE_HFILES): oops, don't ignore
soup-types.h

* docs/reference/libsoup-2.4-docs.sgml:
* docs/reference/libsoup-2.4-sections.txt: Split
SoupSessionFeature into its own file.

svn path=/trunk/; revision=1231

Don't pass NULL to soup_message_headers_replace(), call

* libsoup/soup-cookie-jar.c (request_started): Don't pass NULL to
soup_message_headers_replace(), call soup_message_headers_remove()
if there are no cookies. Likely fix for webkit bug #23240.

* libsoup/soup-message-headers.c (soup_message_headers_append):
g_return_if_fail (value != NULL)

svn path=/trunk/; revision=1227

2.25.4

* NEWS: update

svn path=/trunk/; revision=1225

doh. belatedly commit the 2.25.3 commit, and then bump to 2.25.4

svn path=/trunk/; revision=1224

add these to query session features. #565392.

* libsoup/soup-session.c (soup_session_get_features)
(soup_session_get_feature): add these to query session features.
#565392.

svn path=/trunk/; revision=1223

add some more warning CFLAGS, inspired by Benjamin Otte's blog post,

* configure.in: add some more warning CFLAGS, inspired by Benjamin
Otte's blog post, although none of them picked out any actual
bugs. Annoyingly, the most interesting warnings came from
-Wwrite-strings and -Wshadow, both of which I decided against
keeping, because they had too many false positives.

* libsoup/soup-cookie-jar.c (soup_cookie_jar_get_cookies): rename
a variable to avoid shadowing.

* libsoup/soup-message-headers.c
(soup_message_headers_get_ranges): move a variable declaration to
avoid a possibly-confusing shadowing.

* tests/forms-test.c:
* tests/header-parsing.c:
* tests/range-test.c:
* tests/test-utils.c: constify some "char *"s that should have
already been const.

* tests/get.c (find_hrefs): rename an arg whose name shadowed a
global, to avoid possible future confusion
(get_url): Likewise with a functional-internal shadowing.

svn path=/trunk/; revision=1222

Explicitly document the fact that you have to call soup_uri_set_path()

* libsoup/soup-uri.c (soup_uri_new): Explicitly document the fact
that you have to call soup_uri_set_path() when using
soup_uri_new(NULL), since path is required to be non-%NULL.

* libsoup/soup-connection.c (connect_message): initialize
uri->path

* libsoup/soup-cookie.c (soup_cookie_applies_to_uri):
g_return_val_if_fail() rather than crashing if uri->path is %NULL.
Also, fix the cookie/uri path comparison to not potentially read
off the end of uri->path. #562191, Mark Lee.

svn path=/trunk/; revision=1221

Correctly handle forms that have URI-encoded parameter names. #563302,

* libsoup/soup-form.c (soup_form_decode): Correctly handle forms
that have URI-encoded parameter names. #563302, Evan Nemerson.

* tests/forms-test.c: test that

svn path=/trunk/; revision=1220

disconnect from gconf notifications. Fixes a crash, #563145.

* libsoup/soup-proxy-resolver-gconf.c (finalize): disconnect from
gconf notifications. Fixes a crash, #563145.

svn path=/trunk/; revision=1219

2.25.2

* configure.in: 2.25.2

* NEWS: update

svn path=/trunk/; revision=1217

Fix this so we choose the *strongest* auth type first, rather than the

* libsoup/soup-auth-manager.c (auth_type_compare_func): Fix this
so we choose the *strongest* auth type first, rather than the
weakest. Doh. #562339, Pontus Oldberg.

svn path=/trunk/; revision=1216

make this $(includedir)/libsoup-gnome-2.4/libsoup rather than being the

* libsoup/Makefile.am (libsoupgnomeincludedir): make this
$(includedir)/libsoup-gnome-2.4/libsoup rather than being the same
as $(libsoupincludedir). Makes it easier to split into two
packages.

svn path=/trunk/; revision=1215

fix method name in example. #562411, Andreas Bruse.

* docs/reference/client-howto.xml: fix method name in example.
#562411, Andreas Bruse.

svn path=/trunk/; revision=1214

clarify exactly when stuff gets logged (and in particular, that

* libsoup/soup-logger.c: clarify exactly when stuff gets logged
(and in particular, that SoupSession::authenticate gets emitted
before the response it is authenticating gets logged).

svn path=/trunk/; revision=1207

fix linking with --as-needed. #559342, pointed out by Götz Waschk

* libsoup/Makefile.am (libsoup_gnome_2_4_la_LIBADD): fix linking
with --as-needed. #559342, pointed out by Götz Waschk

svn path=/trunk/; revision=1205

2.25.1

* configure.in: 2.25.1

* NEWS: update

svn path=/trunk/; revision=1203

fix to mention that xan and diegoe did most of the cookie stuff

svn path=/trunk/; revision=1202

misc doc fixes

* libsoup/soup-address.c (soup_address_is_resolved):
* libsoup/soup-cookie.c (soup_cookie_copy):
* libsoup/soup-cookie-jar.c (soup_cookie_jar_class_init):
* libsoup/soup-message-headers.c (SoupMessageHeadersType):
* libsoup/soup-proxy-resolver.c
(soup_proxy_resolver_get_proxy_async)
(soup_proxy_resolver_get_proxy_sync):
* libsoup/soup-status.c (soup_status_proxyify): misc doc fixes

* libsoup/soup-cookie-jar-text.h: remove a "deprecated" API that
was never actually released

* libsoup/soup.h: include soup-proxy-resolver.h

* docs/reference/Makefile.am (SCAN_OPTIONS): set
--deprecated-guards correctly
(IGNORE_HFILES): ignore some more internal files
(GTKDOC_LIBS): link against libsoup-gnome, not libsoup

* docs/reference/libsoup-2.4-docs.sgml:
* docs/reference/libsoup-2.4.types:
* docs/reference/libsoup-2.4-sections.txt: add new stuff

svn path=/trunk/; revision=1201

implementation of SoupCookieJar that persists to a text file in the old

* libsoup/soup-cookie-jar-text.c: implementation of SoupCookieJar
that persists to a text file in the old Mozilla cookies.txt
format.

* libsoup/soup-cookie-jar-sqlite.c: implementation of
SoupCookieJar that persists to an sqlite database in the new
Mozilla cookies.sqlite format. (Part of libsoup-gnome.)

* libsoup/soup-cookie-jar.c: add various functionality needed by
the two new subclasses. Does not break API/ABI compat with 2.24.

* libsoup/soup-cookie.c (soup_cookie_get_type): register
SoupCookie as a boxed type.
(domain_matches): fix a bug here that meant "foo.com" couldn't set
a cookie for domain=.foo.com
(soup_cookie_applies_to_uri): fix path checking

* configure.in: if building --with-gnome, require sqlite3

svn path=/trunk/; revision=1200

don't remove the item from the queue here; it should already have happened

* libsoup/soup-session-sync.c (process_queue_item): don't remove
the item from the queue here; it should already have happened in
all circumstances. Possible fix for #559052.

* libsoup/soup-session.c (cancel_message): don't remove the item
from the queue here; the call to soup_message_finished() will do
that.

* libsoup/soup-message-queue.c (soup_message_queue_remove): This
should only be called once, so g_return_if_fail (!item->removed)

* tests/test-utils.c (test_init): install a new default g_log
handler that increments the error count when it's called so that a
test won't pass if it triggers a g_warning() or
g_return_if_fail().

svn path=/trunk/; revision=1199

fix srcdir/builddir mixup.

* tests/Makefile.am (get_LDADD): fix srcdir/builddir mixup.

svn path=/trunk/; revision=1198

don't leak the header string

* libsoup/soup-message-headers.c (set_content_foo): don't leak the
header string

* libsoup/soup-multipart.c (generate_boundary): avoid a (harmless)
valgrind warning

* libsoup/soup-proxy-resolver-static.c (get_proxy_sync): don't
leak the address on error

* libsoup/soup-session-sync.c (wait_for_connection): don't leak
proxy_addr

* tests/misc-test.c: don't leak the SoupMessage signal ids

* tests/range-test.c (main): don't leak base_uri

* tests/libsoup.supp: update this using lots and lots of wildcards

svn path=/trunk/; revision=1197

Fix the signal handler disconnection here: for "finished" we were passing

* libsoup/soup-session.c (message_finished): Fix the signal
handler disconnection here: for "finished" we were passing the
wrong user_data to g_signal_handlers_disconnect_by_func(), and for
"got_body" it turns out you can't use _disconnect_by_func() when
there's a metamarshal, making it incompatible
withsoup_message_add_header_handler(). Fixes a crash in
evolution-exchange, #559054.

* tests/misc-test.c (do_msg_reuse_test): Ensure that SoupSession
and its features disconnect all of their signals from a message
when they're done with it.

svn path=/trunk/; revision=1196

commit changelog for previous commit. oops

svn path=/trunk/; revision=1195

Add libsoup-gnome, for new features that depend on GNOME
libraries.

* configure.in: Check for libproxy and/or gconf, accept
--without-gnome option, output libsoup-gnome-2.4.pc

* libsoup-gnome-2.4.pc: pc file for libsoup with GNOME support

* libsoup/Makefile.am: build libsoup-gnome.la if so configured

* libsoup/soup-gnome.h: base header for libsoup-gnome

* libsoup/soup-proxy-resolver-libproxy.c: An implementation of
SoupProxyResolver that uses libproxy.

* libsoup/soup-proxy-resolver-gconf.c: An implementation of
SoupProxyResolver that uses the proxy keys in GConf. Does not
completely handle ignore_hosts; this is currently just used as a
fallback if libproxy is not available.

* libsoup/soup-gnome-features.c: provides
SOUP_TYPE_PROXY_RESOLVER_GNOME (abstracting over
SoupProxyResolverGConf and SoupProxyResolverLibproxy) and
SOUP_TYPE_GNOME_FEATURES_2_26, which adds "all GNOME-specific
features in libsoup 2.26", which is currently just the proxy
resolver.

* libsoup/soup-session-async.c (resolved_proxy_addr): set
item->resolved_proxy_addr
(run_queue): resolve the proxy if !item->resolved_proxy_addr, not
if !item->proxy_addr, since the proxy addr might resolve to NULL.

* tests/Makefile.am (get_LDADD):
* tests/get.c: If we built libsoup-gnome, use it in "get" for
automatic proxy support

svn path=/trunk/; revision=1194

rename from libsoup.pc.in; the attempt to keep the source tree

* libsoup-2.4.pc.in: rename from libsoup.pc.in; the attempt to
keep the source tree API-version-generic wasn't really working,
and we're probably not ever going to change the API version again
anyway.

* Makefile.am (pkgconfig_DATA): install the .pc file the normal
way rather than using an install-data-local rule to rename the .pc
file as we install it

* libsoup/Makefile.am (libsoupincludedir):
* tests/Makefile.am (LIBS):
* docs/reference/Makefile.am (GTKDOC_LIBS): Say "2.4" everywhere,
instead of 2.4 in some places and $(SOUP_API_VERSION) in others.

* configure.in: updates for .pc renaming. Also, use
AS_HELP_STRING() in AC_ARG_ENABLE() and AC_ARG_WITH() rules

svn path=/trunk/; revision=1193

new abstract base class for a SoupSessionFeature that determines what

* libsoup/soup-proxy-resolver.c: new abstract base class for a
SoupSessionFeature that determines what proxy to use for a given
URI.

* libsoup/soup-proxy-resolver-static.c: a SoupProxyResolver that
always returns the same value.

* libsoup/soup-session.c (set_property, get_property): implement
the SOUP_SESSION_PROXY_URI property by creating/destroying a
SoupProxyResolverStatic as needed.
(soup_session_get_connection): Use the proxy address passed by the
caller rather than priv->proxy_uri.

* libsoup/soup-session-async.c (run_queue): if the session has a
proxy resolver, use it, and pass the resolved proxy to
soup_session_get_connection().
(request_restarted): clear the previously-resolved proxy address
when restarting the message

* libsoup/soup-session-sync.c (wait_for_connection): if the
session has a proxy resolver, use it, and pass the resolved proxy
to soup_session_get_connection().

* libsoup/soup-message-queue.h (SoupMessageQueueItem): add
proxy-address-resolving fields

* libsoup/soup-status.c (soup_status_proxify): moved from
soup-connection; turn SOUP_STATUS_CANT_RESOLVE into
SOUP_STATUS_CANT_RESOLVE_PROXY, and SOUP_STATUS_CANT_CONNECT into
SOUP_STATUS_CANT_CONNECT_PROXY (and pass all other statuses
through unchanged)

svn path=/trunk/; revision=1192

do directory listings. (wrote this a long time ago, it just never made it

* tests/simple-httpd.c: do directory listings. (wrote this a long
time ago, it just never made it into svn)

svn path=/trunk/; revision=1191

free the expires date, if set

* libsoup/soup-cookie.c (soup_cookie_free): free the expires date,
if set

* libsoup/soup-auth-domain-basic.h:
* libsoup/soup-auth-domain-digest.h:
* libsoup/soup-auth-domain.h:
* libsoup/soup-cookie-jar.h:
* libsoup/soup-logger.h:
* libsoup/soup-multipart.h: add G_BEGIN/END_DECLS

* libsoup/soup-date.c: add some g_return_if_fails

Patches from and inspired by Xan Lopez, #522125

svn path=/trunk/; revision=1190

fix up some of the regression test configuration stuff, and print warnings

* configure.in:
* tests/Makefile.am: fix up some of the regression test
configuration stuff, and print warnings when some tests aren't run
do to missing dependencies

svn path=/trunk/; revision=1189

remove gtk-doc.make. i'm not sure why it was ever committed

svn path=/trunk/; revision=1188

add SOUP_MAINTAINER_FLAGS here too.

* tests/Makefile.am (INCLUDES): add SOUP_MAINTAINER_FLAGS here
too.

* tests/dns.c (main):
* tests/getbug.c (main):
* tests/server-auth-test.c (do_test): replace deprecated glib
functions

svn path=/trunk/; revision=1184

Build with G_DISABLE_DEPRECATED and G_DISABLE_SINGLE_INCLUDES; enforce the

2008-10-20  Cosimo Cecchi  <cosimoc@gnome.org>

* configure.in:
* libsoup/Makefile.am:
* libsoup/soup-status.h:
* libsoup/soup-types.h:
* libsoup/soup-uri.c: (soup_uri_to_string):
Build with G_DISABLE_DEPRECATED and G_DISABLE_SINGLE_INCLUDES; enforce
the first switch under maintainer mode and the second one
unconditionally (#557072).

svn path=/trunk/; revision=1183

compare scheme name case-insensitively, to prevent an infinite loop when

* libsoup/soup-auth.c (soup_auth_update): compare scheme name
case-insensitively, to prevent an infinite loop when it's not in
standard form. #536285

svn path=/trunk/; revision=1182

if the caller passed the total_length of the message body, then sort sort

* libsoup/soup-message-headers.c
(soup_message_headers_get_ranges): if the caller passed the
total_length of the message body, then sort sort the ranges and
merge overlapping ones to generate a minimal set.

* tests/range-test.c: test it

svn path=/trunk/; revision=1181

Verify whether uri is non-NULL and avoid crash on NULL pointer dereference

2008-10-09  Andrew W. Nosenko  <andrew.w.nosenko@gmail.com>

* libsoup/soup-uri.c (soup_uri_to_string): Verify whether uri is
non-NULL and avoid crash on NULL pointer dereference therefore.

svn path=/trunk/; revision=1180

Change the SoupURI properties to SoupAddress properties.

* libsoup/soup-connection.c: Change the SoupURI properties to
SoupAddress properties.

* libsoup/soup-address.c (soup_address_resolve_async)
(soup_address_resolve_sync): Redo slightly so that multiple
simultaneous attempts to resolve the same address won't cause
problems.
(soup_address_hash_by_name, soup_address_equal_by_name):
(soup_address_hash_by_ip, soup_address_equal_by_ip): methods to
hash addresses by name or IP address

* libsoup/soup-message.c (soup_message_get_address): gets a
SoupAddress corresponding to the message's URI

* libsoup/soup-auth-manager.c (SoupAuthHost): hash hosts by
soup_address_hash_by_name() rather than by URI.

* libsoup/soup-session.c (soup_session_get_connection): pass
addresses to soup_connection_new(), not URIs.
(SoupSessionHost): hash hosts by soup_address_hash_by_ip() rather
than by URI. This requires that the addresses will have already
been resolved by the SoupSession subclasses before calling
soup_session_get_connection(), but also means that now requests
made to different virtual hosts on the same IP address can share a
connection.

* libsoup/soup-message-queue.c (SoupMessageQueueItem): add
address-resolving state

* libsoup/soup-session-sync.c (process_queue_item): resolve the
message's address before getting a connection

* libsoup/soup-session-async.c (run_queue, resolve_msg_addr)
(resolved_msg_addr): resolve the message's address before getting
a connection
(request_restarted): if the message gets requeued to a different
host, we'll need to re-resolve the address.

* libsoup/soup-uri.c (soup_uri_copy_root, soup_uri_host_hash)
(soup_uri_host_equal): No longer needed

* libsoup/soup-dns.c (do_async_callback): disconnect from the
cancellable before invoking the callback

* tests/proxy-test.c (tests): fix the 403 example; hostnames are
resolved by the session now, even when sending to a proxy, so we
need to use a hostname that actually exists

svn path=/trunk/; revision=1179

Make this more complicated, with a SoupMessageQueueItem to keep track of

* libsoup/soup-message-queue.c: Make this more complicated, with a
SoupMessageQueueItem to keep track of the session's per-message
state. (Part of the process of moving session-related state out of
SoupMessagePrivate.)

* libsoup/soup-session.c: Update to work in terms of
SoupMessageQueueItem

* libsoup/soup-session-async.c:
* libsoup/soup-session-sync.c: use SoupMessageQueueItem (and get
rid of SoupSessionAsyncQueueData and SoupSessionSyncAsyncData).

svn path=/trunk/; revision=1178

update

svn path=/trunk/; revision=1177

New type and methods for working with multipart HTTP bodies (eg,

* libsoup/soup-multipart.c: New type and methods for working with
multipart HTTP bodies (eg, multipart/form-data and
multipart/byte-ranges)

* libsoup/soup-message-headers.c (soup_message_headers_get_ranges)
(soup_message_headers_set_ranges)
(soup_message_headers_set_range)
(soup_message_headers_get_content_range)
(soup_message_headers_set_content_range): New methods for dealing
with the Range and Content-Range headers.

* libsoup/soup-form.h (SOUP_FORM_MIME_TYPE_URLENCODED)
(SOUP_FORM_MIME_TYPE_MULTIPART): #define these MIME types here

* libsoup/soup-form.c (soup_form_decode_multipart): new utility
for parsing multipart/form-data forms.
(soup_form_request_new_from_multipart): new utility for
constructing multipart/form-data forms

* libsoup/soup-headers.c (soup_headers_parse): this is now
non-static, for use by soup-multipart

* libsoup/soup-message-server-io.c (get_response_headers)
(handle_partial_get): if the client requested a partial GET, and
the SoupServer is returning the full body, rebuild the response to
include only the requested range instead

* tests/forms-test.c: renamed from query-test and updated to do
both application/x-www-form-urlencoded and multipart/form-data
tests

* tests/range-test.c: test of Range/Content-Range functionality

svn path=/trunk/; revision=1176

Update these to deal with RFC2231-encoded UTF-8 header params

* libsoup/soup-headers.c (soup_header_parse_param_list)
(soup_header_parse_semi_param_list): Update these to deal with
RFC2231-encoded UTF-8 header params
(soup_header_g_string_append_param): new utility method to do
parameters with quoted-strings (handling escaping) and RFC2231.

* libsoup/soup-auth-digest.c (get_authorization):
* libsoup/soup-auth-domain-basic.c (challenge):
* libsoup/soup-auth-domain-digest.c (challenge): use
soup_header_g_string_append_param so we handle escaping correctly

* libsoup/soup-message-headers.c
(soup_message_headers_get_content_type)
(soup_message_headers_set_content_type)
(soup_message_headers_get_content_disposition)
(soup_message_headers_set_content_disposition): New convenience
methods.

* tests/header-parsing.c (do_rfc2231_tests): new test of RFC2231
encoded header parsing in Content-Disposition.

* tests/get.c (get_url): use
soup_message_headers_get_content_type()

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

svn path=/trunk/; revision=1175

add a new -s flag to indicate that it's being run from inside

* tests/xmlrpc-test.c (main): add a new -s flag to indicate that
it's being run from inside xmlrpc-server-test.
(test_echo): if we aren't running inside xmlrpc-server-test, and
the response strings don't match the request strings, then compare
them to echo_strings_broken instead; a bug in php-xmlrpc manifests
when using libxml2 >= 2.7.1, resulting in incorrect responses. :-/

* tests/xmlrpc-server-test.c (do_xmlrpc_tests): Pass -s to
xmlrpc-test

svn path=/trunk/; revision=1173

bump version to 2.25.0

* configure.in: bump version to 2.25.0

svn path=/trunk/; revision=1172

doc fixups

* libsoup/soup-cookie-jar.c:
* libsoup/soup-cookie.c:
* libsoup/soup-cookie.h:
* libsoup/soup-headers.c:
* libsoup/soup-logger.c:
* libsoup/soup-session-feature.c:
* libsoup/soup-session-feature.h:
* libsoup/soup-session.c: doc fixups

* docs/reference/libsoup-2.4-docs.sgml:
* docs/reference/libsoup-2.4-sections.txt:
* docs/reference/libsoup-2.4.types: Add missing bits

svn path=/trunk/; revision=1170

soup_value_hash_insert_value() copies the value, so we have to

* libsoup/soup-xmlrpc.c (parse_value):
soup_value_hash_insert_value() copies the value, so we have to
g_value_unset() our copy.

* tests/chunk-test.c:
* tests/misc-test.c:
* tests/ntlm-test.c: fix leaks

* tests/libsoup.supp: update

svn path=/trunk/; revision=1169

store the GSource in priv, don't ref the session. Otherwise the session

* libsoup/soup-session-async.c (do_idle_run_queue): store the
GSource in priv, don't ref the session. Otherwise the session
won't get destroyed if you abort it and then don't return to its
main loop. (addendum to #498509, Arnout Vandecappelle)
(finalize): Destroy the idle_run_queue source when finalizing.
(run_queue, got_connection): Ref the session when calling
soup_connection_connect_async(), and do a
do_idle_run_queue()+unref in got_connection, to ensure correct
handling regardless of what the application does with its own ref
on the session.
(final_finished): Likewise, ref/do_idle_run_queue/unref rather
than calling run_queue directly and playing with weak pointers.

* libsoup/soup-session.c (connect_result): ref the session around
the cancel-if-error loop

Fixes #533473, crash in seahorse when connecting to a
non-responsive key server.

* tests/misc-test.c (do_callback_unref_test): Add a test for the
bug in #533473.

* tests/test-utils.c (soup_test_session_abort_unref): abort and
unref a SoupSession, and consider it an error if the session still
exists afterward. Suggested by Arnout Vandecappelle.
(test_server_shutdown): Likewise, consider it an error if the
server is leaked.

* tests/*.c: Use soup_test_session_abort_unref().

svn path=/trunk/; revision=1168

libsoup/soup-auth-manager-ntlm.c libsoup/soup-auth-manager.c

* libsoup/soup-auth-manager-ntlm.c
* libsoup/soup-auth-manager.c
* libsoup/soup-cookie-jar.c
* libsoup/soup-dns.c
* libsoup/soup-logger.c:
* libsoup/soup-message-body.h:
* libsoup/soup-message.h
* libsoup/soup-misc.h:
* libsoup/soup-xmlrpc.h:

* tests/continue-test.c:
* tests/ntlm-test.c: Fix warnings pointed out by gcc -pedantic.
#553976, Sander Dijkhuis.

svn path=/trunk/; revision=1167

2.24.0.1

* configure.in: 2.24.0.1

* NEWS: Update

* libsoup/soup-session.c (redirect_handler):
* libsoup/soup-message.c (soup_message_new):
(soup_message_new_from_uri, soup_message_set_uri): Revert the
2008-08-25 change; it breaks the rhythmbox DAAP plugin. #553466.
To be revisited.

svn path=/trunk/; revision=1164

2.24.0

* configure.in: 2.24.0

svn path=/trunk/; revision=1161

2.23.92 (belated; I apparently forgot to commit this after disting

2.23.92
(belated; I apparently forgot to commit this after disting 2.23.92)

svn path=/trunk/; revision=1159

a 302 response to HEAD (or any other safe method) should be treated like a

* libsoup/soup-session.c (redirect_handler): a 302 response to
HEAD (or any other safe method) should be treated like a 307, not
a 303. #551190, Jonathan Matthew.

* tests/redirect-test.c: test that

svn path=/trunk/; revision=1158

2.23.91

* configure.in: 2.23.91

* NEWS: update

svn path=/trunk/; revision=1156

new macro to check if a URI is a valid http or https URI.

* libsoup/soup-uri.h (SOUP_URI_VALID_FOR_HTTP): new macro to check
if a URI is a valid http or https URI.

* libsoup/soup-uri.c (soup_uri_new_with_base): Update http/https
check to use SOUP_URI_VALID_FOR_HTTP().

* libsoup/soup-session.c (redirect_handler): Check
SOUP_URI_VALID_FOR_HTTP() and call it an error if the check fails.

* libsoup/soup-message.c (soup_message_new): Remove the uri->host
check from here. Update docs to clarify that @uri must be an
http/https URI.
(soup_message_new_from_uri): Check SOUP_URI_VALID_FOR_HTTP().
Update docs.
(soup_message_set_uri): Check SOUP_URI_VALID_FOR_HTTP(). Update
docs.

Should prevent the crash in #528882, but there's still something
going wrong there at a higher level.

svn path=/trunk/; revision=1155

Add a SoupDate to GTimeVal conversion function, for use in gvfs. #549006,

2008-08-22  Bastien Nocera  <hadess@hadess.net>

* libsoup/soup-date.c (soup_date_to_time_t),
(soup_date_to_timeval):
* libsoup/soup-date.h: Add a SoupDate to GTimeVal conversion
function, for use in gvfs. #549006, with help from Dan Winship
<danw@gnome.org>

svn path=/trunk/; revision=1154

Revert previous commit; that would still fail in the case of an

* libsoup.pc.in (Requires): Revert previous commit; that would
still fail in the case of an application linking against a library
that privately links against libsoup. The only correct solution in
the face of --as-needed (or on OSes where the linker always works
that way) is for the module that actually calls g_thread_init() to
explicitly link against libgthread.

svn path=/trunk/; revision=1153

g_return_if_fail if either username or password is NULL. Noted on the

* libsoup/soup-auth.c (soup_auth_authenticate): g_return_if_fail
if either username or password is NULL. Noted on the mailing list.

* libsoup/soup-auth-basic.c (authenticate): remove redundant check

svn path=/trunk/; revision=1152

Add gthread-2.0; the app must call g_thread_init(), but libsoup won't pull

* libsoup.pc.in (Requires): Add gthread-2.0; the app must call
g_thread_init(), but libsoup won't pull it in itself if built with
--as-needed, so make sure it gets pulled in from here. Noted by
Zeeshan Ali.

svn path=/trunk/; revision=1151

2.23.6

* configure.in: 2.23.6

* NEWS: update

svn path=/trunk/; revision=1149

Fix horrible bizarre brokenness in GIOChannel subclassing. #536417, Tor

* libsoup/soup-gnutls.c: Fix horrible bizarre brokenness in
GIOChannel subclassing. #536417, Tor Lillqvist.

svn path=/trunk/; revision=1148

Add this to schedule a callback in a GMainContext "right away", as opposed

* libsoup/soup-misc.c (soup_add_completion): Add this to schedule
a callback in a GMainContext "right away", as opposed to
soup_add_idle(), which uses a lower priority and therefore may not
end up calling the callback for a long time if the application is
busy with I/O. #536676, Benjamin Otte.

* libsoup/soup-dns.c (resolver_thread, async_cancel)
(soup_dns_lookup_resolve_async):
* libsoup/soup-message-io.c (soup_message_io_unpause):
* libsoup/soup-session-sync.c (queue_message_thread):
* libsoup/soup-session-async.c (do_idle_run_queue):
* libsoup/soup-socket.c (async_cancel)
(soup_socket_connect_async):
* tests/test-utils.c (test_server_shutdown): Use
soup_add_completion() rather than soup_add_idle().

* docs/reference/libsoup-2.4-sections.txt: add soup_add_completion

svn path=/trunk/; revision=1147

don't add a Host header to the message if the caller already added one.

* libsoup/soup-message-client-io.c (get_request_headers): don't
add a Host header to the message if the caller already added one.
#539803, Marc Maurer.

* libsoup/soup-logger.c (print_request): likewise

* tests/misc-test.c: new test file for small miscellaneous test
cases.
(do_host_test): test Host-header overriding

svn path=/trunk/; revision=1146

change read_length, write_length, and written to goffset so we can

* libsoup/soup-message-io.c (SoupMessageIOData): change
read_length, write_length, and written to goffset so we can
properly handle message bodies > 4G. #539861, Peter Christensen.

svn path=/trunk/; revision=1145

if the server response doesn't include an algorithm, it is supposed to

* libsoup/soup-auth-digest.c (soup_auth_digest_parse_algorithm):
if the server response doesn't include an algorithm, it is
supposed to default to MD5. #544681, Mads Chr. Olesen.

svn path=/trunk/; revision=1144

add $(GLIB_LIBS) so this still works when building with weird LDFLAGS.

* tests/Makefile.am (LIBS): add $(GLIB_LIBS) so this still works
when building with weird LDFLAGS. #541506, Götz Waschk.

* docs/reference/Makefile.am (GTKDOC_LIBS): likewise

svn path=/trunk/; revision=1143

The SO_RCVTIMEO and SO_SNDTIMEO options to setsockopt() take int values,

2008-06-04  Tor Lillqvist  <tml@novell.com>

* libsoup/soup-socket.c (set_fdflags): The SO_RCVTIMEO and
SO_SNDTIMEO options to setsockopt() take int values, in
milliseconds, on Windows. Not struct timeval. Eek. So passing a
struct timeval meant that the tv_sec value (which is first in the
struct) is interpreted as milliseconds. setsockopt apparently
doesn't even get upset by the fact that the option size doesn't
match the sizeof(int) it should expect.

svn path=/trunk/; revision=1142

fix the path checking

* libsoup/soup-cookie.c (soup_cookie_applies_to_uri): fix the path
checking

svn path=/trunk/; revision=1141

Fix compilation error in the !HAVE_IPV6 && !HAVE_INET_PTON &&

2008-04-29  Tor Lillqvist  <tml@novell.com>

* libsoup/soup-dns.c (soup_dns_is_ip_address): Fix compilation
error in the !HAVE_IPV6 && !HAVE_INET_PTON && !HAVE_INET_ATON
case.

svn path=/trunk/; revision=1140

2.23.1, bump AGE and CURRENT

* configure.in: 2.23.1, bump AGE and CURRENT

* NEWS: update

svn path=/trunk/; revision=1138

Fixes for GnuTLS support on Win32. #528752, patch from Marc Maurer

* libsoup/soup-gnutls.c (soup_ssl_wrap_iochannel): add an argument
saying whether or not the socket is non-blocking, since there's no
way to determine this from the fd in WinSock.
(do_handshake, soup_gnutls_read, soup_gnutls_write): Update for
that.

* libsoup/soup-socket.c (soup_socket_start_proxy_ssl): Update for
that

* libsoup/soup-nossl.c (soup_ssl_wrap_iochannel): update the
declaration here too

* tests/ssl-test.c: Some updates to get this closer to working on
Windows...

svn path=/trunk/; revision=1137

Check that the cookie was parsed successfully before setting it

        * libsoup/soup-cookie-jar.c (soup_cookie_jar_set_cookie):
        Check that the cookie was parsed successfully before setting it

svn path=/trunk/; revision=1136

Initial HTTP cookie support imported from development git repo,
including patches from Xan Lopez.

TODO: make sure the logic in soup_cookie_jar_get_cookies() is
right. Add a test program to tests/.

* libsoup/soup-cookie.c: Code for parsing and generating HTTP
cookies.

* libsoup/soup-cookie-jar.c: Code for managing SoupCookies and
integrating cookie management with a SoupSession.

* libsoup/soup-date.c (soup_date_is_past): New, checks if a
SoupDate refers to a time in the past

* libsoup/soup-dns.c (soup_dns_is_ip_address): New, checks if a
string is a valid IP address

* libsoup/soup-headers.c (soup_header_parse_semi_param_list): New,
like soup_header_parse_param_list, but for semicolon-delimited
data.

svn path=/trunk/; revision=1135

Make this a GObject and specifically a SoupSessionFeature. Add an

* libsoup/soup-auth-manager.c: Make this a GObject and
specifically a SoupSessionFeature. Add an "authenticate" signal,
and emit that rather than explicitly calling into the SoupSession
and telling it when to emit its own authenticate signal.

* libsoup/soup-auth-manager-ntlm.c: Make this a subclass of
SoupAuthManager, with NTLM support controllable via a property.

* libsoup/soup-session.c (soup_session_init): create an
auth_manager of type SOUP_TYPE_AUTH_MANAGER_NTLM, but defaulting
to USE_NTLM=FALSE. Connect to its "authenticate" signal, and call
soup_session_add_feature() on it.
(set_property, get_property): proxy the USE_NTLM property to the
auth manager.
(auth_manager_authenticate): signal handler for SoupAuthManager
"authenticate" signal. (Replaces soup_session_emit_authenticate(),
which is no longer needed)

svn path=/trunk/; revision=1134

New interface type representing a feature that can be added to a

* libsoup/soup-session-feature.c: New interface type representing
a feature that can be added to a SoupSession.

* libsoup/soup-session.c (soup_session_add_feature): Add a feature
to the session by prepending it to priv->features and calling
soup_session_feature_attach() on it.
(soup_session_add_feature_by_type): Add a feature to the session
by creating an object of the indicated type and passing it to
soup_session_add_feature.
(soup_session_remove_feature)
(soup_session_remove_feature_by_type): Likewise, remove features
(soup_session_class_init, set_property): register/handle
construct-time feature adding/removing properties
(dispose): cleanup features

* libsoup/soup-logger.c: port to SoupSessionFeature

* tests/test-utils.c (soup_test_session_new): Use
soup_session_add_feature rather than soup_logger_attach.

svn path=/trunk/; revision=1133

Having branched for gnome-2-22, bump version to 2.23.0 for the GNOME 2.23

* configure.in: Having branched for gnome-2-22, bump version to
2.23.0 for the GNOME 2.23 series. SOUP_API_VERSION will stay at
2.4, which is confusing but seemed like the best solution at this
point.

svn path=/trunk/; revision=1132

2.4.1. Bump AGE and CURRENT.

* configure.in: 2.4.1. Bump AGE and CURRENT.

* NEWS: update

* docs/reference/libsoup-2.4-sections.txt: add new symbols

svn path=/trunk/; revision=1129

If pausing a message that was waiting to unpause, cancel the unpause.

* libsoup/soup-message-io.c (soup_message_io_pause): If pausing a
message that was waiting to unpause, cancel the unpause.

svn path=/trunk/; revision=1128

Don't cache negative results, even if the DNS server explicitly states

* libsoup/soup-dns.c (resolve_address, resolve_name): Don't
cache negative results, even if the DNS server explicitly states
that the host does not exist; some servers give different answers
to clients inside and outside their firewall. #523269,  Jörgen
Scheibengruber.

svn path=/trunk/; revision=1127

New, replaces SOUP_MESSAGE_OVERWRITE_CHUNKS, but can be set on either the

* libsoup/soup-message-body.c (soup_message_body_set_accumulate)
(soup_message_body_get_accumulate): New, replaces
SOUP_MESSAGE_OVERWRITE_CHUNKS, but can be set on either the
incoming or outgoing message body.
(soup_message_body_get_chunk): update to still dtrt if !accumulate
(soup_message_body_got_chunk, soup_message_body_wrote_chunk): New
methods to handle accumulating/discarding chunks.

* libsoup/soup-message-io.c (read_body_chunk): Use
soup_message_body_got_chunk.
(io_write): Use soup_message_body_wrote_chunk, to discard unneeded
chunks after writing them. Fixes most of #522146.

* libsoup/soup-message.c (soup_message_class_init): add a new
flag, "server-side", to indicate whether the message is
client-side or server-side, and update several methods to use it.
(got_body): Update for accumulate
(soup_message_set_flags): If the caller changes OVERWRITE_CHUNKS,
update the corresponding accumulate flag.

* libsoup/soup-message.h (SOUP_MESSAGE_OVERWRITE_CHUNKS):
deprecated now

* tests/chunk-test.c (do_request_test): Use
soup_message_body_set_accumulate() now, and verify that the chunks
are being discarded appropriately.
(do_response_test): Use
soup_message_body_set_accumulate() instead of OVERWRITE_CHUNKS.

* tests/pull-api.c (do_fully_async_test)
(do_synchronously_async_test): Use
soup_message_body_set_accumulate().

svn path=/trunk/; revision=1126

fix test for AI_ADDRCONFIG. Noticed while looking at #526321.

* libsoup/soup-dns.c (resolve_address): fix test for
AI_ADDRCONFIG. Noticed while looking at #526321.

svn path=/trunk/; revision=1125

globally ignore SIGPIPE rather than only doing it around socket write

* libsoup/soup-socket.c (soup_socket_class_init)
(soup_socket_write): globally ignore SIGPIPE rather than only
doing it around socket write calls, since with SSL even socket
read calls may need to write, and also because SIGPIPE is
completely moronic and no one should be using it, and the previous
"solution" wasn't thread-safe anyway. Fixes #524397, reported by
Curtis Magyar.

svn path=/trunk/; revision=1124

Misc fixes noticed by "sparse" or by running gcc with additional
-W flags

* libsoup/soup-auth-manager-ntlm.c (ntlm_authorize_post): fix a
potentially uninitialized variable. (Grumble. gcc needs
-Wdo-optimization-so-you-can-generate-code-flow-related-warnings-
but-then-emit-unoptimized-code-for-ease-of-debugging)

* libsoup/soup-gnutls.c (soup_gnutls_channel_funcs): make this
static

* libsoup/soup-uri.c (uri_decoded_copy, uri_normalized_copy): add
"static". (This doesn't change the generated code; the prototype
was already declared static and so gcc was treating the function
as static even though the main declaration *wasn't* declared
static. I'm not sure if this is a bug in gcc or an oddity of the
spec, but it's confusing, so...)

* libsoup/soup-xmlrpc.c (soup_xmlrpc_build_method_response):
s/FALSE/NULL/

* libsoup/soup-xmlrpc.h: add G_GNUC_PRINTF to
soup_xmlrpc_build_format

* tests/*.c: misc minor fixes, mostly involving missing "const"s
and "static"s to get better warnings, and then remove some unused
variables.

* tests/continue-test.c (do_message): fix a crash when the test
fails

* tests/test-utils.h (debug_printf): add G_GNUC_PRINTF to
prototype

svn path=/trunk/; revision=1123

Explicitly assign each of the variables to NULL, because that apparently

* libsoup/soup-method.c: Explicitly assign each of the variables
to NULL, because that apparently causes the OS X linker to treat
them differently than if they are left implicitly NULL. #522957.

svn path=/trunk/; revision=1122

Explicitly assign each of the variables to NULL, because that apparently

* libsoup/soup-method.c: Explicitly assign each of the variables
to NULL, because that apparently causes the OS X linker to treat
them differently than if they are left implicitly NULL. #522957.

svn path=/trunk/; revision=1121

add a new signal "wrote-body-data" to address the problem that

* libsoup/soup-message.c: add a new signal "wrote-body-data" to
address the problem that "wrote-chunk" is not usable for progress
info (especially with non-chunked encoding). #525101, suggested by
Christian Kellner.

* libsoup/soup-message-io.c (write_data): emit wrote-body-data as
appropriate.
(io_write): update so that (a) Content-Length writes can be done
in multiple chunks (as long as the caller explicitly sets the
Content-Length header beforehand), and (b) the body data doesn't
get copied an extra time. Based on a patch from Christian.

* libsoup/soup-message-client-io.c (get_request_headers): Don't
update the Content-Length header if it's already set, even if it
doesn't match the (current) body length.

* tests/chunk-test.c: test some chunk-encoding-related behavior

svn path=/trunk/; revision=1120

Be more aggressive about closing unused persistent connections
when needed, to avoid slow load times in WebKit.

* libsoup/soup-session-async.c (run_queue): Remove the
"try_pruning" flag from here and from all the callers, and
*always* try pruning idle connections if it would help.

* libsoup/soup-session.c (soup_session_try_prune_connection):
Rather than only closing a single connection, close all idle
connections.

svn path=/trunk/; revision=1119