Open Bug 630432 Opened 13 years ago Updated 13 years ago

Read GetURL's whitelist of supported protocols from the plist

Categories

(Camino Graveyard :: OS Integration, defect)

All
macOS
defect
Not set
minor

Tracking

(Not tracked)

People

(Reporter: alqahira, Unassigned)

References

Details

To fix bug 629850, I added a whitelist of supported protocols to GetURLCommand.  In his review, Chris Peterson suggested reading those protocols from the plist (where we already announce to the world what protocols we claim to handle).  Stuart agreed, particularly so we don't get out-of-sync between the two as we add and remove protocol support going forward, though he noted it could be a low-priority follow-up.
There's an unfortunate wrinkle to this, which is described in bug 629850 comment 19. :(
We could do plist-plus-additional-entries, so we'd read everything from the plist, then append about: in code.
(In reply to comment #2)
> We could do plist-plus-additional-entries, so we'd read everything from the
> plist, then append about: in code.

We eventually allowed about:, data:, and view-source: as allowed protocols in bug 629850 (data: for parity with Safari, about: and view-source: mainly for toolbar scripts).
You need to log in before you can comment on or make changes to this bug.