GError for CamelTcpStreamSSL

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make _set_g_error_from_errno() public

We need it in the SSL stream

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Format error strings, don't include them inline

Fix method name

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Missing include

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Missing comma

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Fix cut&paste thinko

Signed-off-by: Federico Mena Quintero <federico@novell.com>

GError **, not GError *

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Oops, fix endianness of port numbers

ai_addr.sin_port and sin6_port are in network byte order...

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Use 'localhost' instead of NULL to resolve ports

camel_getaddrinfo() does not take NULL hostnames, in contrast with
getaddrinfo().  So for now we'll use localhost and try to do
better in the future.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Don't try SOCKS4 on IPv6 addresses; it doesn't handle them.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Try SOCKS5, then SOCKS4

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Try all the struct addrinfo when connecting to a proxy, not just the first one

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Add new GError codes for SOCKS proxies - CamelProxyError

We differentiate between "the proxy does not support our protocol"
and "the proxy does not support our authentication type".

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Basic logic to request connection from a SOCKS5 proxy

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Separate function to just connect to the proxy host

We will use this in each function to connect to a different kind of proxy.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Derive CamelTcpStreamSSL from CamelTcpStreamRaw

The only overriden vmethod is ::connect(), which starts up SSL on the socket
after the Raw implementation opens the socket and connects.  By this point,
we are already past the SOCKS proxy if it exists.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Add function to replace CamelTcpStreamRaw's sockfd with another one

We need this in the TcpSSL stream to replace the normal socket with
the SSL-ified one.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make _set_errno_from_pr_error() public

We need to use this both in the raw TCP stream and in the SSL one.
Maybe later we can switch to full exceptions.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make CamelTcpStreamRaw use PRFileDesc internally

This is cut&pasted from CamelTcpStreamSSL, with the SSL bits removed.
This will let us make CamelTcpStreamSSL actually derive from
CamelTcpStreamRaw and just slap the SSL bits on top.

This also removes camel_tcp_stream_raw_get_fd(), in favor of the general
camel_tcp_stream_get_file_desc(), which only the imapx provider uses.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Implement ::get_file_desc() in CamelTcpStreamSSL

This lets us get rid of camel_tcp_stream_ssl_sockfd(), in favor
of camel_tcp_stream_get_file_desc().

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Move CamelTcpStreamRaw's fields to a private structure

We also add a temporary camel_tcp_stream_raw_get_fd(), used by the imapx provider.
This function will go away in favor of the general camel_tcp_stream_get_file_desc().

Signed-off-by: Federico Mena Quintero <federico@novell.com>

New method CamelTcpStream::get_file_desc()

This returns a (PRFileDesc *).  We will make CamelTcpStream derivates deal only
in PRFileDesc objects, instead of plain FDs.  This will allow us to implement
TCP socket functionality only once, instead of twice - one for raw streams, one
for SSL streams.  Mainly this will let us have a raw stream and then an
SSL stream overlaid on that one.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelNNTPStore to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelIMAPXServer to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelImapStore to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelSmtpTransport to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelPOP3Store to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Change CamelHttpStream to use the new API

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Do name resolution inside camel_tcp_stream_connect()

This function and the associated vmethod used to take a struct addrinfo *.
However, this doesn't let us have SOCKS4a or SOCKS5 proxies that do name
resolution in the proxy.  So, now the main camel_tcp_stream_connect() takes
a hostname and service name, and implementations do name resolution
by themselves.  Later we will modify the proxy code to do name resolution
in the proxy.

We allow passing a fallback port to camel_tcp_stream_connect(), which
is used by Camel's providers when the system's services database
(/etc/services) doesn't have an entry for a particular service name.
If getaddrinfo() can't find the service name, then we use a
fallback/hardcoded port number.  See bgo#267898

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Eek, typo.

Fix "system" EBook/ECal creation functions.

The local "system" URIs are trivial now, it's just "local:system".

    e_book_new_system_addressbook()
    e_cal_new_system_calendar()
    e_cal_new_system_tasks()
    e_cal_new_system_memos()

Bug 602287 - Avoid racing with camel_operation_mute()

Bug #624891 - Various compiler warnings in git/master

Refactor ECalBackendCache.

ECalBackendCache doesn't need to know the URI of the backend, it just
needs a file name.  Backends must now provide the name of the cache
file.  This breaks the API, but eliminates duplicate logic.

Couple other miscellaneous functions removed:

    e_book_backend_construct()
    e_book_backend_cache_exists()
    e_book_backend_db_cache_exists()

libedata-book's shared object name is bumped for the API break.

Adapt backends to the new API.

Refactor ECalBackendStore / ECalBackendFileStore.

These classes don't need to know the URI and source type of the backend,
they just need a file name.  Backends must now provide the name of the
cache file.  This breaks the API, but eliminates duplicate logic.

libedata-cal's shared object name is bumped for the API break.

Adapt backends to the new API.

ECalBackendFile overrides its cache directory.

It doesn't need a cache, so its cache directory is its data directory.

EDataCal: Add a getCacheDir() D-Bus method.

ECal calls getCacheDir() to find the local attachment store.  ECal
caches the result, so getCacheDir() is only called once per instance.

ECalBackend: Add a "cache-dir" string property.

Holds the directory where data should be cached for that specific
backend instance.

e_cal_backend_get_cache_dir()
e_cal_backend_set_cache_dir()

ECalBackend cleanup.

ECalBackendStore cleanup.

ECalBackendFileStore cleanup.

Updated Galician translations

Fix bug #624909

EBookBackend: Add a "cache-dir" string property.

Holds the directory where data should be cached for that specific
backend instance.

e_book_backend_get_cache_dir()
e_book_backend_set_cache_dir()

EBookBackend cleanup.

There is no need to call e_book_backend_construct().

Header file cleanup.

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Updated Norwegian bokmål translation

Add missing files to POTFILES.in. Fixes bug #624743.

Updated Norwegian bokmål translation

Fixed headers in galician translation file

Updated Spanish translation

ECalBackendFile: Fix an uninitialized variable

Remove deprecated ECalBackend::set_default_timezone

It's deprecated since 2006-07-10

Broke a few things with the local source URI change

ECalBackendFileFactory:

- The factory's get_protocol() method returns "local" now.

- uri_to_path() needs to actually build the path to its ICS file
  now instead of just converting the backend's URI to a filename.

EBookBackendFileFactory:

- Similar fixes.

Avoid absolute paths in "On This Computer" source groups.

A base URI of "local:" is sufficient to identify the "On This Computer"
source group.  No other source group uses that URI scheme, and it avoids
an entire class of portability problems that come with using an absolute
"file:" URI to point to user-specific data:

- What if the user's home directory is renamed?
- What if settings are moved to a new host under a different user name?
- What if settings are copied to another user account on the same host?
- Migrating to XDG base dirs is otherwise a giant PITA.

Fix e-data-book-factory.c regression.

CamelOperation debugging improvement

When calling camel_operation_unref() with a non-empty status stack,
show what status messages were left over in the status stack.

Bug 624417 - Fix error handling in e_cal_backend_sync_get_timezone()

Don't create a GError if we're successful.

EDataBookFactory cleanup.

EDataCalFactory cleanup.

Bug 624029 - Copying messages across stores is broken

Bug 624316 - Disable debug messages in GroupWise calendar backend

Bug 608299 - Cut/copy error when selection has trailing comma

When cutting or copying a selection that includes a trailing comma, the
subsequent address was also getting copied to the clipboard.  The fix is
to ignore trailing whitespace and commas when scanning the selection for
email addresses.

Removed folders from connection when operations gets done - handles when jobs return back without executing

Add connection manager to imapx

Bug 624325 - CamelPOP3Store missing some methods

There's a corresponding change on the Evolution side to catch and clear
the CAMEL_STORE_ERROR_NO_FOLDER that CamelPOP3Store's get_folder_info()
method now sets to satisfy the new runtime checks.

CamelPOP3Store cleanup.

Bug #612082 - Do not expose off_t in public API, use goffset instead

Updated Norwegian bokmål translation

libebook.pc.in missing camel-1.2 requirement.

EDestination uses CamelInternetAddress.

Updated Galician translations

Post-release version bump.

NEWS update for 2.31.5 release.

Bug #618938 - Crash when reading calendar acknowledgement message

Bug #623936 - 75 second delay when creating new DAV entries in Zimbra

Fix another g_propagate_error() runtime warning.

imapx: Ensure imapx_stream_fill() sets the error on zero-sized read.

We treat an empty read as an error, since we should never be here unless
we're either in the middle of reading a response, or poll() says the fd
is readable. Before commit 46938c348 (the conversion to GError), we'd set
a generic 'IO Error' exception in camel_imapx_stream_token() when
imapx_stream_fill() returned -1 to indicate an error. But now we don't
do that any more, so imapx_stream_fill() needs to reliably set the GError.

Otherwise we end up in an endless loop in imapx_step().

Fix runtime warning in camel_filter_driver_flush().

Only propagate the error if there's an error to propagate.

Improve error handling in CamelStreamBuffer::read().

If we managed to successfully read some data before camel_stream_read()
failed, discard the GError and return the number of bytes successfully
read.  If camel_stream_read() failed immediately, propagate the GError
and return -1.

imapx: camel_imapx_stream_astring() returns 0 on success.

Update API documentation.

Coding style and whitespace cleanup.

[i18n] Updated German translation

Updated Norwegian bokmål translation

Coding style and whitespace cleanup.

imapx: Avoid running FETCH_NEW_MESSAGES and REFRESH_INFO jobs simultaneously

There are various places where we interpret FETCH results and use
imapx_match_active_job to find the current job, which will behave badly
if there are two jobs which could potentially be responsible for the FETCH.

In particular, this was causing a problem when we triggered a fetch of new
messages from select_done(), and that command was submitted at the same time
as a refresh_info command to fetch all flags. The server (Dovecot) was
returning all the untagged FETCH results before either completion line,
and all the flags were getting "assigned" to the fetch_new_messages job,
causing a bunch of 'g_array_append_vals: assertion `array' failed' warnings,
and all messages to disappear because the refresh_info job didn't see them.

Updated Norwegian bokmål translation

imapx: Fix handling of return from camel_imapx_stream_text()

It's another of those functions that return 0 for success.

imapx: More improvements to server unseen count tracking

We're using the unseen count as part of our check for when things have
changed on the server (or when QRESYNC has gone wrong), and we need to
rescan all flags. But the server doesn't *tell* us about changes to the
unseen count, like it does for the total count of messages. So we have to
be more careful about tracking it -- especially since the server doesn't
even tell us in SELECT, and we can't issue STATUS to find it out when the
folder is selected.

One place that it was getting out of sync was when the server notified us
about new messages in SELECT, which weren't there when we last issued STATUS.
For messages with a UID equal to or higher than what we thought was the
folder's UIDNEXT, we also have to update the unseen count.

So far, the _only_ time we've hit the QRESYNC sanity check which triggers
a full rescan has been due to this kind of cosmetic unseen count discrepancy.
It's half tempting to disable that check in the QRESYNC sanity check, but
actually it's useful to get it right for the legacy servers too, because it
might cause us to miss updates if we get out of sync.

nntp: Fix two more 'returns FALSE on error' vs. 'returns 0 on success' errors

We really need to start being consistent about this.

Pass the GError pointer through to camel_folder_summary_save_to_db()
from nntp_folder_sync() too... although it seems to be NULL a lot of the
time anyway, but that's the fault of our caller. No need to compound it.

Fix some linker issues.

Bug 624018 - Fix error handling in groupwise_get_folder()

Updated Spanish translation

Bug #623988 - build failure in git/master.

Add the marshal list and also a typo in weather.

Update user documentation about E_CAL/BOOK_DISABLE_DEPRECATED

Bug #623204 - Be able to report detailed errors from backends

[i18n] Updated German translation