Release 1.25.0 -- Buddy Idle Time, RTF

purple: add missing "const" to "char *"

Enabling -Wwrite-strings flagged all the places where the const
qualifier was incorrectly left out.

The g_hash_table_insert()'s are also correct, because the hash tables
are using string contants as keys.

core: add missing "const" to "char *"

Enabling -Wwrite-strings flagged all the places where the const
qualifier was incorrectly left out.

NOTE: the sipe-sign.c implenentation would need a complete rewrite to be
      -Wwrite-strings clean.

configure: document that -Wwrite-strings is missing

Quote from GCC manual:

   "-Wwrite-strings

    When compiling C, give string constants the type const
    char[length] so that copying the address of one into a non-const
    char * pointer produces a warning. These warnings help you find at
    compile time code that can try to write into a string constant, but
    only if you have been very careful about using const in declarations
    and prototypes. Otherwise, it is just a nuisance.

    This is why we did not make -Wall request these warnings."

While this statement may be true for old code, new code/APIs should
compile with the flag enabled.

At least now this ommission is documented and it is easy for developers
to activate it to check their code.

update files in preparation for next release

OBS: add BR libnm for Mageia 8+

It seems that that dependency is missing in libpurple-devel.

purple: maintain libpurple < 2.14.0 compatibility

media: use purple purple_media_set_require_encryption api

This patch uses the require-encryption property of the farstream
conference element, to block by default the stream at the srtpdec
element at creation time, until we know from sdp whether it will be
encrypted or not. The installation of the keys in the element unblocks
the stream, by processing it through the decoder, else the toggle of the
property to "false" also unblocks the stream in "pass through" mode.

This patch prevents a crypted stream to pass through the srtpdec element
in the short time frame before the crypto elements have been installed,
and to reach the rest of the pipeline, causing codec decoder errors
where decrypted stream is expected.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/1016

purple: update to latest 3.x.x xfer interfaces

purple: rename Iface -> Interface in 3.x.x API

purple: update to latest 3.x.x blist interfaces

configure: update minimum build tool versions

Minimum versions required for maintainer builds, i.e. to regenerate the
configure script:

- autoconf >= 2.69
- automake >= 1.16
- gettext  >= 0.19.8

Minimum versions required for release builds, i.e. to execute configure:

- intltool >= 0.41.0

documentation: remove HACKING file

The URL given in the text was obsolete. Replace the file with a new
section added to the main README file.

documentation: add build from git instructions

This is not meant to be comprehensive but as a starting point for a
developer.

documentation: update configure rebuild steps

Update the instructions to reflect the latest contents of autogen.sh.

configure: restore GITVERSION generation

Too eager delete key...

configure: simplify autogen.sh

Instead of executing every tool by hand just use the autoreconf tool
which already executes all tools in the correct order.

configure: update gettext integration

- configure.ac: set gettext minimum version to 0.19.8
- autogen.sh:   add autopoint to setup sequence
- .gitignore:   add new maintainer files to ignore
- Makefile.am:  add new maintainer files to delete
- remove dummy ABOUT-NLS/config.rpath files again

configure: remove AC_PROG_RANLIB

According to autoreconf warning it is obsoleted by LT_INIT.

configure: remove m4macros directory

As we no longer have any source code in it, we no longer need it in the
build. Change the aclocal options to use/generate the standard "m4"
directory on-the-fly and tell git to ignore it.

configure: makefile cleanup

- Makefile.am: add ylwrap to list of maintainer files
- purple.m4:   this file seems to be obsolete, just remove it

configure: fix build against latest gettext

- configure.ac: replace old macros with AM_GNU_GETTEXT
- ABOUT-NLS:    empty file, but it must exist or auto* fails
- config.rpath: empty file, but it must exist or auto* fails

Fix #358: FTBFS with glib-2.0 >= 2.62.0

The new glib-2.0 release will introduce new deprecations. The
devel-release is already in Fedora Rawhide and Mageia Cauldron.

telepathy: include config.h in more modules

Otherwise GLIB_VERSION_MIN_REQUIRED is not defined, which causes build
failures due to new deprecations in glib-2.0 >= 2.62.0.

purple: include config.h in more modules

Otherwise GLIB_VERSION_MIN_REQUIRED is not defined, which causes build
failures due to new deprecations in glib-2.0 >= 2.62.0.

configure: set min version for glib-2.0 >= 2.62.0

- libpurple API uses GParameter, which will be deprecated in 2.62.0.
- telepathy-glib API uses GTimeVal, which will be deprecated in 2.62.0.
- libpurple 2.x API uses G_CONST_RETURN, which has been deprecated since
  2.30.0, but only in 2.62.0 a versioned deprecation has been added.

The configure code now sets GLIB_VERSION_MIN_REQUIRED to the lowest of
2_60, 2_30 or 2_28 depending on what backends are configured in the
build.

core: add generic tests module

Start by adding tests for the re-implemented time conversion APIs in
utils module.

utils: use GDateTime when glib-2.0 >= 2.56.0

GTimeVal and related APIs will be marked as deprecated in 2.62.0.
All APIs we need to implement our functionality using GDateTime are
available since 2.56.0.

l10n: sync translations with transifex.com (fr, nl, pt, pt_BR, zh_CN)

contrib: remove support for obsolete platforms

Ubuntu 16.10, 17.04 & 17.10 have reached EOL. As their builds have a new
BR the build will fail, because the packages for those older versions
have been removed and the build will only succeed by accident if the
build system has the packages cached somewhere.

core: fix md4sum() stub signature in test

Now the stub in the test code agrees with the signature in md4sum.h.

core: fix -Wmaybe-unitialized warnings

Reported by GCC on openSUSE Tumbleweed.

adium: add RTF module to build

Add sipe-rtf.l to the Xcode build.

The default "System Lex rule" doesn't work for us as it expects the
wrong output file name in the wrong location. Create a custom build
rule using a script that calls $(LEX), which is actually "flex", and
generates files in the derived file directory with our file name
convention.

purple-media: fix a gstreamer leak

OBS: add freerdp & gssntlmssp for Mageia 7+

This enables full appshare & single sign-on experience.

contrib: switch to SPDX identifier GPL-2.0-or-later (II)

This fixes commit 96832b90503c6cd8248b649bbbec43b65e60594b

Add the missing changelog entries in the SPEC files.

OBS: add BR gss-ntlmssp-dev to Debian/Ubuntu

gss-ntlmssp 0.7.0 is available since Debian 9.0 / Ubuntu 16.04. Update
the DSC files so that users on those systems can enjoy the full single
sign-on experience with SIPE.

purple: fix "(null)" login value in finch

Pidgin works fine with NULL as default login name, but finch returns the
incorrect string "(null)" for it. Change the default to empty string.

purple: replace deprecated utility API

purple_user_dir() has been declared deprecated in the 3.x.x API. Replace
it with purple_config_dir().

I'm not sure if this will break the functionality of the generated
fs-code.conf file, though...

purple: update to latest 3.x.x plugin interfaces

purple: add wrappers for authorization callbacks

The callback type for the purple_account_request_authorization() API has
changed in 3.x.x. It is no longer compatible with the callback type
defined in our core<->backend API.

Hide the differences by introducing a new data structure and wrappers
that ignore the additional parameter introduced in 3.x.x API.

purple: remove const from presence/status types

In 3.x.x API the return types are no longer marked as const. Removing
const is compatible with 2.x.x API.

purple: update for action menu API 3.x.x rename

PurpleMenuAction         .> PurpleActionMenu
purple_menu_action_new() -> purple_action_menu_new()

purple: add _PurpleProtocolAction fwd declaration

This fixes most of the compilation errors against latest
3.x.x API.

Add action.h where necessary to get the real type.

purple: move D-Bus functionality to 2.x.x API

D-Bus support has been removed from upstream master branch.

core: switch to SPDX identifier GPL-2.0-or-later

Newer distros no longer accept the old GPLv2+ identifier.

purple: add note about SPDX identifier

Unfortunately most appstream versions do not accept the latest
official identifier GPL-2.0-or-later.

contrib: switch to SPDX identifier GPL-2.0-or-later

Newer distros no longer accept the old GPLv2+ identifier.

OBS: disable empathy on newer RPM distros

Fedora 30+ and openSUSE LEAP 15.0+ no longer package empathy. Introduce
flagging into the SPEC file to handle this case.

configure: document oldest supported LTS releases

Add a table documenting the oldest LTS releases SIPE supports and what
the versions of the main dependencies for SIPE are in those releases.
Also document the EOL for each release.

This should ease the future retirement work.

contrib: remove support for obsolete platforms

Debian 7 and Ubuntu 14.04 have reached EOL.

purple: move version check

The minimum version check is for purple-2, hence it should be in the
purple-2 only code to simplify the flagging.

telepathy: add minimum version check

contrib: rause BR telepathy-glib >= 0.24.0

Update DEB and RPM files accordingly.

configure: rause BR telepathy-glib >= 0.24.0

Debian 7 & Ubuntu 14.04 have reached end of life, hence we can raise
the BR for telepathy-glib to the lowest common denominator.

purple: discard the use of the global stun server

When the user has configured a stun server in the network perferences of
pidgin, it'll be passed to the farstream backend automatically, and may
generate srflx local candidates, that fail to be nominated by SfB, when
pidgin/sipe is in controlled mode (ie when in the position a receive a
stream from a SfB peer), and behind a NAT-ted network.

ms-client-diagnostics: 21
"Call failed to establish due to a media connectivity
failure where one endpoint is of unknown type"

A supplementary condition is that the NAT does not randomize ports
allocation, so the srflx candidate remains usable during the conncheck.

Just removing the possibily to generate these srflx candidates at
gathering time, by discarding the use of a custom stun server seems to
resolve this issue.  This is not a loss a pairs combinations, because in
the typical situation, the srfx local candidate discovered by the stun
server, is also discovered by the conncheck, as a prflx local candidate
this time.  And a candidate of this type is happily accepted by SfB.

core: fix memory leaks

media: schedule a timeout for streams not established yet

This fixes a bug where adding video to an audio stream caused the
whole session to disconnect after a 30 seconds timeout.

purple-media: workaround a heap-use-after-free

core: make connection debug messages consistent

Another round to simplify filtering of debug logs:

- rename sip_transport_disconnect -> sip_transport_drop
- remove "dropping connection" text
- always use format '<host>:<port>'(0x<address>)

Example filter strings for the Pidgin debug window:

    _transport_(connected|drop)
    '[a-z0-9.]+:[0-9]+'\(0x[0-9a-f]+\)

core: take sipmsg_find_to_header() into use

DRY cleanup.

core: take sipmsg_find_from_header() into use

DRY cleanup.

core: take sipmsg_find_expires_header() into use

DRY cleanup.

core: take sipmsg_find_event_header() into use

DRY cleanup.

core: take sipmsg_find_cseq_header() into use

DRY cleanup.

core: take sipmsg_find_content_type_header() into use

DRY cleanup.

core: take sipmsg_find_call_id_header() into use

DRY cleanup.

sipmsg: add helpers to find common headers

DRY cleanup

core: fix inconsistency in Contact address parsing

Address from Contact header was parsed in 2 places and both were using a
different approach. Added sipmsg_parse_contact_address() API to remove
this inconsistency, similar to the handling of From/To headers.

I also noticed that the address parsing APIs can and should accept a
const struct sipmsg pointer. Updated code accordingly.

core: factor out code that adds tag to To: header

DRY cleanup: the exact same code was repeated in 3 different places.

Fix #350: Inconsistent parsing of From:/To: headers

sipmsg_breakdown_parse() is now parsing the address out of those fields
like all other places. Message signature verification should now work
also in those cases where URIs are not wrapped in "<...>".

sipmsg: fix test build error on RHEL6

It complains about missing field initializers.

sign: fix inconsistency in From/To address parsing

sipmsg_breakdown_parse() was hard-coded to look for "<...>" wrapped URIs
in the From/To headers. Use parse_from() like all other places. This
means that messages with non-"<...>"-wrapped URIs will now pass message
verification.

core: take sipmsg_parse_address_from_header() into use

DRY cleanup. Now only two places outside sipmsg.c are left that call
parse_from() directly.

core: take sipmsg_parse_to_address() into use

DRY cleanup.

core: take sipmsg_parse_from_address() into use

DRY cleanup.

sipmsg: add parse address from header APIs

Start of DRY cleanup for parse_from(sipmsg_find_header(...)) code.

- generic API    sipmsg_parse_address_from_header()
- special cases: sipmsg_parse_(from|to)_address()

sipmsg: add tests for P-Asserted-Identity parsing

Convert the examples in the sipmsg.h comments to real tests.

sipmsg: add test program for message tests

Factor out non-NTLM tests from sip-sec-ntlm-tests.c to the new tester
sipe-sipmsg-tests.c. Those tests should always be executed, even when
SIPE NTLM implementation isn't compiled in.

rtf: fix tests for older C compilers

Move the variable definition out of the for loop.

rtf: fix build error in NTLM tests

NTLM tests include sipmsg.c which now depends on sipe-rtf.c

rtf: rewrite state initialization

On some platforms GCC requires to initialize all fields, even the ones
that don't need it. Rewrite the initialization to only set those state
fields that require it.

rtf: fix build error on Windows

This fixes commit d7faa8e036db6b9495f08d03e4e5bdfe5ab6b483

Feature #77: RTF support (incoming)

Instead of showing an error message to the user we now extract the plain
text from incoming RTF and pass it as HTML to the backend.

incoming: allow MIME type text/rtf

get_html_message() now knows hwo to transform it to HTML.

sipmsg: take sipe_rtf_to_html() into use

Convert MIME type text/rtf to HTML in get_html_message().

rtf: add helper functions to tests

Now the RTF tests compile and pass.

rtf: take parser function into use

This completes the initial implementation for the RTF module.

rtf: add parser function

A simple parser function, based on a Bison parser, that processes RTF
keywords. It currently only recognizes the following two keywords

- \uc<N> - N replacement characters after Unicode keyword (\u)
- \par   - paragraph, transformed to HTML <br/>

Everything else is ignored.

rtf: add lexer rules

- consume anything that we do not want to parse
- plain text and characters are added directly to the output buffer
- RTF keywords and parameters are passed to the parser

rtf: add tests for RTF to HTML transform

rtf: add memory allocation code for lexer

We use the flex options noyyalloc/noyyfree/noyyrealloc to be able to
provide a memory allocation implementation based on glib2.

Fix file mode flags

Remove execute flag from files that do not need it.

rtf: add module for RTF processing

This adds a function to extract plain text from RTF and convert it to
HTML. The implementation will use a flex-based lexer and a hand-written
parser.

Start off by adding the lexer module and integrating it to the build.
The release tarball will include the pre-generated lexer C code, i.e.
the flex tool is not required for a normal build. Update the
documentation accordingly.

Feature #107: Provide idle start time for a buddy

The value of the optional "lastActive" attribute in a presence
notification is now parsed and passed down as seconds since epoch
timestamp to the backend.

User visible changes in the Pidgin UI:
* idle users are now "greyed out" like offline users
* idle timers now work when "Show" -> "Idle Times" is enabled

notify: parse lastActive attribute from state node

Buddy presence "application/msrtc-event-categories+xml" does have an
optional attribute "lastActive" in the categories/category/state XML
node that indicates when the buddy entered the current idle state.
Parse the contents of that attribute as ISO8601 timestamp and pass it
to sipe_core_buddy_got_status().

purple: update for sipe_core_buddy_got_status()

Purple backend calls this API to implement a workaround. Use 0 for the
last_active value, i.e. "unknown when buddy entered idle state".

buddy: update sipe_core_buddy_got_status() API

Add a new time_t parameter that provides an indication to the backend
when the buddy has entered the current idle state. This parameter is
only valid when the "activity" indicates an idle state. The parameter
may be 0 if it is unknown when the buddy entered the idle state or he
is not idle.

Pass the new parameter down to sipe_backend_buddy_set_status().

miranda: update sipe_backend_buddy_set_status()

Add new last_active as unused parameter.

telepathy: update sipe_backend_buddy_set_status()

Add new last_activity as unused parameter.