Bug 313061 - evolution fails to represent URLs with ampersands correctly
Summary: evolution fails to represent URLs with ampersands correctly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: gtkhtml3
Version: rawhide
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Matthew Barnes
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-30 13:38 UTC by Thomas J. Baker
Modified: 2007-12-11 21:03 UTC (History)
1 user (show)

Fixed In Version: gtkhtml3-3.16.2-1.fc8
Clone Of:
Environment:
Last Closed: 2007-12-11 21:03:25 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
ebay email (7.35 KB, application/octet-stream)
2007-10-01 15:20 UTC, Thomas J. Baker
no flags Details


Links
System ID Private Priority Status Summary Last Updated
GNOME Bugzilla 485700 0 None None None Never

Description Thomas J. Baker 2007-09-30 13:38:26 UTC
An EBay email containing a link to an auction is misrepresented in evolution.
Mousing over the link displays the link in the status area with the ampersands
represented as "&". Clicking the link passes it to firefox with the same
error and copying the link also contains the error. 

I filed this against gtkhtml3 but it might be further up in evolution where the
error actually begins.

Here is an example mangled URL:

http://cgi.ebay.com/ws/eBayISAPI.dll?ViewItem&item=220155499798&ssPageName=ADME:B:SS:US:1

Comment 1 Matthew Barnes 2007-10-01 14:18:12 UTC
Evolution 2.12 / GtkHtml 3.16 seems to handle a random Amazon.com URL with
ampersands correctly for me.  I tried sending it to myself in both text/plain
and text/html, thinking the MIME type might be the key.  I couldn't reproduce
the problem in either case.

So that I can investigate further, can you please:

1) Indicate which version of Evolution and GtkHtml you were using.

2) Try to reproduce the problem after upgrading to the latest Evolution and
   GtkHtml packages in Rawhide.

3) If you still see the ampersand problem, export a sample mail exhibiting
   the problem as an mbox file and attach it to this bug.  Please remember
   to sanitize any sensitive data before posting.

Comment 2 Thomas J. Baker 2007-10-01 15:12:04 UTC
I'm already running rawhide:

evolution-2.12.0-3.fc8.x86_64
gtkhtml3-3.16.0-1.fc8.x86_64

I'll include an ebay search list email the next time I get one.

Comment 3 Thomas J. Baker 2007-10-01 15:20:30 UTC
Created attachment 212391 [details]
ebay email


Here is an ebay email that should show the problem. I don't think my sanitizing
messed anything up.

Comment 4 Milan Crha 2007-10-17 15:56:15 UTC
Hi Thomas, thanks for your data. It will be better to track this bug in upstream
bug http://bugzilla.gnome.org/show_bug.cgi?id=485700 , as you found it already,
so closing this one.

Comment 5 Thomas J. Baker 2007-10-23 17:20:39 UTC
Since you submitted a patch upstream, why couldn't it be applied to F8 gtkhtml?
This is quite an annoying bug for anyone who uses ebay.

Comment 6 Matthew Barnes 2007-10-23 17:39:39 UTC
The patch will be included in Fedora once upstream approves it.

Comment 7 Thomas J. Baker 2007-10-29 14:08:56 UTC
According to the referenced gnome bug report, the change has been committed
upstream by Milan. I'm willing to test private or koji builds.

Comment 8 Matthew Barnes 2007-10-30 03:06:26 UTC
Should be fixed in gtkhtml3-3.17.1-1.fc9.  Feel free to re-open if not.

Comment 9 Thomas J. Baker 2007-10-30 12:46:16 UTC
Yes it does. Is this going to make F8 release?

Comment 10 Matthew Barnes 2007-10-30 13:52:33 UTC
It won't make the initial release of Fedora 8 (that's frozen now) but it will be
included in the first stable update.

Comment 11 Thomas J. Baker 2007-11-29 02:50:09 UTC
Just a ping to request this gets pushed to F8.

Comment 12 Matthew Barnes 2007-11-29 11:20:25 UTC
This should be fixed in Evolution 2.12.2, which is available in Updates Testing.

http://download.fedora.redhat.com/pub/fedora/linux/updates/testing/8/

Comment 13 Thomas J. Baker 2007-12-03 21:55:04 UTC
The current release in updates testing seems to work fine.

Comment 14 Matthew Barnes 2007-12-11 21:03:25 UTC
Thanks.  Closing this as CURRENTRELEASE.


Note You need to log in before you can comment on or make changes to this bug.