Less beeping for ancient messages.

Add chime_conversation_get_created_on()

Don't fetch all rooms at startup if we haven't been mentioned.

This takes a huge amount of time for new clients.

Don't specify organiser to generate_comp() at all.

It was using 'Calendar' for me, and resulting in a non-editable meeting.

Having carefully chosing the right calendar, we should be able to leave
it as NULL and let the default for the calendar be used.

Do not try to get the attachment if the node is NULL

This fixes lots of warnings when trying to get the attachments

Allow joinable meetings to be added

This paves the way for something that looks at the calendar and inserts
the meetings where the organiser forgot to invite meet@chime.aws, or
where distribution lists were invited instead of people, so Chime can't
pop up automatically.

Merge branch 'fix_Evolution_email_mismatch' of https://github.com/dlenski/pidgin-chime

Add End Meeting support

modify Evolution plugin to ensure that a calendar event is always associated with the email address of the source for that calendar

This addresses issue #44, wherein Evolution prevents the user from editing
the created meeting if the Chime username/address doesn't exactly match an
email address configured in Evolution.

Fix for attachment ignore problem

xcbimagesrc: disconnect xcb when clearing the context

xcbimagesrc: remove unused method

Prevent double-close of meetings when chat is closed.

xcb: gather all the screen data using xcb_screen_t

Fix 'Join with audio' on already-open meeting, ending audio closes chat

shm check

wip

Add gst-xcbimagesrc to specfile

Add xcbimagesrc for Nacho to port to XCB

It's still just a copy of ximagesrc from gst-plugins-good, passed through sed.

Fix potential memory leak in screen_send_packet()

Not that we use it in that mode, because it sucks as noted even with
the leak fixed.

No really, only send screen if there's a viewer.

Handle unexpected closure of screen websocket

I don't think this was a cause of dropouts because the reported behaviour
was a stall (no movement in the picture). And when the websocket drops,
the shared screen drops and other participants see that it's gone.

But we should handle it appropriately anyway. By visibly dropping and
reporting an error for now. If necessary we'll make it automatically
reconnect like the audio stream does.

Only send screen data when a viewer is actually present

That is, when we see the first KEY_REQUEST.

Merge branch 'master' of github.com:awslabs/pidgin-chime

Fix outbound screenshare dropouts

I think I've found the reason, along with the reason I could never
reproduce it on demand. The videorate element insists on producing
precisely one output frame for every third of the second... since the
epoch (when GStreamer started up). Not from the start of the call.

So if you've only been running Pidgin for a few seconds and you make
a test call, and it's the first GStreamer pipeline, it all works fine.

Hang up, wait an hour and then share another screen, and it'll send
10800 duplicate copies of the first frame before it ever sends anything
else.

Revert "Reduce outbound screenshare bandwidth"

This reverts commit c1d4c357b4ebda3c3d3faab9f2f21acdd385016b. It wasn't that.

chime-call-transport: use the right format type for size_t

Fix compiler warning in chime_connection_send_message_async()

 (1) Event suspicious_sizeof: Passing argument "&message" of type
     "gchar const **" and argument "8L /* sizeof (gchar const **) */"
     to function "g_checksum_update" is suspicious. In this case,
     "sizeof (gchar const **)" is equal to "sizeof (gchar const *)",
     but this is not a portable assumption.
 (2) Event remediation: Did you intend to use "sizeof (gchar const *)"
     instead of "sizeof (gchar const **)"?

chime-connection: remove useless check

The server variable is checked right before and set to SIGNIN_DEFAULT
if needed.

chat: set error to the right purple media

Fix clang static analysis warnings

Even though the code is ok better to deallocate the data
after it is removing it from the list, to avoid the warning
and to make the code clearer.

Reduce outbound screenshare bandwidth

I've been sharing my screen for an hour now, without any dropouts. Not
really sure if this is it; it's never really been reproducible on
demand when I've been trying to debug it anyway...

Fix use-after-free crash in message gathering

When cleanup_msgs() is called, there may be a pending fetch. We were
seeing crashes when fetch_msgs_cb() ended up using the freed msgs
structure. Let cleanup_msgs() do the freeing instead of the caller
(since we already have this nasty hack that the msgs struct needs
to be at the start of the chat/im structure. And then defer the
final free() until fetch_msgs_cb() is called, if necessary.

Stop i386 builds failing

We don't care about them much, but the PPA builds for i386 and the failures
are gratuitous and annoying.

Sort room list and implement 'Leave Room'

We can't leave rooms from the room list display because Pidgin doesn't
let us add action buttons there, but we could do our own version as a
search results box, triggered from the account menu.

Also haven't implemented 'delete room' yet.

Tag version 1.3

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add Do Not Disturb state

Also fix chime_availability_name() not to crash when it sees a new state
value that isn't in its list.

chat: do not crash if clicking IM without selecting a contact

Tag version 1.2

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Fix dist files to include README.md not README

Drop libecal-1.2 BR for specfile

It's not clear why we have to change the actual pkgconfig module name
instead of just the version. But any attempt to BR one or the other
would be fairly distro-specific anyway, based on %_fedora. So we might
as well just *not* BR it, and depend on the equally distro-specific
knowledge that it's in the evolution-data-server-devel package along
with all the other things that we *do* BR.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Attempt to drop only whole frames, not arbitary gaps.

Perhaps this stops the strange effects we get when we lose outbound
audio frames on a busy system and something goes out of sync.

Update ecal-2.0 port for further upstream API changes

Merge pull request #38 from jtojnar/libecal-2.0

Port to libecal-2.0

Port to libecal-2.0

https://gitlab.gnome.org/GNOME/evolution-data-server/issues/33

Another quickfix to the WarpDrive login.

Update user search to use Express API

Hm, how are we supposed to find it?

Fix g_error_free criticals

Merge pull request #34 from etanol/master

Quickfix to update WarpDrive sign in.

Quickfix to update WarpDrive sign in.

Fix RPM build

Fix C89 build failure on Fedora 28

BUILDSTDERR: prpl/attachments.c: In function 'get_mime_type':
BUILDSTDERR: prpl/attachments.c:523:3: error: 'for' loop initial declarations are only allowed in C99 mode
BUILDSTDERR:    for (int i = 0; i < sizeof(file_types) / sizeof(struct FileType); i++) {
BUILDSTDERR:    ^
BUILDSTDERR: prpl/attachments.c:523:3: note: use option -std=c99 or -std=gnu99 to compile your code

Translate the README file to README.md

Now that we are on GitHub, provide a more appealing README file by translating
the current one to Markdown.

Merge pull request #30 from etanol/master

Update WarpDrive authentication.

Update WarpDrive authentication.

Bring the WD authentication scrapping up to date with the current web version.
The JS file used to extract the GWT policy ID no longer exists.  Also, use newer
versions of the GWT request structures.

Merge branch 'master' of github.com:awslabs/pidgin-chime

Merge pull request #29 from michaelgataws/typing-notifications

Hide own typing notifications

Hide own typing notifications

Own typing notifications (which may appear from other devices) shouldn't
end up in a bogus conversation with oneself.

Change gnutls dep to libgnutls28-dev, working around Ubuntu strangeness

Merge pull request #22 from gqmelo/attachment-upload

Add support to send attachment on a conversation

Tag version 1.1

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Add support to send attachment on a conversation

An attachment can be sent on Pidgin through Send File dialog.
This also supports showing progress and cancelling the transfer.

Currently it works with conversations only (no chat), as the file
transfer callbacks offered by PurplePluginProtocolInfo are not used for
chats.

Merge pull request #26 from etanol/master

The arms race continues

Update sign-in code to match the latest website updates.

Explicitly require and link camel-1.2

https://github.com/awslabs/pidgin-chime/issues/23

Reported-by: Buchan Milne <bgmilne@gmail.com>

Handled "none required" in $ac_search_cv_dlopen on non-Linux

Detect library containing dlopen/dlsym #23

Add missing libraries to underlinked targes #23

* libapp-transmitter.la needs $(FARSTREAM_LIBS)
* libchimeprpl.la needs -ldl

Tag version 1.0

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Revert "Do not BR gcc"

This reverts commit ed8cd436991c6a20715a73cefa814a17f964673d. Turns out it
is needed after all, and fedora-review is out of date.

Fix .deb changelog generation for tagged versions

Tag version 0.95

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

More Fedora review fixes

Reduce Pidgin dep to 2.13.0

Support screen share when built against older libpurple

We can use dlsym() to get at purple_request_screenshare_media(), and now
we can have full functionality even when built against Pidgin 2.13.0 and
only *running* with 2.14.0.

This means the real Fedora build (once it's reviewed) won't be less
functional than the COPR build from the same source.

Support .get_cb_alias even when built against older libpurple

It's harmless to extend the PRPL struct and provide the feature bits as
they'll be ignored. So just do it and don't make the newer libpurple a
build dependency.

media cleanup

Tag version 0.94

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Complete the removal of 24/32 pixmaps

Tag version 0.93

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Always require autohate for COPR builds

Tag version 0.92

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Import tag/dist bits from openconnect

Do not BR gcc

Own /etc/pki/purple-chime

Remove 24x24 and 32x32 pixmaps which Pidgin doesn't use

v0.91

Fix EPEL compat harder

Add certs to dist

Fix COPR build

Allow certs to be put elsewhere

Clean up spec generation.

Do it at autogen.sh and 'make dist' time, not during configure.
Do it with a shell script and simplify .copr/Makefile

Make DTLS cert verification work on EPEL7

Trust Amazon certificates

Some of the media endpoints have certificates issued by the Amazon internal
CA. Trust them. Might as well have the Amazon public trust roots too, just
for good measure.

Display which account we're authenticating to in prompts

Don't soup_uri_free(NULL)

Do not automatically connect upon object construction.

chime_connection_new() automatically calls chime_connection_connect(), but if
the session token is not given, there hasn't been any chance to connect the
"authenticate" signal in order to properly transfer control through the code.

Instead of implicitly connecting upon object creation, let the caller do it
explicitly.  The purple plugin is also changed to conform to that.

Remove PurpleConnection from ChimeConnection.

This commit efectively completes the decoupling of libpurple specific glue code
from the self contained Chime client code.  Fixes #2.

Refactor the login/sign-in code.

To conform with the backend/frontend split of this package, the sign in code has
been refactored by separtaing the screen scrapping parts from the front
end (user interaction) parts.

The split between sign in provider types is no more, so all the login dirty code
lives together in chime/chime-signin.c.

Also, a new utility (chime-get-token) has been incorporated to ease the debug of
the sign in process, and also as an easy way to obtain a session token for Chime
bot developers.

Validate DTLS server certificate

Hm, it really shouldn't be so easy to forget this. There really is a case
to be made that you should get GNUTLS_E_CERTIFICATE_VERIFICATION_ERROR by
default *unless* you set up verification. Fail safe, not open.