Thu, 22 Aug 2013 01:36:58 +0530
Merged soc.2013.gobjectification branch
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
1 | /* |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
2 | * purple - Jabber Protocol Plugin |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
3 | * |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
4 | * Purple is the legal property of its developers, whose names are too numerous |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
5 | * to list here. Please refer to the COPYRIGHT file distributed with this |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
6 | * source distribution. |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
7 | * |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
8 | * This program is free software; you can redistribute it and/or modify |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
9 | * it under the terms of the GNU General Public License as published by |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
10 | * the Free Software Foundation; either version 2 of the License, or |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
11 | * (at your option) any later version. |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
12 | * |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
13 | * This program is distributed in the hope that it will be useful, |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
14 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
15 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
16 | * GNU General Public License for more details. |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
17 | * |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
18 | * You should have received a copy of the GNU General Public License |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
19 | * along with this program; if not, write to the Free Software |
|
28321
c8d617c408ab
Update various header copyrights thanks to licensecheck.
Paul Aurich <darkrain42@pidgin.im>
parents:
28048
diff
changeset
|
20 | * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
21 | * |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
22 | */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
23 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
24 | #include "internal.h" |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
25 | |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
26 | #include "http.h" |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
27 | #include "useravatar.h" |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
28 | #include "pep.h" |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
29 | #include "debug.h" |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
30 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
31 | #define MAX_HTTP_BUDDYICON_BYTES (200 * 1024) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
32 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
33 | static void update_buddy_metadata(JabberStream *js, const char *from, xmlnode *items); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
34 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
35 | void jabber_avatar_init(void) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
36 | { |
| 26959 | 37 | jabber_add_feature(NS_AVATAR_1_1_METADATA, |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
38 | jabber_pep_namespace_only_when_pep_enabled_cb); |
| 26959 | 39 | jabber_add_feature(NS_AVATAR_1_1_DATA, |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
40 | jabber_pep_namespace_only_when_pep_enabled_cb); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
41 | |
| 26959 | 42 | jabber_pep_register_handler(NS_AVATAR_1_1_METADATA, |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
43 | update_buddy_metadata); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
44 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
45 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
46 | static void |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
47 | remove_avatar_0_12_nodes(JabberStream *js) |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
48 | { |
|
27025
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
49 | #if 0 |
|
27035
2246f8a9d680
The correct fix here is to place non-code in a comment (yes, even if it's
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
27034
diff
changeset
|
50 | /* See note below for why this is #if 0'd */ |
|
27025
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
51 | |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
52 | /* Publish an empty avatar according to the XEP-0084 v0.12 semantics */ |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
53 | xmlnode *publish, *item, *metadata; |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
54 | /* publish the metadata */ |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
55 | publish = xmlnode_new("publish"); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
56 | xmlnode_set_attrib(publish, "node", NS_AVATAR_0_12_METADATA); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
57 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
58 | item = xmlnode_new_child(publish, "item"); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
59 | xmlnode_set_attrib(item, "id", "stop"); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
60 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
61 | metadata = xmlnode_new_child(item, "metadata"); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
62 | xmlnode_set_namespace(metadata, NS_AVATAR_0_12_METADATA); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
63 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
64 | xmlnode_new_child(metadata, "stop"); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
65 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
66 | /* publish */ |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
67 | jabber_pep_publish(js, publish); |
|
27025
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
68 | #endif |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
69 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
70 | /* |
|
27025
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
71 | * This causes ejabberd 2.0.0 to kill the connection unceremoniously. |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
72 | * See https://support.process-one.net/browse/EJAB-623. When adiumx.com |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
73 | * was upgraded, the issue went away. |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
74 | * |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
75 | * I think it makes a lot of sense to not have an avatar at the old |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
76 | * node instead of having something interpreted as "no avatar". When |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
77 | * a contact with an older client logs in, in the latter situation, |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
78 | * there's a race between interpreting the <presence/> vcard-temp:x:update |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
79 | * avatar (non-empty) and the XEP-0084 v0.12 avatar (empty, so show no |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
80 | * avatar for the buddy) which leads to unhappy and confused users. |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
81 | * |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
82 | * A deluge of frustrating "Read error" bug reports may change my mind |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
83 | * about this. |
|
cda390e09aaf
Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents:
27017
diff
changeset
|
84 | * --darkrain42 |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
85 | */ |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
86 | jabber_pep_delete_node(js, NS_AVATAR_0_12_METADATA); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
87 | jabber_pep_delete_node(js, NS_AVATAR_0_12_DATA); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
88 | } |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
89 | |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
90 | void jabber_avatar_set(JabberStream *js, PurpleStoredImage *img) |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
91 | { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
92 | xmlnode *publish, *metadata, *item; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
93 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
94 | if (!js->pep) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
95 | return; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
96 | |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
97 | /* Hmmm, not sure if this is worth the traffic, but meh */ |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
98 | remove_avatar_0_12_nodes(js); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
99 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
100 | if (!img) { |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
101 | publish = xmlnode_new("publish"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
102 | xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_METADATA); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
103 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
104 | item = xmlnode_new_child(publish, "item"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
105 | metadata = xmlnode_new_child(item, "metadata"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
106 | xmlnode_set_namespace(metadata, NS_AVATAR_1_1_METADATA); |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
107 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
108 | /* publish */ |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
109 | jabber_pep_publish(js, publish); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
110 | } else { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
111 | /* |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
112 | * TODO: This is pretty gross. The Jabber PRPL really shouldn't |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
113 | * do voodoo to try to determine the image type, height |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
114 | * and width. |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
115 | */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
116 | /* A PNG header, including the IHDR, but nothing else */ |
|
33766
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
117 | /* ATTN: this is in network byte order! */ |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
118 | const struct { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
119 | guchar signature[8]; /* must be hex 89 50 4E 47 0D 0A 1A 0A */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
120 | struct { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
121 | guint32 length; /* must be 0x0d */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
122 | guchar type[4]; /* must be 'I' 'H' 'D' 'R' */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
123 | guint32 width; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
124 | guint32 height; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
125 | guchar bitdepth; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
126 | guchar colortype; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
127 | guchar compression; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
128 | guchar filter; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
129 | guchar interlace; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
130 | } ihdr; |
|
33766
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
131 | } *png = NULL; |
|
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
132 | |
|
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
133 | if (purple_imgstore_get_size(img) > sizeof(*png)) |
|
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
134 | png = purple_imgstore_get_data(img); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
135 | |
|
26947
176ef2bfef9f
Use purple_strequal and g_str_equal
Paul Aurich <darkrain42@pidgin.im>
parents:
25811
diff
changeset
|
136 | /* check if the data is a valid png file (well, at least to some extent) */ |
|
33766
49eb0136f2ec
jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents:
29910
diff
changeset
|
137 | if(png && png->signature[0] == 0x89 && |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
138 | png->signature[1] == 0x50 && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
139 | png->signature[2] == 0x4e && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
140 | png->signature[3] == 0x47 && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
141 | png->signature[4] == 0x0d && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
142 | png->signature[5] == 0x0a && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
143 | png->signature[6] == 0x1a && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
144 | png->signature[7] == 0x0a && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
145 | ntohl(png->ihdr.length) == 0x0d && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
146 | png->ihdr.type[0] == 'I' && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
147 | png->ihdr.type[1] == 'H' && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
148 | png->ihdr.type[2] == 'D' && |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
149 | png->ihdr.type[3] == 'R') { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
150 | /* parse PNG header to get the size of the image (yes, this is required) */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
151 | guint32 width = ntohl(png->ihdr.width); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
152 | guint32 height = ntohl(png->ihdr.height); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
153 | xmlnode *data, *info; |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
154 | char *lengthstring, *widthstring, *heightstring; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
155 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
156 | /* compute the sha1 hash */ |
|
29910
f1c6e586e5c1
jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents:
29782
diff
changeset
|
157 | char *hash = jabber_calculate_data_hash(purple_imgstore_get_data(img), |
|
f1c6e586e5c1
jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents:
29782
diff
changeset
|
158 | purple_imgstore_get_size(img), |
|
f1c6e586e5c1
jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents:
29782
diff
changeset
|
159 | "sha1"); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
160 | char *base64avatar = purple_base64_encode(purple_imgstore_get_data(img), |
|
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
161 | purple_imgstore_get_size(img)); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
162 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
163 | publish = xmlnode_new("publish"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
164 | xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_DATA); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
165 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
166 | item = xmlnode_new_child(publish, "item"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
167 | xmlnode_set_attrib(item, "id", hash); |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
168 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
169 | data = xmlnode_new_child(item, "data"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
170 | xmlnode_set_namespace(data, NS_AVATAR_1_1_DATA); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
171 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
172 | xmlnode_insert_data(data, base64avatar, -1); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
173 | /* publish the avatar itself */ |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
174 | jabber_pep_publish(js, publish); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
175 | |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
176 | g_free(base64avatar); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
177 | |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
178 | lengthstring = g_strdup_printf("%" G_GSIZE_FORMAT, |
|
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
179 | purple_imgstore_get_size(img)); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
180 | widthstring = g_strdup_printf("%u", width); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
181 | heightstring = g_strdup_printf("%u", height); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
182 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
183 | /* publish the metadata */ |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
184 | publish = xmlnode_new("publish"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
185 | xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_METADATA); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
186 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
187 | item = xmlnode_new_child(publish, "item"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
188 | xmlnode_set_attrib(item, "id", hash); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
189 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
190 | metadata = xmlnode_new_child(item, "metadata"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
191 | xmlnode_set_namespace(metadata, NS_AVATAR_1_1_METADATA); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
192 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
193 | info = xmlnode_new_child(metadata, "info"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
194 | xmlnode_set_attrib(info, "id", hash); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
195 | xmlnode_set_attrib(info, "type", "image/png"); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
196 | xmlnode_set_attrib(info, "bytes", lengthstring); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
197 | xmlnode_set_attrib(info, "width", widthstring); |
|
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
198 | xmlnode_set_attrib(info, "height", heightstring); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
199 | |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
200 | jabber_pep_publish(js, publish); |
|
25480
6e01ab2bc943
Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents:
25479
diff
changeset
|
201 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
202 | g_free(lengthstring); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
203 | g_free(widthstring); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
204 | g_free(heightstring); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
205 | g_free(hash); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
206 | } else { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
207 | purple_debug_error("jabber", "Cannot set PEP avatar to non-PNG data\n"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
208 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
209 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
210 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
211 | |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
212 | static void |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
213 | do_got_own_avatar_0_12_cb(JabberStream *js, const char *from, xmlnode *items) |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
214 | { |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
215 | if (items) |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
216 | /* It wasn't an error (i.e. 'item-not-found') */ |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
217 | remove_avatar_0_12_nodes(js); |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
218 | } |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
219 | |
|
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
220 | static void |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
221 | do_got_own_avatar_cb(JabberStream *js, const char *from, xmlnode *items) |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
222 | { |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
223 | xmlnode *item = NULL, *metadata = NULL, *info = NULL; |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
224 | PurpleAccount *account = purple_connection_get_account(js->gc); |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
225 | const char *server_hash = NULL; |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
226 | |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
227 | if (items && (item = xmlnode_get_child(items, "item")) && |
|
28048
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
228 | (metadata = xmlnode_get_child(item, "metadata")) && |
|
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
229 | (info = xmlnode_get_child(metadata, "info"))) { |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
230 | server_hash = xmlnode_get_attrib(info, "id"); |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
231 | } |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
232 | |
|
28048
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
233 | /* |
|
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
234 | * If we have an avatar and the server returned an error/malformed data, |
|
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
235 | * push our avatar. If the server avatar doesn't match the local one, push |
|
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
236 | * our avatar. |
|
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
237 | */ |
|
28881
244fb329e5df
jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents:
28828
diff
changeset
|
238 | if ((!items || !metadata) || |
|
28048
cdc92f9d634a
jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents:
27358
diff
changeset
|
239 | !purple_strequal(server_hash, js->initial_avatar_hash)) { |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
240 | PurpleStoredImage *img = purple_buddy_icons_find_account_icon(account); |
|
26950
842628304397
Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents:
26949
diff
changeset
|
241 | jabber_avatar_set(js, img); |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
242 | purple_imgstore_unref(img); |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
243 | } |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
244 | } |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
245 | |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
246 | void jabber_avatar_fetch_mine(JabberStream *js) |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
247 | { |
|
28881
244fb329e5df
jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents:
28828
diff
changeset
|
248 | if (js->initial_avatar_hash) { |
|
29782
ca4fedd9e890
jabber: Allow jabber_pep_request_item() to take NULL for JID; server's routing takes care of the rest
Paul Aurich <darkrain42@pidgin.im>
parents:
28881
diff
changeset
|
249 | jabber_pep_request_item(js, NULL, NS_AVATAR_0_12_METADATA, NULL, |
|
28881
244fb329e5df
jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents:
28828
diff
changeset
|
250 | do_got_own_avatar_0_12_cb); |
|
29782
ca4fedd9e890
jabber: Allow jabber_pep_request_item() to take NULL for JID; server's routing takes care of the rest
Paul Aurich <darkrain42@pidgin.im>
parents:
28881
diff
changeset
|
251 | jabber_pep_request_item(js, NULL, NS_AVATAR_1_1_METADATA, NULL, |
|
28881
244fb329e5df
jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents:
28828
diff
changeset
|
252 | do_got_own_avatar_cb); |
|
244fb329e5df
jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents:
28828
diff
changeset
|
253 | } |
|
25477
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
254 | } |
|
ada9e5c4a10a
Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents:
25476
diff
changeset
|
255 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
256 | typedef struct _JabberBuddyAvatarUpdateURLInfo { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
257 | JabberStream *js; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
258 | char *from; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
259 | char *id; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
260 | } JabberBuddyAvatarUpdateURLInfo; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
261 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
262 | static void |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
263 | do_buddy_avatar_update_fromurl(PurpleHttpConnection *http_conn, |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
264 | PurpleHttpResponse *response, gpointer _info) |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
265 | { |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
266 | JabberBuddyAvatarUpdateURLInfo *info = _info; |
|
27358
a1e3486558a0
Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents:
27035
diff
changeset
|
267 | gpointer icon_data; |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
268 | const gchar *got_data; |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
269 | size_t got_len; |
|
27358
a1e3486558a0
Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents:
27035
diff
changeset
|
270 | |
|
34287
6cd0c77b1f6a
HTTP: successful is spelled with one l
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
34269
diff
changeset
|
271 | if (!purple_http_response_is_successful(response)) { |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
272 | purple_debug_error("jabber", "do_buddy_avatar_update_fromurl " |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
273 | "got error \"%s\"", |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
274 | purple_http_response_get_error(response)); |
|
25476
d31fcddd65a4
Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents:
25475
diff
changeset
|
275 | goto out; |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
276 | } |
| 25481 | 277 | |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
278 | got_data = purple_http_response_get_data(response, &got_len); |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
279 | icon_data = g_memdup(got_data, got_len); |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
280 | purple_buddy_icons_set_for_user(purple_connection_get_account(info->js->gc), info->from, icon_data, got_len, info->id); |
|
25476
d31fcddd65a4
Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents:
25475
diff
changeset
|
281 | |
|
d31fcddd65a4
Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents:
25475
diff
changeset
|
282 | out: |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
283 | g_free(info->from); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
284 | g_free(info->id); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
285 | g_free(info); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
286 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
287 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
288 | static void |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
289 | do_buddy_avatar_update_data(JabberStream *js, const char *from, xmlnode *items) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
290 | { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
291 | xmlnode *item, *data; |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
292 | const char *checksum; |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
293 | char *b64data; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
294 | void *img; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
295 | size_t size; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
296 | if(!items) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
297 | return; |
| 25481 | 298 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
299 | item = xmlnode_get_child(items, "item"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
300 | if(!item) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
301 | return; |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
302 | |
|
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
303 | data = xmlnode_get_child(item, "data"); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
304 | if(!data) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
305 | return; |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
306 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
307 | checksum = xmlnode_get_attrib(item,"id"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
308 | if(!checksum) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
309 | return; |
| 25481 | 310 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
311 | b64data = xmlnode_get_data(data); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
312 | if(!b64data) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
313 | return; |
| 25481 | 314 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
315 | img = purple_base64_decode(b64data, &size); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
316 | if(!img) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
317 | g_free(b64data); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
318 | return; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
319 | } |
| 25481 | 320 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
321 | purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, img, size, checksum); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
322 | g_free(b64data); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
323 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
324 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
325 | static void |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
326 | update_buddy_metadata(JabberStream *js, const char *from, xmlnode *items) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
327 | { |
|
34728
8efd73063ecf
Renamed buddy list functions to more appropriate/simler names.
Ankit Vani <a@nevitus.org>
parents:
34227
diff
changeset
|
328 | PurpleBuddy *buddy = purple_blist_find_buddy(purple_connection_get_account(js->gc), from); |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
329 | const char *checksum; |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
330 | xmlnode *item, *metadata; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
331 | if(!buddy) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
332 | return; |
| 25481 | 333 | |
|
25809
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
334 | if (!items) |
|
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
335 | return; |
|
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
336 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
337 | item = xmlnode_get_child(items,"item"); |
|
25809
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
338 | if (!item) |
|
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
339 | return; |
|
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
340 | |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
341 | metadata = xmlnode_get_child(item, "metadata"); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
342 | if(!metadata) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
343 | return; |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
344 | |
|
25809
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
345 | checksum = purple_buddy_icons_get_checksum_for_user(buddy); |
|
f0d37a73663a
I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents:
25543
diff
changeset
|
346 | |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
347 | /* <stop/> was the pre-v1.1 method of publishing an empty avatar */ |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
348 | if(xmlnode_get_child(metadata, "stop")) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
349 | purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, NULL, 0, NULL); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
350 | } else { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
351 | xmlnode *info, *goodinfo = NULL; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
352 | gboolean has_children = FALSE; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
353 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
354 | /* iterate over all info nodes to get one we can use */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
355 | for(info = metadata->child; info; info = info->next) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
356 | if(info->type == XMLNODE_TYPE_TAG) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
357 | has_children = TRUE; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
358 | if(info->type == XMLNODE_TYPE_TAG && !strcmp(info->name,"info")) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
359 | const char *type = xmlnode_get_attrib(info,"type"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
360 | const char *id = xmlnode_get_attrib(info,"id"); |
| 25481 | 361 | |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
362 | if(checksum && id && !strcmp(id, checksum)) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
363 | /* we already have that avatar, so we don't have to do anything */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
364 | goodinfo = NULL; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
365 | break; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
366 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
367 | /* We'll only pick the png one for now. It's a very nice image format anyways. */ |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
368 | if(type && id && !goodinfo && !strcmp(type, "image/png")) |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
369 | goodinfo = info; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
370 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
371 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
372 | if(has_children == FALSE) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
373 | purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, NULL, 0, NULL); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
374 | } else if(goodinfo) { |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
375 | const char *url = xmlnode_get_attrib(goodinfo, "url"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
376 | const char *id = xmlnode_get_attrib(goodinfo,"id"); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
377 | |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
378 | /* the avatar might either be stored in a pep node, or on a HTTP(S) URL */ |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
379 | if(!url) { |
|
27017
02ddb8e8c3d2
Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents:
26959
diff
changeset
|
380 | jabber_pep_request_item(js, from, NS_AVATAR_1_1_DATA, id, |
|
25479
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
381 | do_buddy_avatar_update_data); |
|
416673fd83b4
Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents:
25477
diff
changeset
|
382 | } else { |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
383 | PurpleHttpRequest *req; |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
384 | JabberBuddyAvatarUpdateURLInfo *info = g_new0(JabberBuddyAvatarUpdateURLInfo, 1); |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
385 | info->js = js; |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
386 | |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
387 | req = purple_http_request_new(url); |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
388 | purple_http_request_set_max_len(req, MAX_HTTP_BUDDYICON_BYTES); |
|
34269
26b4c9a54166
HTTP: Use PurpleHttpConnectionSet instead of GSLists of PurpleHttpConnection structs
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
34227
diff
changeset
|
389 | purple_http_connection_set_add(js->http_conns, |
|
26b4c9a54166
HTTP: Use PurpleHttpConnectionSet instead of GSLists of PurpleHttpConnection structs
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
34227
diff
changeset
|
390 | purple_http_request(js->gc, req, |
|
26b4c9a54166
HTTP: Use PurpleHttpConnectionSet instead of GSLists of PurpleHttpConnection structs
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
34227
diff
changeset
|
391 | do_buddy_avatar_update_fromurl, info)); |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
392 | purple_http_request_unref(req); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
393 | |
|
34227
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
394 | info->from = g_strdup(from); |
|
6d016a081401
HTTP: migrate purple_util_fetch_url to new API for prpls
Tomasz Wasilczyk <tomkiewicz@cpw.pidgin.im>
parents:
33774
diff
changeset
|
395 | info->id = g_strdup(id); |
|
25475
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
396 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
397 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
398 | } |
|
ad7612a5d678
Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff
changeset
|
399 | } |