Eliminate GTimeVal from rooms.c

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

Eliminate GTimeVal from chime-room.c

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

Eliminate GTimeVal from chat.c

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

Kill parse_time() and replace it with iso8601_to_ms()

GTimeVal has been deprecated; we need to use GDateTime to parse ISO8601
strings. All we really want is milliseconds so just create a helper to
do that, and convert the one user of parse_time() to it for now. Other
callers of g_time_val_from_iso8601() will use it next...

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

Remove deprecated g_type_class_add_private() for ChimeConnection

Sadly we then have to add an accessor function for it, which shows just
how non-private it actually is...

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

Update login to use external web browser

The login process changed again and this time we really do need a browser.

Use purple_notify_uri() to spawn it, and now we get to specify our own
callback_uri for it to redirect to with the token. Use chime:// because
we already have the scripting set up to capture that.

This means we can fix the problem of multiple accounts with the chime:
scheme handler, by passing the devtoken of the account in question. We
can also pass the UI name so that we know which UI to re-enable it in.

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

Don't run autogen.sh from debian rules

Let's use 'make tmp-dist' to prepare tarballs.

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

Use make variables in debian/rules.maint

This makes it easier to override them when doing an out-of-tree build
after 'make tmp-dist'.

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

Use POSIX tar format for dist

Tag version 1.4.1

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

Add markdown.h to dist

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

Add desktop file to EXTRA_DIST

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

Add scripts to EXTRA_DIST

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

Tag version 1.4

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

Fix meeting ended signal handling

The arguments for the handler have *always* been wrong; it only ever
worked by fluke because the 'conn' must have ended up in the right
register for the third argument.

Add hackery for authentication via browser

Rename joinable.py to chime-joinable.py and install it

Merge branch 'deb-updates' of github.com:nmeyerhans/pidgin-chime

packaging: templatize debian/control

Perform substitution of Build-Depends and Depends per Debian/Ubuntu
release.  Build and binary dependencies for several releases are
provided; additional releases are simple to add.

In order to generate the .deb packaging metadata, a workflow such as
the following can be used (note that this assumes apt >= 2.0, which
introduced the 'satisfy' subcommand; on earlier releases, use a
combination of dpkg-checkbuilddeps and manual apt invocations or
pbuilder(8)):

$ make -f debian/rules.maint debian/changelog
$ sudo apt satisfy --no-install-recommends build-essential \
  "$(grep-dctrl -n -s Build-Depends . debian/control)"
$ dpkg-buildpackage -uc -us

This should generate proper Debian binary and source packages
targeting the current distribution provided the following conditions
are met:
1. The build is happening within the target distribution
2. The make, git, lsb-release, and dctrl-tools are installed

debian/control is generated with Build-Depends and binary Depends
substitute from the appropriate debian/deps/build-* and
debian/deps/bin-* files, respectively.

debian/changelog is generated based on git tag data with various
release and version fields substituded with the appropriate values for
the current distribution environment (e.g. Ubuntu focal or Debian
buster).

Signed-off-by: Noah Meyerhans <nmeyerha@amazon.com>

packaging: update external links in debian/control

Add missing Vcs-* headers and correct the Homepage location

Signed-off-by: Noah Meyerhans <nmeyerha@amazon.com>

Added a new 'video' indicator

Also added a 'ping' handled for video; but it seems harmless and without
any affect :)

Fix CHIME_PROPS_NEWOBJ

Where nick != name the properties weren't getting set. For some reason
I wasn't seeing the warnings that Nacho was.

Fix leak of machine_id

Merge branch 'wip/nacho/fixups' of github.com:awslabs/pidgin-chime

signin: do not use a weird opaque structure

We can save the data inside the object itself if required.

chime-connection: set the default value of the server property

This is in case that we want to use g_object_new instead of the
default constructor

Add attachment upload support for chats

Based on PR#36 by Guilherme Melo but teaching Pidgin that chats can
have files instead of manually adding a separate 'Send File' menu item:
https://bitbucket.org/pidgin/main/pull-requests/701

Cope with screen==NULL in screen_appsink_new_sample()

I saw a crash with this once, in a long sequence of sharing from A to B
and then B to A testing screen sharing startup/teardown.

It looksl like a race condition when we change to the no_appsink_callbacks
but the callback is already in progress, and the cbdata gets set to NULL.

Fix EPEL7 build.

Yeah, we still can't use C99 unless we explicity ask for it.

prpl/markdown.c: In function 'do_markdown':
prpl/markdown.c:48:2: error: 'for' loop initial declarations are only allowed in C99 mode
  for (char *p = *outbound; *p; p++) {
  ^

Add libmarkdown dep to specfile

Merge github.com:phillipberndt/pidgin-chime

Update display name for newly-created peers as they're discovered

I said instantiate *conversations* when they're opened...

Include <string.h> for dbus.c

Instantiate conversations as soon as they're opened; not on first send

Refine criteria for setting PURPLE_MESSAGE_DELAYED flag

Not for *all* messages in the last day, but only for the *last* message, if
it's less than a day old.

Fetch ancient messages in 2-week chunks

Because the server doesn't play them in chronological order and we have to
sort them, this takes a lot of memory if we do years worth of messages in
one hit. Instead, fetch a heuristically reasonable sized amount at a time.

Fix msg_newer() which was returning the wrong answer, while splitting
out msg_newer_than().

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

Improve markdown experience

This replaces some unsupported HTML tags back to text manually.

Process markdown in Chime chat messages

Add markdown support

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

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