Bug 22508 - missing feature: HTTP connection
Summary: missing feature: HTTP connection
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: butterfly (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/jo...
Whiteboard: review+
Keywords: patch
: 20452 (view as bug list)
Depends on:
Blocks: 27121
  Show dependency treegraph
 
Reported: 2009-06-27 01:31 UTC by Nicolò Chieffo
Modified: 2010-04-08 06:12 UTC (History)
7 users (show)

See Also:
i915 platform:
i915 features:


Attachments
patch (2.33 KB, patch)
2009-07-23 08:37 UTC, Laurent Bigonville
Details | Splinter Review
fallback to http automatically (1.95 KB, patch)
2009-11-05 18:49 UTC, Matt Trudel
Details | Splinter Review

Description Nicolò Chieffo 2009-06-27 01:31:57 UTC
FROM A LAUNCHPAD BUG

There seems to be no way to connect to MSN through another method than using port 1863.

In some environments, ports other than, for example, 80 and 443 are not allowed to egress. Users then commonly try to use gateway.messenger.msn.com over port 80 (or is it 443)?

This is behavior that is already implemented in Pidgin, as well as the official MSN clients.
Comment 1 Laurent Bigonville 2009-07-23 08:37:37 UTC
Created attachment 27953 [details] [review]
patch

This is a quick patch, not even tested
Comment 2 Louis-Francis Ratté-Boulianne 2009-07-24 11:09:31 UTC
The ideal scenario would be that we don't have to expose that property to the user. tp-butterfly should auto-magically use the gateway if necessary.
Comment 3 Nicolò Chieffo 2009-09-03 01:51:35 UTC
Did someone test this patch on a HTTP only connection?
My connection lets me use the MSN port so I can't
Comment 4 Tim O'Hare 2009-10-18 16:57:20 UTC
I attempted the patch but I realized I have additional requirements.
It needs to go through a HTTP proxy with authentication on the HTTP proxy.
I have the Gnome system proxy set up but I could not find where to set it up in Empathy.
Comment 5 Matt Trudel 2009-11-05 18:49:50 UTC
Created attachment 31006 [details] [review]
fallback to http automatically

Inspired by Laurent's patch and Louis-Francis' indications...

Here's a patch that tries a connection to the server and port specified in the user's settings, and if it doesn't connect within 10 seconds, falls back to an HTTP connection.

/ Matt
Comment 6 Olivier Le Thanh Duong 2009-12-19 10:19:14 UTC
Hi Matt,

Sorry for the wait, I took a look at your patch and it looks good but we would rather not have a separate test connection but use the http method if the normal one fail, do you think you can make that adjustement?

Also where the code test if proxy is None, in the else you should set use_http_method to True since it make sense if you have a http proxy to use the HTTP method.

Regards,
Olivier
Comment 7 Matt Trudel 2009-12-20 11:20:20 UTC
Olivier, sorry I didn't respond any sooner I had only seen the comment on Launchpad. I certainly can make the change. I had opted to use a separate test because it does cause one to wait substancially less for the standard connection to fail (since you can wait a long time if a firewall just drops packets rather than sending an ICMP reply). I however know that the test itself was kind of poorly done, with an arbitrary timeout...

Thanks for the response about using the HTTP transport by default for proxy connections, it does make sense. That said, I'd then also remove "proxies" from the standard connection since it would always be empty in that case.

/ Matt
Comment 8 Olivier Le Thanh Duong 2010-02-18 04:07:49 UTC
*** Bug 20452 has been marked as a duplicate of this bug. ***
Comment 9 Jonny Lamb 2010-03-14 05:58:35 UTC
It would be nice to not need a property to use HTTP and just automatically use it when we can't use the 1863, but the direct connection takes too long to time out, so perhaps we should also have a "use-http-connection" property to force the behaiviour?
Comment 10 Jonny Lamb 2010-03-14 06:42:19 UTC
I implemented this:

    http://git.collabora.co.uk/?p=user/jonny/telepathy-butterfly.git;a=shortlog;h=refs/heads/http

I added a "use-http" parameter (using that name to be consistent with haze's MSN parameter) to force using an HTTP connection so if users know 1863 won't work, they can avoid the long timeout delay.

I played with my router (no doubt annoying my housemates) and it all appears to work with appropriate ports blocked.

Another one for you to review, Olivier!
Comment 11 Olivier Le Thanh Duong 2010-03-30 08:04:49 UTC
can't test is here but the code is ok


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.