Replace occurrence of `http` in templates with `https` #1157

Clpsplug · 2017-04-03T08:45:22Z

Closes #1156.

giginet

今回の変更には問題ないけど、
元々ハードコードしているところを何カ所かみつけてしまった。
気が向いたらそちらも直して欲しい

giginet · 2017-04-03T08:48:02Z

src/templates/activities/hatenablog/hatenablogentry_created.hipchat.html

@@ -1 +1 @@
-<strong>広報ブログに「<a href="http://{{ site.domain }}{{ object.get_absolute_url }}">{{ object }}</a>」が投稿されました</strong>
+<strong>広報ブログに「<a href="https://{{ site.domain }}{{ object.get_absolute_url }}">{{ object }}</a>」が投稿されました</strong>


これそもそもリンクあってるんだっけ

{{ object.get_absolute_url }}だけで良い気がする･･････
帰ったら確認する

giginet · 2017-04-03T08:48:36Z

src/templates/activities/products/product_release_added.hipchat.html

@@ -1,2 +1,2 @@
-<strong>{{ object.last_modifier.nickname }}さんが「{{ release.product.title }}」に「<a href="http://{{ site.domain }}{{ object.get_absolute_url }}">{{ release.label }}</a>」を追加しました</strong>
+<strong>{{ object.last_modifier.nickname }}さんが「{{ release.product.title }}」に「<a href="https://{{ site.domain }}{{ object.get_absolute_url }}">{{ release.label }}</a>」を追加しました</strong>


HipChatもう消して良さそうだなｗ

giginet · 2017-04-03T08:51:36Z

src/templates/registration/acceptance_email.txt

@@ -22,7 +22,7 @@ http://{{ site.domain }}{% url 'registration_activate' activation_key=activation

 また、KawazではSlackというチャットツールの設定を使って交流やゲーム開発を行っています。
 詳しい登録作業については、以下のページを参照してください。
-http://www.kawaz.org/helps/slack/
+https://www.kawaz.org/helps/slack/


ついでにhttps://{{site.domain}}/helps/slack/に変えておいて欲しい

giginet · 2017-04-03T08:51:51Z

src/templates/registration/activation_email.txt


 より再設定を行ってください。今後とも宜しくお願いいたします。

 Kawazでは登録いただいたこのポータルサイトやチャットツールを使って、メンバー間の交流をしています。
 詳しくはこのページをどうぞ！
-http://www.kawaz.org/helps/welcome/
+https://www.kawaz.org/helps/welcome/


{{ site.domain }}

giginet · 2017-04-03T08:52:38Z

src/templates/registration/password_reset_email.html

@@ -15,7 +15,7 @@
 -------------------------------------------------------
 札幌ゲーム製作者コミュニティ - Kawaz

-ホームページ:	http://www.kawaz.org/
+ホームページ:	https://www.kawaz.org/


これハードコードされてるのがおかしいな。9行目見る感じ

{{ protocol }}://{{ domain }}で直せそう。お願いします

giginet · 2017-04-03T08:52:53Z

src/templates/registration/notification_email.txt

@@ -27,7 +27,7 @@ http://{{ site.domain }}/central-dogma/registration/registrationprofile/{{ profi
 -------------------------------------------------------
 札幌ゲーム製作者コミュニティ - Kawaz

-ホームページ: http://{{ site.domain }}
+ホームページ: https://{{ site.domain }}


ここと同様に行けないかな

https://github.com/kawazrepos/Kawaz3rd/pull/1157/files#r109366496

giginet · 2017-04-03T08:53:02Z

src/templates/registration/registration_email.txt

@@ -19,7 +19,7 @@
 -------------------------------------------------------
 札幌ゲーム製作者コミュニティ - Kawaz

-ホームページ: http://www.kawaz.org/
+ホームページ: https://www.kawaz.org/


https://github.com/kawazrepos/Kawaz3rd/pull/1157/files#r109366496

giginet · 2017-04-03T08:53:12Z

src/templates/registration/rejection_email.txt

@@ -3,7 +3,7 @@
 {{ user.username }}さま

 申し訳ありませんが新規会員登録申請が管理者によって拒否されました。
-利用規約（ http://www.kawaz.org/rules/ ）にあるとおり、以下の点についてもう一度ご確認ください
+利用規約（ https://www.kawaz.org/rules/ ）にあるとおり、以下の点についてもう一度ご確認ください


https://github.com/kawazrepos/Kawaz3rd/pull/1157/files#r109366496

giginet · 2017-04-03T08:56:01Z

コメントした中で、HipChat消す奴はこっちでやっておくので無視してOK

giginet · 2017-04-19T17:46:26Z

@Clpsplug #1163 rebaseして、指摘ヶ所直してー

giginet · 2017-06-20T15:54:27Z

@Clpsplug これどうなってるー？

forgot to replace some of httpses

Clpsplug · 2017-06-20T16:58:07Z

多分ハードコードしたhttpsは取り除けたと思います

coveralls · 2017-06-20T17:05:24Z

Coverage remained the same at 97.696% when pulling 758ea09 on plug/template_https into 8f9a850 on develop.

giginet

これムズいんだけど

password_reset_emailはDjangoの標準機能、その他の奴はdjango-inspectional-registrationから来ている。

django-inspectional-registrationのメールテンプレートでもprotocol, domainという変数が使えるかちゃんと確認してからマージした方が良さそう。

他はLGTM

giginet · 2017-06-20T17:09:28Z

src/templates/registration/acceptance_email.txt

@@ -8,7 +8,7 @@ Kawazへの登録申請ありがとうございます。会員登録が承認さ

 会員登録を完了するために{{ expiration_days }}日以内に以下のURLをクリックしてアカウントを有効にしてください。

-https://{{ site.domain }}{% url 'registration_activate' activation_key=activation_key %}
+{{ protocol }}://{{ domain }}{% url 'registration_activate' activation_key=activation_key %}


ドキュメントを読んだ感じ、site.domainの方が正しそうだけど大丈夫かな。これで動いた？

https://django-inspectional-registration.readthedocs.io/en/latest/about_registration_templates.html

@lambdalisue

coveralls · 2017-06-20T17:14:14Z

Coverage remained the same at 97.696% when pulling 758ea09 on plug/template_https into 8f9a850 on develop.

coveralls · 2017-06-20T17:15:29Z

Coverage remained the same at 97.696% when pulling 758ea09 on plug/template_https into 8f9a850 on develop.

giginet approved these changes Apr 3, 2017

View reviewed changes

Clpsplug added 2 commits June 21, 2017 01:52

replace occurrence of http in templates with https

1a474c0

Got rid of hardcoding in email templates

c29a853

Clpsplug force-pushed the plug/template_https branch from d39e714 to c29a853 Compare June 20, 2017 16:53

OOPS

758ea09

forgot to replace some of httpses

giginet approved these changes Jun 20, 2017

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace occurrence of `http` in templates with `https` #1157

Replace occurrence of `http` in templates with `https` #1157

Clpsplug commented Apr 3, 2017

giginet left a comment

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet Apr 3, 2017

giginet commented Apr 3, 2017

giginet commented Apr 19, 2017

giginet commented Jun 20, 2017

Clpsplug commented Jun 20, 2017

coveralls commented Jun 20, 2017 •

edited

Loading

giginet left a comment

giginet Jun 20, 2017

coveralls commented Jun 20, 2017 •

edited

Loading

coveralls commented Jun 20, 2017 •

edited

Loading

		@@ -1 +1 @@
		<strong>広報ブログに「<a href="http://{{ site.domain }}{{ object.get_absolute_url }}">{{ object }}</a>」が投稿されました</strong>
		<strong>広報ブログに「<a href="https://{{ site.domain }}{{ object.get_absolute_url }}">{{ object }}</a>」が投稿されました</strong>

		@@ -1,2 +1,2 @@
		<strong>{{ object.last_modifier.nickname }}さんが「{{ release.product.title }}」に「<a href="http://{{ site.domain }}{{ object.get_absolute_url }}">{{ release.label }}</a>」を追加しました</strong>
		<strong>{{ object.last_modifier.nickname }}さんが「{{ release.product.title }}」に「<a href="https://{{ site.domain }}{{ object.get_absolute_url }}">{{ release.label }}</a>」を追加しました</strong>

Replace occurrence of http in templates with https #1157

Are you sure you want to change the base?

Replace occurrence of http in templates with https #1157

Conversation

Clpsplug commented Apr 3, 2017

giginet left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

giginet commented Apr 3, 2017

giginet commented Apr 19, 2017

giginet commented Jun 20, 2017

Clpsplug commented Jun 20, 2017

coveralls commented Jun 20, 2017 • edited Loading

giginet left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

coveralls commented Jun 20, 2017 • edited Loading

coveralls commented Jun 20, 2017 • edited Loading

Replace occurrence of `http` in templates with `https` #1157

Replace occurrence of `http` in templates with `https` #1157

coveralls commented Jun 20, 2017 •

edited

Loading

coveralls commented Jun 20, 2017 •

edited

Loading

coveralls commented Jun 20, 2017 •

edited

Loading