9:26 p.m.
http://bugzilla.moblin.org/show_bug.cgi?id=8879
Summary: missing bluetooth adapter not noticed when syncing
Classification: Moblin Projects
Product: SyncEvolution
Version: upstream
Platform: Netbook
OS/Version: Moblin Linux
Status: NEW
Severity: minor
Priority: Undecided
Component: SyncEvolution
AssignedTo: patrick.ohly(a)intel.com
ReportedBy: jku(a)linux.intel.com
QAContact: yanshuang.zheng(a)intel.com
CC: syncevolution(a)lists.intel.com
My laptop Bluetooth was somehow disabled when I tried to sync. The result was
500 with
[ERROR] TransportException while sending SAN package
[ERROR] Server Alerted Sync init failed
There should be a check for an existing local bluetooth adapter and a
corresponding error.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
5:13 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
Chen Congwu <congwu.chen(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |congwu.chen(a)intel.com
AssignedTo|patrick.ohly(a)intel.com |congwu.chen(a)intel.com
--- Comment #1 from Chen Congwu <congwu.chen(a)intel.com> 2010-01-04 22:13:22 PST
---
While looking at this, it also applies to HTTP transport (failed as "Cannot
resolve hostname" and retries several times before finally abort).
I propose add TransportAgent::isOnLine() to check whether the underlying
adapter is available or connected, and check this before actually starting
sync.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
7:38 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |patrick.ohly(a)intel.com
--- Comment #2 from pohly <patrick.ohly(a)intel.com> 2010-01-05 00:38:14 PST ---
(In reply to comment #1)
While looking at this, it also applies to HTTP transport (failed as
"Cannot
resolve hostname" and retries several times before finally abort).
I propose add TransportAgent::isOnLine() to check whether the underlying
adapter is available or connected, and check this before actually starting
sync.
Who would call this and how would "not online" be reported back to the sync-ui?
Can't this be done entirely inside the TransportAgent, by returning a
"transport offline" error? Then we don't have to introduce a new API.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
9:03 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
--- Comment #3 from Chen Congwu <congwu.chen(a)intel.com> 2010-01-05 02:03:24 PST
---
(In reply to comment #2)
(In reply to comment #1)
> While looking at this, it also applies to HTTP transport (failed as "Cannot
> resolve hostname" and retries several times before finally abort).
> I propose add TransportAgent::isOnLine() to check whether the underlying
> adapter is available or connected, and check this before actually starting
> sync.
Who would call this and how would "not online" be reported back to the
sync-ui?
Can't this be done entirely inside the TransportAgent, by returning a
"transport offline" error? Then we don't have to introduce a new API.
When do you check transport online? During the *first* send() in a session?
Thus transport API still need to be changed to know whether this is the *first*
send.
From this point, adding a new API is a bit clear. The added API maybe
also
extended further to cover presence detection.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
11:30 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
--- Comment #4 from pohly <patrick.ohly(a)intel.com> 2010-01-05 04:30:46 PST ---
(In reply to comment #3)
(In reply to comment #2)
> (In reply to comment #1)
> > While looking at this, it also applies to HTTP transport (failed as
"Cannot
> > resolve hostname" and retries several times before finally abort).
> > I propose add TransportAgent::isOnLine() to check whether the underlying
> > adapter is available or connected, and check this before actually starting
> > sync.
>
> Who would call this and how would "not online" be reported back to the
sync-ui?
>
> Can't this be done entirely inside the TransportAgent, by returning a
> "transport offline" error? Then we don't have to introduce a new API.
When do you check transport online? During the *first* send() in a session?
Thus transport API still need to be changed to know whether this is the *first*
send.
This is already implied by the way how transport agents are created: they are
created anew for each session. Therefore the first message send after creation
can be considered as this "special" first send without resending.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
4:07 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
--- Comment #5 from Chen Congwu <congwu.chen(a)intel.com> 2010-02-21 21:07:20 PST
---
This is not applicable from the UI (with presence detection). Whether we should
implement this in the command line is debatable: There is no distribution
independent way to query status of local BT adapater or HTTP Adapter without
depending on dbus service (connman or network manager).
I am inclined to leave this as it is and the command line will benefit from
presence detection once its implementation changed to use syncevo-dbus-server.
Thoughts?
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:38 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
--- Comment #6 from pohly <patrick.ohly(a)intel.com> 2010-02-21 23:38:01 PST ---
(In reply to comment #5)
This is not applicable from the UI (with presence detection).
You mean the UI will never run into this situation because it will only start
syncing when Bluetooth is enabled? I disagree. Some GUIs might not use presence
detection, there are race conditions, without ConnMan it doesn't work.
Whether we should
implement this in the command line is debatable: There is no distribution
independent way to query status of local BT adapater or HTTP Adapter without
depending on dbus service (connman or network manager).
I am inclined to leave this as it is and the command line will benefit from
presence detection once its implementation changed to use syncevo-dbus-server.
Thoughts?
I agree with this. But there's still one aspect of this issue which has to be
solved. When Bluetooth is unavailable, the error reporting is insufficient.
I get:
[ERROR] TransportException while sending SAN package
[ERROR] Server Alerted Sync init failed
fatal error (local, status 10500)
The TransportException should be reported back up to the top level. Instead it
seems that initSAN() catches it and then just reports back that it didn't
succeed. Why do you catch the exception instead of letting it propagate?
Because of that, we end up with two ERROR lines for the same thing and an
unspecific 500 status.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
6:08 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
--- Comment #7 from Chen Congwu <congwu.chen(a)intel.com> 2010-02-23 23:08:36 PST
---
(In reply to comment #6)
I get:
[ERROR] TransportException while sending SAN package
[ERROR] Server Alerted Sync init failed
fatal error (local, status 10500)
The TransportException should be reported back up to the top level. Instead it
seems that initSAN() catches it and then just reports back that it didn't
succeed. Why do you catch the exception instead of letting it propagate?
I agree, just pushed to mb8879, here is the output after this patch:
[ERROR] ObexTransport Bluetooth sdp connect failed
Synchronization failed, see
/home/congwu/.cache/syncevolution/bt-2010-02-24-14-58/syncevolution-log.html
for details.
Changes applied during synchronization:
+---------------|-----------------------|-----------------------|-CON-+
| | LOCAL | REMOTE | FLI |
| Source | NEW | MOD | DEL | ERR | NEW | MOD | DEL | ERR | CTS |
+---------------+-----+-----+-----+-----+-----+-----+-----+-----+-----+
| addressbook | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 |
+---------------+-----+-----+-----+-----+-----+-----+-----+-----+-----+
| start Wed Feb 24 14:58:04 2010, duration 0:00min |
| external transport failure (local, status 20043) |
+---------------+-----+-----+-----+-----+-----+-----+-----+-----+-----+
First ERROR encountered: ObexTransport Bluetooth sdp connect failed
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
7:52 p.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
--- Comment #8 from pohly <patrick.ohly(a)intel.com> 2010-02-24 00:52:22 PST ---
Merged.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
3:28 a.m.
New subject: [Bug 8879] missing bluetooth adapter not noticed when syncing
http://bugzilla.moblin.org/show_bug.cgi?id=8879
pohly <patrick.ohly(a)intel.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|FIXED |INTEGRATED
--- Comment #9 from pohly <patrick.ohly(a)intel.com> 2010-02-24 08:28:08 PST ---
Integrated in 1.0 beta 2 = 0.99.11.
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
4589
days inactive
4646
days old
syncevolution-issues@syncevolution.org
9 comments
1 participants
participants (1)
-
bugzilla＠moblin.org