840699 - Use HTTPS instead of HTTP for 30boxes.com integration

Status: VERIFIED FIXED

(Firefox :: File Handling, defect)

Description

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

+++ This bug was initially created as a clone of Bug #771788 +++

gecko.handlerService.schemes.webcal.0.uriTemplate;http://30boxes.com/external/widget?refer=ff&url=%s

This should use an https:// link.

Comment 1

[Raymond Lee [:raymondlee]]

Attachment: v1

A minor change

Comment 2

[:Gavin Sharp [email: [e-mail]]]

We'll need to connect with the folks at 30box to ensure they're OK with this change. This is old code, I don't know whether we've maintained a relationship with them. 

Joanne, is this something you can help with?

Comment 3

[Joanne Nagel]

Sorry, Gavin, I don't know anyone at 30box - I only manage our search & commerce relationships (Amazon, eBay, Bing, Yahoo, etc.)

Comment 4

[Raymond Lee [:raymondlee]]

(In reply to Joanne Nagel from comment #3)
> Sorry, Gavin, I don't know anyone at 30box - I only manage our search &
> commerce relationships (Amazon, eBay, Bing, Yahoo, etc.)

Does it mean we don't maintain a relationship with 30box?

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 720573 [details] [diff] [review]
v1

Let's just do this.

Comment 6

[Raymond Lee [:raymondlee]]

Attachment: Patch to checkin

Comment 7

[Raymond Lee [:raymondlee]]

(In reply to Raymond Lee [:raymondlee] from comment #6)
> Created attachment 780148 [details] [diff] [review]
> Patch to checkin

BTW this patch also increment the gecko.handlerService.defaultHandlersVersion number

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/74633f25ca05

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/74633f25ca05

Comment 10

[Axel Hecht [:Pike]]

I'm wondering, did we figure out the removal of the http handler piece that we're suffering with with the yahoo version of this patch? bug 840705 comment 25 et al sounds like we're not really replacing stuff.

Comment 11

[Raymond Lee [:raymondlee]]

(In reply to Axel Hecht [:Pike] from comment #10)
> I'm wondering, did we figure out the removal of the http handler piece that
> we're suffering with with the yahoo version of this patch? bug 840705
> comment 25 et al sounds like we're not really replacing stuff.

No, I have filed Bug 904350.

Comment 12

[Camelia Badau [:cbadau], Desktop Test Engineering]

Verified fixed on Windows 7 64bit, Ubuntu 13.10 and Mac OSX 10.8.5 using latest Nightly 34.0a1.