Closed Bug 955364 Opened 10 years ago Closed 10 years ago

Add hidden option to allow using Facebook beta server

Categories

(Chat Core :: XMPP, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: clokep, Assigned: clokep)

Details

Attachments

(1 file, 1 obsolete file)

*** Original post on bio 1927 at 2013-04-13 15:43:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***
Attached patch (Untested) patch v1 (obsolete) — Splinter Review
*** Original post on bio 1927 as attmnt 2354 at 2013-04-13 15:43:00 UTC ***

I've wanted to start using the beta Facebook chat server to ensure we don't break with any changes coming down the line from them. Currently I'd have to create an extension or hack my Instantbird install to do this, it seems fairly trivial to add a hidden (about:config) option to use their beta server.

My patch is currently untested, I'll request review once I test it.
*** Original post on bio 1927 at 2013-04-13 15:44:42 UTC ***

Assigning.
Assignee: nobody → clokep
Status: NEW → ASSIGNED
*** Original post on bio 1927 at 2013-04-13 21:27:35 UTC ***

Would it be better if the pref was a char pref containing the hostname to connect to? (In case the name of the beta server changes, or whatever...)

Just my 2 cents, I don't mind either way.
Attached patch Patch v2Splinter Review
*** Original post on bio 1927 as attmnt 2365 at 2013-04-16 01:31:00 UTC ***

Actually, we already have a "server" pref for normal XMPP, it probably makes sense to use the same thing.
Attachment #8354132 - Flags: review?(florian)
Comment on attachment 8354121 [details] [diff] [review]
(Untested) patch v1

*** Original change on bio 1927 attmnt 2354 at 2013-04-16 01:31:40 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8354121 - Attachment is obsolete: true
Comment on attachment 8354132 [details] [diff] [review]
Patch v2

*** Original change on bio 1927 attmnt 2365 at 2013-04-16 10:41:31 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8354132 - Flags: review?(florian) → review+
*** Original post on bio 1927 at 2013-04-16 23:25:33 UTC ***

Checked in as http://hg.instantbird.org/instantbird/rev/1c6b7e0316e1
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.4
You need to log in before you can comment on or make changes to this bug.