After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 333821 - randomly(?) attempts to ping server when offline
randomly(?) attempts to ping server when offline
Status: VERIFIED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.6.x (obsolete)
Other All
: High blocker
: ---
Assigned To: Veerapuram Varadhan
Evolution QA team
: 334346 334932 340677 342117 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-03-07 23:40 UTC by Luis Villa
Modified: 2013-09-10 14:04 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
Shows weird/stupid error message. (150.46 KB, image/png)
2006-03-07 23:40 UTC, Luis Villa
  Details
Patch to fix the issue (1.17 KB, patch)
2006-03-21 10:39 UTC, Shreyas Srinivasan
none Details | Review
patch (1.19 KB, patch)
2006-07-13 11:10 UTC, Johnny Jacob
none Details | Review
Fix. (850 bytes, patch)
2006-07-26 06:58 UTC, Veerapuram Varadhan
committed Details | Review

Description Luis Villa 2006-03-07 23:40:08 UTC
Please describe the problem:
Attached picture pretty much says it all; it attempts to ping the IMAP server
while I'm offline for some reason. Of *course* I have to be online, so *don't
ping the server*. Don't try to ping it, and then give me an error message
randomly...

Steps to reproduce:



Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Luis Villa 2006-03-07 23:40:54 UTC
Created attachment 60877 [details]
Shows weird/stupid error message.
Comment 2 André Klapper 2006-03-09 16:59:11 UTC
at least a second person which can also see this behaviour.
Comment 3 Steve Fox 2006-03-20 14:24:54 UTC
Seeing this on 2.6.0 final as shipped by FC5.
Comment 4 André Klapper 2006-03-20 14:53:38 UTC
*** Bug 334346 has been marked as a duplicate of this bug. ***
Comment 5 André Klapper 2006-03-20 14:58:26 UTC
we will get many, many bug reports dealing with this as it is very visible and intrusive.
perhaps a duplicate of bug 329712.

shres?
Comment 6 André Klapper 2006-03-20 20:09:29 UTC
*** Bug 334932 has been marked as a duplicate of this bug. ***
Comment 7 Shreyas Srinivasan 2006-03-21 09:47:21 UTC
bug 329712: Is just a case of network being up and evolution --offline being used. 

This bug is much more serious and not a duplicate. 
Comment 8 Shreyas Srinivasan 2006-03-21 10:37:19 UTC
Groan, suckage 

  • #0 camel_imap_store_connected
    at camel-imap-store.c line 2956
  • #1 imap_noop
    at camel-imap-store.c line 1649
  • #2 camel_store_noop
    at camel-store.c line 1183
  • #3 ping_store_ping
    at mail-folder-cache.c line 870
  • #4 mail_msg_received
    at mail-mt.c line 570
  • #5 thread_received_msg
    at e-msgport.c line 974
  • #6 thread_dispatch
    at e-msgport.c line 1055
  • #7 _pthread_body

which means imap_noop needs to call camel_imap_store_connected to know the network state. Which sets an exception which the ui picks up :(
Comment 9 Shreyas Srinivasan 2006-03-21 10:39:32 UTC
Created attachment 61679 [details] [review]
Patch to fix the issue
Comment 10 André Klapper 2006-04-05 04:23:10 UTC
boy, this is really annoying. any news on this?
Comment 11 Sebastien Bacher 2006-04-05 14:12:12 UTC
Ubuntu bug about that: https://launchpad.net/distros/ubuntu/+source/evolution/+bug/38154

"Since I use Dapper I have a strange and senseless error, while I am working offline:

"Error while Pinging IMAP server imap4.vr.in-berlin.de.

You must be working online to complete this operation"

This error comes 5-10minutes after I go offline and then every X minutes (maybe 10 or 15minutes), after I click on ok.

I am working offline in evolution, because I don't have a network connection and I didn't want that evolution try to connect a server. Why evolution tries to "ping" an IMAP server, when I am offline. This seems to be a bug."


Any reason to not use the patch from that bug before GNOME 2.14.1?
Comment 12 André Klapper 2006-04-07 19:13:44 UTC
shres, today i applied your patch to yesterday's checkout of cvs head and it does not fix the issue for me. i still get the same popups.
suse9.3/gnome2.10
Comment 13 André Klapper 2006-05-09 01:30:30 UTC
*** Bug 340677 has been marked as a duplicate of this bug. ***
Comment 14 Srinivasa Ragavan 2006-05-10 03:46:21 UTC
Sankar: Can you look into this?
Comment 15 Sebastien Bacher 2006-05-16 16:33:58 UTC
Any news on this patch?
Comment 16 André Klapper 2006-05-17 16:10:50 UTC
*** Bug 342117 has been marked as a duplicate of this bug. ***
Comment 17 Srinivasa Ragavan 2006-07-12 04:33:07 UTC
It is really annoying. bumping the priority.
Comment 18 Johnny Jacob 2006-07-13 11:10:04 UTC
Created attachment 68855 [details] [review]
patch
Comment 19 Sankar P 2006-07-13 11:29:46 UTC
How did you arrive at the conclusion that noop is the function that tries to make the n/w call? 

Anyway, on imap_noop I see:

	CAMEL_SERVICE_LOCK (imap_store, connect_lock);

	if (!camel_imap_store_connected(imap_store, ex))
		goto done;

	<code>

	done: 
	CAMEL_SERVICE_LOCK (imap_store, connect_lock);


So, I dont think noop will try to ping the server if the imap_store is not connected. Am I missing something ? 
Comment 20 Sankar P 2006-07-13 11:33:34 UTC
The statement below the "done" label is:

	CAMEL_SERVICE_UNLOCK (imap_store, connect_lock);

Sorry for the Copy & Paste error...
Comment 21 Johnny Jacob 2006-07-13 13:28:53 UTC
The API doc says :
/**
 * camel_store_noop:
 * @store: a #CamelStore object
 * @ex: a #CamelException
 *
 * Pings @store so that its connection doesn't timeout.
 **/

camel_imap_store_connected calls camel_disco_store_check_online which sets the exception. (so therez no actual pinging. just the error) 


Comment 22 Johnny Jacob 2006-07-14 11:41:12 UTC
Varadhan has a better fix for this issue.
Adding Varadhan to cc list.
Comment 23 André Klapper 2006-07-18 20:52:35 UTC
varadhan: *poke*. this must get into 2.6.3!
Comment 24 Poornima 2006-07-24 05:23:01 UTC
bnc bug https://bugzilla.novell.com/show_bug.cgi?id=194416
Comment 25 Veerapuram Varadhan 2006-07-26 06:45:25 UTC
Camel library shouldn't do any such checks.  The application should be doing the checks.  Attaching an appropriate fix.
Comment 26 Veerapuram Varadhan 2006-07-26 06:58:14 UTC
Created attachment 69635 [details] [review]
Fix.

Patch applies to CVS HEAD.
Comment 27 Srinivasa Ragavan 2006-07-26 09:57:40 UTC
Atlast we have a fix for this. Awesome Varadhan :)
Comment 28 Veerapuram Varadhan 2006-07-26 13:21:40 UTC
Thanks, Srini. (don't know whether this is a sarcasm. ;-), anyway, taking in usual way)

Patch committed both to 2.6.3 and HEAD.

@Andre: Special thanks for poking me. ;-)
Comment 29 Srinivasa Ragavan 2006-07-26 16:28:54 UTC
No Sarcasm really :) This is one bug, I saw in every one's laptop in GUADEC and I hate to see it again. Thx dude.
Comment 30 Poornima 2006-08-03 10:00:54 UTC
Verified in 2.6.3.