Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

httpsサーバに乗せるとシステム一覧が取得できない #7

Open
Minori-Akizuki opened this issue Aug 16, 2017 · 11 comments

Comments

@Minori-Akizuki
Copy link
Owner

bcdice-api を外部に投げており、それがhttpなのが原因。こいつも一緒に立てる必要がある。

@Minori-Akizuki
Copy link
Owner Author

で、クエリの投げ先を変数にして追い出す必要がある。

@Minori-Akizuki
Copy link
Owner Author

それかサーバ側からシステム一覧を取得する。ダイス振るのはサーバからだし、その方が統一感ある?

@daicyan
Copy link

daicyan commented Aug 16, 2017

クライアントで取得だと結果を頑張れば偽装できちゃうのでサーバーから投げるほうに一票。
HTTPS対応版のAPIサーバーを用意することもできますが、どっちにしろ別ドメインのデータを取得するのはクロスサイトスクリプティング攻撃の穴になりやすいので個人的には辞めたほうが良いと思います。

@deflis
Copy link

deflis commented Aug 16, 2017

bcdiceをjavascriptに移植してクライアントサイドなりに押し込められないですかね?

@deflis
Copy link

deflis commented Aug 16, 2017

他の案としては、CORS制約をちゃんと設定してやるとか、同じHTTPSサーバ内に共存させるというのが良いかと思います。

@c6h4clch3
Copy link
Contributor

個人的にはダイスボットはサーバ側で実装がいいと思います。
ロール結果メッセージをサーバ側から発行すれば結果の改ざん、
サーバ負荷の両方ケアできますし。

各々で立てたサーバ内で処理が完結したほうがいいと思うのです。

@Minori-Akizuki
Copy link
Owner Author

サーバ内に立てるのが一番ですかね。いま公式鯖のところにAPI投げちゃってますが、単体でたててローカルホストにAPI投げるようにすれば一元化できそうですし。ちょっとその方向性で考えてみます。

@daicyan
Copy link

daicyan commented Aug 23, 2017

公式鯖にHTTPS版APIサーバー建てました。
https://www.taruki.com/bcdice-api/v1/

@Minori-Akizuki
Copy link
Owner Author

Minori-Akizuki commented Aug 24, 2017

公式鯖にHTTPS版APIサーバー建てました。

うわぁありがとうございます……!!
まさかの公式鯖でHTTPS対応! なんか申し訳ないです(´・ω・`)
一旦向き先をそちらにして対処します。

@c6h4clch3
Copy link
Contributor

https://github.com/ukatama/bcdice-js
こちらでBCDiceのnpmモジュール化進めてらっしゃるみたいですし、
こういった感じのが使えればいいんですけどねー

@deflis
Copy link

deflis commented Aug 24, 2017

おお、BCDiceをOpalで動かすプロジェクトあるんですね。
これを同じサーバで動かせば良さそうな気がします。

BCDice-APIの呼び出し方法がJSONPになってるのが個人的にはかなりアレだなと思うので、
できれば Access-Control-Allow-Origin: * ヘッダを付けてもらってクロスドメインでAJAXアクセス出来るようにしたほうが良いと思います。詳しくはこの辺のMDNを参照してもらえれば…
(アイマストドン関連とかで忙しくなければこの�辺への貢献もう少し出来るのですが…。申し訳ありません。)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants