No need for backslashes

attachments: remove exec permission

More audio cleanup

Fix up lifetime handling on the appsrc/appsink and uninstall the callbacks
when we're done.

Also fix up the timer handling for empty outbound packets. When the system
was slow we were sending 'muted' RT packets in between the audio frames,
so set the timers *only* when we're actually muted. If we fail to send
audio at other times, let it just do a red mic like it should.

Slight cleanup for call media handling

The call is connecting before do_join_chat() is ever called, which means we
may miss some of the early state transitions. Try to cope with that.

We probably want to drive the call connection directly from chat code in
the end, but this will do for now.

Also decouple the media setup and connection; the call window in Pidgin
should now say 'Calling...' and then 'Call in progress' once it's connected.

Hopefully this helps to deal with some of the issues with slow call setup.
I *was* able to reproduce the ever-slower call connection but now I can't.
Not sure if that's just coincidence...

Fix up some muted/silent confusion

These need not to be const now that we actually free them.

Slight cleanup of media setup, leak fixes

Fix unimplemented free_participant()

Probably didn't need valgrind to see *that* one was a leak...

Clean up fetch_presences() idle handling

Properly avoid re-running the function when it's already scheduled, and
ensure we cancel it when we need to too. Also ensure that disposed contacts
are removed from the 'contacts_needed' list.

Free members list for group conversations

Fix leak of data messages

Make the package require gst-plugins-bad

Only consider active members for mentions

Inactive members are effectively not part of the chat anymore and
shouldn't be considered for mentioning. This is especially helpful when
a member changed their profile (e.g., mail, domain, whatnot) but not
their display name.

Prevent double-replacements in mentions

Outgoing mentions replace a name with a reference to the Chime user.
Sometimes you have two users with the same name in a chat room, or two
people where one name is a prefix of another one. This could end up with
something like `<@id1|<@id2|Last, First>>`, which is broken.

This change makes sure to only replace at word borders and not if a `|`
is in front. This isn't perfect, but better than before.

Fix error handling when chime payloader not present

Enforce mono

Don't force either application/x-rtp or application/x-srtp

It shouldn't matter if gst-plugins-bad is installed.

cf. https://bugzilla.gnome.org/show_bug.cgi?id=793704

Dump gst graph on PURPLE_MEDIA_STATE_CONNECTED to get better view

The first dump isn't as useful because a lot of it isn't fully
configured until packets start flowing.

Remove Pidgin 2.13.0 checks for audio UI

Now we're using fsrtpconference, it won't crash.

Add GStreamer module to debian package

Add prototype for fs_init_plugin

Merge branch 'audio'.

I've been using it for every call I've made this week. There are still a
few things to be cleaned up, but I suppose I should concede it's time it
was merged into the master branch for others to test and heckle.

This ought to now work with the Ubuntu 16.04 packages of GStreamer,
FarStream, etc.

Use camel_address_length() to fix build with newer EDS

Add Chime-specific RTP payload handling

RFC7587 defines the clock-rate for the OPUS payload format to be 48000.
This is painful since we're really quite like to be able to transpose
timestamps between the RTP and on-the-wire Chime protocol (which uses
16000) without conversion. It was OK having to multiply by three for the
appsrc, but dividing by three in the appsink would have overflow issues
and we'd have to reconstitute the high bits somehow.

By defining our own payload format we can work around that, and also
force it to encode only a single channel. That one might well be a bug
in the existing gstopuspay; if the output caps include 'stereo=0' it
*isn't* enforcing 'channels=1' on its input caps.

Both of the above could probably be fixed up in the OPUS payloaders
upstream, and once this code is public we can have a look at doing that.

For now though, having our own CHIME payload helps it work on Farstream
0.2.7 on Ubuntu 16.04, which can't cope with payloaders that have
multiple names (OPUS, X-GST-OPUS-DRAFT-SPITTKA-00). So it'll do for now
until we can get more traction on upstreaming the required fixes.

Cope with RTP as application/x-srtp by depayloading for ourselves

Farstream's FsRtpConference always installs the srtpenc/srtpdec elements
even when there is no encryption. In that case they just pass packets
through unmodified. Unfortunately the caps are negotiated wrong — it's
still marked as application/x-srtp. This means that rtpopusdepay won't
accept it, for example.

https://bugs.freedesktop.org/show_bug.cgi?id=105193
https://bugzilla.gnome.org/show_bug.cgi?id=793704

Work around this by actually *asking* for SRTP even though that's not
what we need. And doing the depayload for ourselves.

Now this should work with and upgraded FarStream 0.2.8 and everything
else on Ubuntu 16.04 out of the box.

Create RTP directly for incoming audio

It doesn't look like GStreamer/Farstream can cope with Opus with an RTP
clock rate other than 48000, but for incoming we can cope by just
multiplying by three.

Switch to using RTP for audio

We get a jitterbuffer for free this way, which makes it a lot more usable.

It also means I can drop a lot of the timestamp mangling crap and just
transpose the protocol's sample counts into RTP headers... I think. The
idea is to drop the 'opusrtppay' part and feed RTP directly into the
appsrc, and the converse for the sink.

It does require farstream 0.2.8 *and* to remove libgstsrtp.so for now.

Fix error handling in conv_membership_jugg_cb()

... and a typo spotted while checking it.

Fix local 'node' shadowing parameter in send_message_cb()

Fix ChimeContact dispose/finalize method typo

Remove some chime-connection-private.h includes

Hopefully make Coverity a little less unhappy with hashing &message

Fix copy-paste error spotted by Coverity

drop audioconvert/audioformat from src pipeline

Fix media close

Clean up FsAppTransmitter

Synchronise mute state with server

Farstream kind of working now...

Attempt to migrate audio hacks to FarStream (non-functional so far)

Put back the auto-connect to see if it helps (it doesn't)

Fix FsAppTransmitter for pipelines with more than one element

Oops, I broke muted calls. Fix timed RT send again.

Move audio hacks out to chat.c and leave the Chime parts relatively sane

One step closer to doing things "properly"...

Further audio cleanup

Start splitting out the parts we want to keep, from the temporary
pipeline hacks.

Let fs-app-transmitter launch a pipeline by its description

Instead of hard-coded audiotestsrc/filesink, let the pipeline descriptions
get passed in in the FsCandidate's 'host' and 'username' fields, much as
the FsShmTransmitter does for the filenames.

This might not last, as we can only reference named elements in a pipeline
description if they were created in that same pipeline, AFAICT. So my plan
of pre-creating the appsrc and then referencing it from the description
probably isn't going to work.

Which means we might end up with a dedicated FsChimeTransmitter after all,
but hey, at least I have a better understanding of how some of this stuff
fits together...

Start cleaning up audio support, handle mute

There are locking issues here. The appsink_new_sample() function can get
invoked from another thread, which means it can race with the timer.

I think if the g_source_remove() in do_send_rt_packet() runs while a
timed_send_rt_packet() is waiting for the rt_lock, it can deadlock.

The simple fix is probably to shift all the actual packet processing into
a separate thread with a simple packet-sending loop which does both audio
and muted frames.

Or maybe we don't want a thread wakeup to add to the outbound packet
latency, and we'll allow appsink_new_sample() to send directly for itself
with some simple locking scheme between it and the thread.

The thread can then handle DTLS and auth and everything else sequentially,
which lets me do DTLS "natively" with GnuTLS and saves me from lamenting
the absence of GDtlsClientConnection too much.

Reinstate paulbohm@'s gstreamer hacks

Emit signal for audio transport state

Add UI for rejoining call without mute

Expose UI for joining audio calls

Initial attempt at debian packaging

Fix never-notify pref matching

Add libmarkdown (not used yet, but it's being worked on)

Merge pull request #11 from gqmelo/fix/attachment-messages

Fix attachment messages when the user sends it from other clients

Fix protobuf deps harder

chime-call.c needs the generated protobuf headers

Disconnect signals before tearing down state

Otherwise, the new-meeting and other signals could fire and add things
back when we're tearing it down.

Don't update joinable if it isn't being displayed

This fixes a use-after-free problem on shutdown, as the signals are
still connected.

Fix two refcounting issues in pin_join_cb()

First, own the meeting. Second, unref the task when we're done (by removing
the stray 'return').

Ref connection for fetch_presences()

Fix attachment messages when the user sends it from other clients

On an IM conversation between "A" and "B", if "A" sent an attachment
from other client (e.g. Web), "A" would see the message on pidgin as
being sent by "B".
Also "A" would receive a notification from pidgin for its own
attachments.

Fix #10
https://github.com/awslabs/PRIVATE-purple-chime/issues/10

Fix audiows leak on close

Fix meeting refcount leaks

Stop leaking meeting organiser

Call cleanup

Split out chime-call-transport.c so we can add DTLS more easily. Deal with
opening/closing the "Call Participants" window by adding a chat menu, which
will also be the basis for starting an audio call.

Don't send LastSeen updates when there are no new messages

Add ref to ChimeConnection from each ChimeObject

This makes life a lot easier.

Destroy chats on disconnect

Fix message handling during disconnect

This was causing a refcount leak as the task wasn't completing.

Fix support for more than 100 contacts

Oh joy, there's a completely separate way to handle fetching of contacts
in stages. The 'NextToken' is in a special header instead of in the JSON
response, but ultimately it's very similar. Make fetch_contacts() work
like fetch_conversations() and fetch_rooms() do.

This fixes the disappearance of contacts, when you have more than 100.
Which was partly masked by the fact that we weren't deleting the buddies
from Pidgin.

Fix buddy removal when they're no longer in the contacts list

We're supposed to remove purple buddies when they are no longer in the
contacts list, but we weren't removing them if they were visible to us
at all (by virtue of being in a conversation or a room).

Explicitly close ChimeConnection in chime_purple_close()

Instead of allowing it to be implicitly closed by the final unref, close
it explicitly. This allows us to fix up the refcounting elsewhere, so
the objects associated with the connection can just hold a ref to it.

It was kind of OK for rooms/conversations but it's getting silly with
meetings adn calls.

Send correct message ID

Merge branch 'master' of ssh://kaos-source.amazon.com//kaos/git/stash/dwmw2/chime

Only mark messages as read when they really are

Remove some redundant HTTP messaging code.

SoupSession has a provision to define the User-Agent to use for all the
SoupMessages sent through it.  This simplifies the login code a bit.

Add a comment about how GWT escapes "|".

Escape the "|" character in GWT requests.

Apart from the backslash "\", the pipe character "|" also needs to be escaped
for obvious reasons.  Not escaping it breaks the format of the GWT requests.

Fix type in configure.ac

Typo caused ./configure: line 13311: ]: command not found

Translate messages and show attachments as file:// links

Initial support for receiving attachments

- uses purple_util_fetch_url_len() rather than hijacking the soup session
- saves attachments in $HOME/.purple/chime-purple/$email/downloads
- displays attachment in the chat or conversation if content type is image
- notifies of downloaded file if attachment is not an image
- does *not* handle variants

Fix build against farstream 0.2.7 properly this time

Bump release to 0.02

Fix reliability of login with Amazon (public) account

The form is always named 'signIn', it seems. But doesn't always have an id.

Build FsAppTransmitter as a separate library to make it work with 0.2.7

Require farstream 0.2.8 for audio support

Remove 'wait-for-connection' property

Now we should have FsAppTransmitter "working", at least with the
audiotestsrc and filesink for now. Hooking it up to an actual
appsrc/appsink and to paulbohm's code remains to be done...

Switch to "pidgin" as platform type

We should be OK to stop telling people not to use this now, and automatically
publish it to APT repositories etc.

Not working with filesink

The fsappbin1 bin never entes the PLAYING state, for some reason. Hopefully
I can get a coherent response to
https://lists.freedesktop.org/archives/farstream-devel/2017-November/000079.html

Working with audiotestsrc

Add a FsAppTransmitter

Close audio when done

Fix another conversation create failure

Clean up participants 'search results' properly on close

Shut up!

Show call participants

It'll pop up again as soon as the user closes it, which we should fix and
also make it possible for them to bring it up.

Explicitly open/close ChimeCall, clean up participant handling

A little less "Look Ma! I can get audio data!" and a little more
competent software engineering.

Fix lack of cast in logout_done()

Abort connection when server kicks us off

Oops, this snuck in with another commit. I'm willing to rewrite
history to fix that, even though I wouldn't do it to fix the stupid
"calender" typo that inexplicably slipped in last week.

Implement log out

We will want to change out platform type from "osx" to "pidgin", which
requires logging out and back in again. So implement logout...