Bug 26847 - Invalid parameters can confuse fargo, requiring a fargo restart.
Summary: Invalid parameters can confuse fargo, requiring a fargo restart.
Status: NEW
Alias: None
Product: Telepathy
Classification: Unclassified
Component: fargo (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: David Laban
QA Contact: Simon McVittie
URL: http://git.collabora.co.uk/?p=user/al...
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2010-03-02 13:01 UTC by David Laban
Modified: 2010-03-22 05:43 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description David Laban 2010-03-02 13:01:41 UTC
Sometimes, registering with invalid parameters can cause fargo to get confused. This is because the signin code is more complicated than it should be, and contains bugs.

my register-and-signin branch does a bit of cleaning up and documenting how everything works.

Eventually, making Connection more like Call would simplify the signin code path quite a lot, but that's far too much refactoring for me right now.

There is a slight limitation in how well we can do error handling because sofiasip doesn't emit ConnectionError, so if sofiasip emits StatusChanged(Disconnected, None_Specified) we will always say "No reason specified. Please check your legacy parameters."

Adding some automated tests might be a good idea before we ship this, as all testing so far has been by hand.
Comment 1 David Laban 2010-03-22 05:43:06 UTC
Rebased and cleaned up http://git.collabora.co.uk/?p=user/alsuren/telepathy-fargo.git;a=shortlog;h=refs/heads/register-and-signin to make it easier to review.


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.