操作
Backport(バックポート) #3523
完了google mapのURLがhttpsで始まる場合、小窓化されない
開始日:
2010-11-02
期日:
進捗率:
100%
予定工数:
説明
Overview (現象)¶
google mapのURLがhttpsで始まる場合、小窓化されない。
httpで始まるURLであれば小窓化されます。
再現バージョン¶
下記のバージョンで再現確認済み。
- OpenPNE3.6.13
- OpenPNE3.8.9
再現手順¶
日記にhttpsで始まる google map の URLを入力し、表示を確認する
Causes (原因)¶
小窓化用の maps.google.com.js がhttpsに対応していないため
Way to fix (修正内容)¶
https://redmine.openpne.jp/issues/1755#note-4 に添付されている差分より
diff --git a/web/cmd/maps.google.co.jp.js b/web/cmd/maps.google.co.jp.js
index a38b2c1..99ede78 100644
--- a/web/cmd/maps.google.co.jp.js
+++ b/web/cmd/maps.google.co.jp.js
@@ -1,6 +1,6 @@
function url2cmd(url, googlemapsUrl) {
- var maps_google_co_jp_maps = url.match(/^http:\/\/maps\.google\.co\.jp\/maps\?(.+)/);
- var maps_google_co_jp = url.match(/^http:\/\/maps\.google\.co\.jp\/\?(.+)/);
+ var maps_google_co_jp_maps = url.match(/^https?:\/\/maps\.google\.co\.jp\/maps\?(.+)/);
+ var maps_google_co_jp = url.match(/^https?:\/\/maps\.google\.co\.jp\/\?(.+)/);
if (maps_google_co_jp_maps || maps_google_co_jp) {
var id = RegExp.$1;
diff --git a/web/cmd/maps.google.com.js b/web/cmd/maps.google.com.js
index 9de34e3..4e8eee5 100644
--- a/web/cmd/maps.google.com.js
+++ b/web/cmd/maps.google.com.js
@@ -1,6 +1,6 @@
function url2cmd(url, googlemapsUrl) {
- var maps_google_com_maps = url.match(/^http:\/\/maps\.google\.com\/maps\?(.+)/);
- var maps_google_com = url.match(/^http:\/\/maps\.google\.com\/\?(.+)/);
+ var maps_google_com_maps = url.match(/^https?:\/\/maps\.google\.com\/maps\?(.+)/);
+ var maps_google_com = url.match(/^https?\/\/maps\.google\.com\/\?(.+)/);
if (maps_google_com_maps || maps_google_com) {
var id = RegExp.$1;
isao sano さんがほぼ11年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
元チケットの修正を取り込んだものをpull request行いました。
https://github.com/openpne/OpenPNE3/pull/100
Shinichi Urabe さんがほぼ11年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
修正内容自体問題ないと判断しプルリクエストを取り込みました。
フィードバックします
- Google マップ以外にも YouTube など、他の小窓でも本現象が発生しますが、対応する必要はないでしょうか
- 別チケットでの対応でよいかとは思いますが、バグチケットは見当たりませんでした
- 元々の問題ですが本文の URL が https | http であるか区別せず小窓を表示する対応になっている点は問題無いですが、表示しているページの URL が https の場合にユーザが http の URL を貼り付けた場合、ブラウザからエラーが通知されたり、コンテンツが表示されない現象が発生しますが、その対処は不要でしょうか
Youichi Kimura さんがほぼ11年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 273d4013113611a20fb5b41086c7bfaaa00c2da0 で適用されました。
Mutsumi Imamura さんがほぼ11年前に更新
- コピー先 Bug(バグ) #3551: 表示しているページのURLがhttpsの場合にユーザが httpのURL の小窓を作成した場合ブラウザのエラーが表示される を追加
Mutsumi Imamura さんがほぼ11年前に更新
- ステータス を Rejected(差し戻し) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Shinichi Urabe は書きました:
フィードバックします
- Google マップ以外にも YouTube など、他の小窓でも本現象が発生しますが、対応する必要はないでしょうか
- 別チケットでの対応でよいかとは思いますが、バグチケットは見当たりませんでした
- 元々の問題ですが本文の URL が https | http であるか区別せず小窓を表示する対応になっている点は問題無いですが、表示しているページの URL が https の場合にユーザが http の URL を貼り付けた場合、ブラウザからエラーが通知されたり、コンテンツが表示されない現象が発生しますが、その対処は不要でしょうか
フィードバック内容については下記のチケットで対応することとします。
よって、本チケットの修正範囲としては問題無いと判断し、ステータスを変更します。
Chiharu Nakajima さんがほぼ11年前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
操作