commandline for external protocol handler is mishandled

RESOLVED FIXED

Status

Core Graveyard
File Handling
RESOLVED FIXED
10 years ago
10 months ago

People

(Reporter: Rich Walsh, Assigned: Peter Weilbacher)

Tracking

({fixed1.8.1.12})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
Depending on where an external protocol handler is specified, the target app will receive different commandline arguments.  The examples below use "mailto:" but the behavior described applies to all supported protocols.

If you use os2.ini->DefaultMailParameters, specifying either "-compose" or "-compose %url%" will produce the same commandline:
e.g. " -compose mailto:xxx@yyy.com".
 
Using prefs.js->applications.mailto.parameters will produce different results. Entering just "-compose" results in the same commandline as above.  However, entering "-compose %url%" produces:
e.g. " -compose xxx@yyy.com mailto:xxx@yyy.com".
(Assignee)

Updated

10 years ago
Component: OS Integration → File Handling
Product: Firefox → Core
QA Contact: os.integration → file-handling
(Assignee)

Comment 1

10 years ago
Created attachment 294574 [details] [diff] [review]
don't append URL if already replaced full URL

This should fix this. At least on the 1.8 branch where this feature still works. (I would check the same change into trunk but there this method was moved to nsMIMEInfoOS2 so I would patch nsMIMEInfoOS2.cpp instead when checking it in.)
Assignee: nobody → mozilla
Status: NEW → ASSIGNED
Attachment #294574 - Flags: review?(mozilla)

Updated

10 years ago
Attachment #294574 - Flags: review?(mozilla) → review+
(Assignee)

Comment 2

10 years ago
Fix checked into trunk (uriloader/exthandler/os2/nsMIMEInfoOS2.cpp v1.11) and 1.8 branch (uriloader/exthandler/os2/nsOSHelperAppService.cpp v1.41.18.1).
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: fixed1.8.1.12
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.