Closed
Bug 1396927
Opened 7 years ago
Closed 7 years ago
Diec2 search plugin is broken
Categories
(Firefox :: Search, defect)
Firefox
Search
Tracking
()
RESOLVED
FIXED
Firefox 57
People
(Reporter: flod, Assigned: flod)
References
()
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
mkaply
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
From dev-l10n
> A search plugin included in Firefox for Catalan (ca) locale stopped working due to the site having switched to https.
> The searchplugin is diec2.xml (https://dlc.iec.cat).
Assignee | ||
Comment 1•7 years ago
|
||
Uhm, the search plugin loads http://dlc.iec.cat/results.asp It redirects to https, but the problem is that the https version doesn't load at all in Nightly https://dlc.iec.cat/ Switching the URL in the search plugin wouldn't solve the problem on Nightly, only on Beta and Release. The URL loads in Chrome and Firefox Beta, where security settings are less strict. https://www.ssllabs.com/ssltest/analyze.html?d=dlc.iec.cat @Jordi Are you in touch with them? Any chance they can fix the certificate problems on their side?
Flags: needinfo?(jordis)
Assignee | ||
Comment 2•7 years ago
|
||
For reference. * This works in Beta https://dlc.iec.cat/results.asp?txtEntrada=test&OperEntrada=0, but fails in Nightly because of the certificate issue. * This fails in Beta with a VBScript error http://dlc.iec.cat/results.asp?txtEntrada=test&OperEntrada=0
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
I realised that Nightly does not open the site at all, while Firefox 55 (release) does. I posted a question about this: https://support.mozilla.org/es/questions/1174309 and they point me to the same SSL Labs site issues. I already sent a mail to the site telling them about the SSL certificate issues, let's see if they react. Fixing it for Release+Beta would be already a step forward.
Flags: needinfo?(jordis)
Assignee | ||
Comment 5•7 years ago
|
||
@Mike To clarify. This would fix at least the issue on the Beta/Release channel (I need to ask for approval), hopefully they'll fix the certificate in the meantime.
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8904660 [details] Bug 1396927 - Switch Diec2 (Catalan) search plugin to https https://reviewboard.mozilla.org/r/176466/#review181428
Attachment #8904660 -
Flags: review?(mozilla) → review+
Pushed by francesco.lodolo@mozillaitalia.org: https://hg.mozilla.org/integration/autoland/rev/c3ce49cb24a6 Switch Diec2 (Catalan) search plugin to https r=mkaply
Assignee | ||
Comment 8•7 years ago
|
||
Ugh, I need to move this to be able to ask for approval :-\
Assignee | ||
Updated•7 years ago
|
Component: ca / Catalan → Search
Product: Mozilla Localizations → Firefox
Assignee | ||
Comment 9•7 years ago
|
||
Comment on attachment 8904660 [details] Bug 1396927 - Switch Diec2 (Catalan) search plugin to https Approval Request Comment [Feature/Bug causing the regression]: a server change side broken the search plugin currently shipping for Diec2 on both Fennec and Firefox [User impact if declined]: search plugin is broken [Is this code covered by automated tests?]: no [Has the fix been verified in Nightly?]: no, but manually tested in Beta [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: - [Is the change risky?]: no, only affects a search plugin for one locale (Catalan) [Why is the change risky/not risky?]: - [String changes made/needed]: -
Attachment #8904660 -
Flags: approval-mozilla-beta?
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c3ce49cb24a6
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•7 years ago
|
status-firefox56:
--- → affected
Comment 11•7 years ago
|
||
Comment on attachment 8904660 [details] Bug 1396927 - Switch Diec2 (Catalan) search plugin to https Sounds like the scope of this fix is fairly limited. Let's try uplift for beta 10.
Attachment #8904660 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 12•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/abc3387aaa6f
You need to log in
before you can comment on or make changes to this bug.
Description
•