/soc/2015/mmcc/main: bc63cdee394b: remove default file transfer ...

Changeset: bc63cdee394bc489ba8d5a395dfe8d83af4f9f21
Author:	 Michael McConville <[email protected]>
Date:	 2015-07-13 12:57 -0400
Branch:	 default
URL: https://hg.pidgin.im/soc/2015/mmcc/main/rev/bc63cdee394b

Description:

remove default file transfer proxy (proxy.eu.jabber.org) as per #16535 and etan

diffstat:

 libpurple/protocols/jabber/gtalk.c  |  5 +----
 libpurple/protocols/jabber/jabber.c |  3 +--
 libpurple/protocols/jabber/jabber.h |  1 -
 libpurple/protocols/jabber/xmpp.c   |  5 +----
 4 files changed, 3 insertions(+), 11 deletions(-)

diffs (55 lines):

diff --git a/libpurple/protocols/jabber/gtalk.c b/libpurple/protocols/jabber/gtalk.c
--- a/libpurple/protocols/jabber/gtalk.c
+++ b/libpurple/protocols/jabber/gtalk.c
@@ -89,10 +89,7 @@ gtalk_protocol_init(PurpleProtocol *prot
 						  option);
 
 	option = purple_account_option_string_new(_("File transfer proxies"),
-						  "ft_proxies",
-						/* TODO: Is this an acceptable default?
-						 * Also, keep this in sync as they add more servers */
-						  JABBER_DEFAULT_FT_PROXIES);
+						  "ft_proxies", NULL);
 	protocol->account_options = g_list_append(protocol->account_options,
 						  option);
 
diff --git a/libpurple/protocols/jabber/jabber.c b/libpurple/protocols/jabber/jabber.c
--- a/libpurple/protocols/jabber/jabber.c
+++ b/libpurple/protocols/jabber/jabber.c
@@ -1096,9 +1096,8 @@ jabber_login(PurpleAccount *account)
 	if (js == NULL)
 		return;
 
-	/* TODO: Remove this at some point.  Added 2010-02-14 (v2.6.6) */
 	if (g_str_equal("proxy.jabber.org", purple_account_get_string(account, "ft_proxies", "")))
-		purple_account_set_string(account, "ft_proxies", JABBER_DEFAULT_FT_PROXIES);
+		purple_account_set_string(account, "ft_proxies", NULL);
 
 	/*
 	 * Calculate the avatar hash for our current image so we know (when we
diff --git a/libpurple/protocols/jabber/jabber.h b/libpurple/protocols/jabber/jabber.h
--- a/libpurple/protocols/jabber/jabber.h
+++ b/libpurple/protocols/jabber/jabber.h
@@ -92,7 +92,6 @@ typedef struct _JabberStream JabberStrea
 #define JABBER_PROTOCOL_GET_CLASS(obj)   (G_TYPE_INSTANCE_GET_CLASS((obj), JABBER_TYPE_PROTOCOL, JabberProtocolClass))
 
 #define JABBER_DEFAULT_REQUIRE_TLS    "require_starttls"
-#define JABBER_DEFAULT_FT_PROXIES     "proxy.eu.jabber.org"
 
 /* Index into attention_types list */
 #define JABBER_BUZZ 0
diff --git a/libpurple/protocols/jabber/xmpp.c b/libpurple/protocols/jabber/xmpp.c
--- a/libpurple/protocols/jabber/xmpp.c
+++ b/libpurple/protocols/jabber/xmpp.c
@@ -80,10 +80,7 @@ xmpp_protocol_init(PurpleProtocol *proto
 						  option);
 
 	option = purple_account_option_string_new(_("File transfer proxies"),
-						  "ft_proxies",
-						/* TODO: Is this an acceptable default?
-						 * Also, keep this in sync as they add more servers */
-						  JABBER_DEFAULT_FT_PROXIES);
+						  "ft_proxies", NULL);
 	protocol->account_options = g_list_append(protocol->account_options,
 						  option);
 

_______________________________________________
Commits mailing list
[email protected]
https://pidgin.im/cgi-bin/mailman/listinfo/commits