Attachment 54444 Details for Bug 16615 – Bug 17103 - Google API Loader jsapi called over http

[patch] Bug 17103 - Google API Loader jsapi called over http

Bug-17103---Google-API-Loader-jsapi-called-over-ht.patch (text/plain), 1.94 KB, created by Mark Tompsett on 2016-08-14 17:07:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2016-08-14 17:07:51 UTC

Size: 1.94 KB

patch

obsolete

>From bbdf675b801544171c4eab1339a195b1b4c20ac6 Mon Sep 17 00:00:00 2001
>From: "Indranil Das Gupta (L2C2 Technologies)" <indradg@l2c2.co.in>
>Date: Sat, 13 Aug 2016 05:55:14 +0530
>Subject: [PATCH] Bug 17103 - Google API Loader jsapi called over http
>
>This patch replaces the hardcoded protocol ("http") from the URI with
>https since that is what Google defacto uses to serve JS resources
>Prevents MIXED CONTENT failure, allowing GoogleIndicTransliteration
>to function correctly when used in SSLized OPACs.
>
>Test plan
>---------
>
>1/ Make sure your OPAC is being served both over HTTP and HTTPS
>2/ Set GoogleIndicTransliteration syspref to "Show"
>3/ Access the OPAC over http, Google transliteration element will
>   show up in masthead searchbox.
>4/ Access the OPAC over https. The element will not show. Console
>   will show MIXED CONTENT error and failure of google loader.
>5/ Apply patch. Repeat steps #3 and #4 again. Now in both cases
>   GoogleIndicTransliteration will work.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>
>https://bugs.koha-community.org/show_bug.cgi?id=16615
>---
> koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>index 9d9af2c..3348c21 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>@@ -209,7 +209,7 @@ $.widget.bridge('uitooltip', $.ui.tooltip);
>     </script>
> [% END %]
> [% IF ( GoogleIndicTransliteration ) %]
>-    <script type="text/javascript" src="http://www.google.com/jsapi"></script>
>+    <script type="text/javascript" src="https://www.google.com/jsapi"></script>
>     <script type="text/javascript" src="[% interface %]/[% theme %]/js/googleindictransliteration.js"></script>
> [% END %]
> [% IF persona %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16615: 51912 | 54440 | 54441 | 54442 | 54443 | 54444 | 54456 | 54462 | 54463 | 54464