evolution mailto CLI cannot handle attachments with "#" in filename

Bug #304036 reported by Bob Blanchard
6
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Medium
evolution (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs
Intrepid
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

When attempting to attach a file from the command line using "evolution 'mailto:?attachment=file:///...' " and the filename has a "#" (0x23) in it, the filename gets truncated at the "#", and therefore cannot attach the file. URI-encoded filenames have the same problem (with %23).

This is a fairly serious problem since OpenOffice uses this method for attaching files with choosing "File -> Send..". So any OpenOffice document with "#" in the filename, cannot be sent with evolution. (of course workaround is saving the file, and then launching evolution, and attaching from a compose window ... a total pain).

**Steps To Reproduce... Create an OpenOffice document with a "#" in the name.. set evolution as your mail handler.. and try to send the document. OR try from the command line: evolution 'mailto:?attachment=file:///home/user/some # document'

Below is example of how OO uri-encodes the filename, but evolution still truncates after the %23

++ echo 'file:///tmp/svjia.tmp/svjm4.tmp/K1 London #2 (asdf) CofA.pdf'
++ /usr/lib/openoffice/program/uri-encode
+ MAILTO='attachment=file:///tmp/svjia.tmp/svjm4.tmp/K1%20London%20%20%232%20%20%20(asdf)%20CofA.pdf'
+ shift
+ shift
+ '[' '' '!=' '' ']'
+ MAILTO='mailto:?attachment=file:///tmp/svjia.tmp/svjm4.tmp/K1%20London%20%20%232%20%20%20(asdf)%20CofA.pdf'
+ evolution 'mailto:?attachment=file:///tmp/svjia.tmp/svjm4.tmp/K1%20London%20%20%232%20%20%20(asdf)%20CofA.pdf'
+ exit 0

Related branches

Revision history for this message
Pedro Villavicencio (pedro) wrote :

confirming, will look upstream, thanks for the report.

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Pedro Villavicencio (pedro) wrote :

I've sent this upstream at: http://bugzilla.gnome.org/show_bug.cgi?id=562886 ; thanks for reporting.

Changed in evolution:
status: Confirmed → Triaged
Revision history for this message
Pedro Villavicencio (pedro) wrote :

this is fixed upstream already, thanks for reporting.

Changed in evolution:
status: Triaged → Fix Committed
Changed in evolution:
status: Unknown → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

upstream rolled a new stable version today which has been uploaded as a candidate update to intrepid now

Revision history for this message
Steve Langasek (vorlon) wrote :

Accepted into intrepid-proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in evolution:
status: New → Fix Committed
Revision history for this message
Bob Blanchard (blabj) wrote :

Will this fix make it into hardy? It is an LTS release.. and is my companies primary platform.

Revision history for this message
Sebastien Bacher (seb128) wrote :

that seems to be a good candidate for a backport once the fix has been confirmed to work on intrepid and jaunty

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evolution - 2.24.3-0ubuntu1

---------------
evolution (2.24.3-0ubuntu1) intrepid-proposed; urgency=low

  * New upstream version:
    Bug Fixes:
    - #332729: Invalid write with outspring from e-config.c
    - #337082: Unnamed imap folder (lp: #37227)
    - #435452: Adding People to Meeting Loses Attendee if Add or
               Busy Search Selected
    - #435694: Retract of Email from Sent Items Does not Work
    - #439733: Checking for junk before training
    - #446285: Meeting Entry Does Not Allow Multiple Names
    - #458968: Don't warn the user about changes being lost if he is trying to
               delegate the meeting
    - #546637: Mail opened from the "Unread mails" displays empty (lp: #275952)
    - #551599: Do not fallback to default 15 minutes alarm offset if alarm is
               triggered at start
    - #552583: Passwords don't get stored when using other authentication
            type than "Password"
    - #552583: Free the right url
    - #552583: Account checking logic a bit more consistent
    - #552583: Fix bugs in the account checking logic
    - #552583: Compare the protocol, user, host and port and disregard the rest
    - #555663: Evolution crashed with SIGSEGV in gconf-bridge (lp: #280675)
    - #556303: Evolution crashed with SIGSEGV in camel_mime_part_get_filename
               (lp: #304385)
    - #557176: Crash while trying to click on 'Contacts'
    - #558337: Evolution crashes while managing IMAP folder subscriptions
    - #559153: Unable to convert mailboxes from Hardy to Intrepid versions
    - #560420: Failed spamassassin pipe on Evolution (lp: #275746)
    - #562091: New share memo window doesn't come up
    - #562155: Evolution stopped to start
    - #562450: Edit as New Message doesn't copy the content of a
               GPG signed message
    - #562886: Evolution mailto CLI cannot handle attachments with
               "#" in filename (lp: #304036)
    - #563077: Evolution crashes if gnome-settings-daemon is not
               started on Solaris
    - #563250: Save and restore custom composer accelerators
    - #563369: "Templates" folder not translated
    - #564007: Messages do not show inline decrypted message by default
    - #564860: Evolution crashes in e_util_labels_parse if the labels don't have
               the correct format
    - #565553: Right click on an attachment in calendar event
               and chose application does not work
    - #566653: Local folder's DnD aren't saved well to DB
  * debian/control:
    - updated evolution-data-server requirement

 -- Sebastien Bacher <email address hidden> Mon, 12 Jan 2009 16:23:51 +0100

Changed in evolution:
status: Fix Committed → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

the new version is in jaunty now

Changed in evolution:
status: Fix Committed → Fix Released
Changed in evolution:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.