sanitize new search format

RESOLVED FIXED in Firefox 2 beta1

Status

()

defect
P2
normal
RESOLVED FIXED
14 years ago
5 years ago

People

(Reporter: Pike, Assigned: Gavin)

Tracking

({fixed1.8.1})

2.0 Branch
Firefox 2 beta1
Points:
---
Dependency tree / graph
Bug Flags:
blocking-firefox2 +

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment)

bug 317107 comment 34 says:

>> I appreciate Axel's point about there being confusion and likely errors on the
>> part of localizers when creating these files. The best way to avoid that is to:
>> 
>> - use strict namespace checking in the search service and pro-actively throw
>> exceptions when there are failures
>> - use a namespace prefix on the root, e.g.
>> 
>> <moz:SearchPlugin xmlns:moz="http://www.mozilla.org/2006/browser/search/"
>> xmlns="http://a9.com/-/spec/opensearchdescription/1.1/">
>>   <ShortName>Amazon.com</ShortName>
>>   ...
>>   <moz:Alias>am</moz:Alias>
>> </moz:SearchPlugin>
>> 
>> since this is what people copy from, and it makes it clear what our extensions
>> are.
>
> If you don't have any objection I'd rather leave the format as is for now -
> making ours be the default namespace simplifies checking which type of plugin
> we're dealing with. This can be revisited for a2.
I'd really love to just use OpenSearch and forget about this almost-OpenSearch stuff.
Assignee: nobody → gavin.sharp
Priority: -- → P1
Status: NEW → ASSIGNED
Whiteboard: [swag: 1d]
Cleanup is b1, we need to get features implemented for testing/iteration first.  I know the l10n people won't love this, but b1 should still give them months.
Flags: blocking-firefox2+
Target Milestone: Firefox 2 alpha2 → Firefox 2 beta1
Priority: P1 → P2
With search suggest we do have an extension to OS that warrants the introduction
of a mozilla namespace into the search plugin, so I'd take down in writing the 
outcome of a discussion that gavin and I had a while back which was that we'd like
to have one namespace, and it'd be fine to just drop the OS one from our plugin
and have the parser for search xml engines accept the OS namespace in addition to
the mozilla one.
Blocks: 339783
This simplifies the file format, and the parser doesn't care since it just checks localNames.
Attachment #223991 - Flags: review?(mconnor)
Attachment #223991 - Flags: approval-branch-1.8.1?(mconnor)
Whiteboard: [swag: 1d] → [patch-r?]
Attachment #223991 - Flags: review?(mconnor)
Attachment #223991 - Flags: review+
Attachment #223991 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #223991 - Flags: approval-branch-1.8.1+
Checked in branch and trunk!
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [patch-r?]
Blocks: 1039003
You need to log in before you can comment on or make changes to this bug.