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

ローカル環境でのtraQのOAuthの認証がうまく行かない問題 #500

Merged
merged 2 commits into from
Dec 3, 2023

Conversation

Luftalian
Copy link
Member

close #499

@Luftalian
Copy link
Member Author

Luftalian commented Dec 3, 2023

作成したClientの認証情報を、外部に流出させない。
Clientで受信した各種情報を、ユーザーないしはtraPの承諾無しに外部へ公開しない。

https://bot-console.trap.jp/docs/client/rule

compose.ymlにローカル環境のための認証情報を書いているのですが、それを公開する許可をほしいです

CLIENT_ID: ${CLIENT_ID:-aYj6mwyLcpBIrxZZD8jkCzH3Gsdqc9DJqle2}

ここに新しいClient IDを使いたいです。

@Luftalian
Copy link
Member Author

mdの方も許可出たら修正します。

@ras0q
Copy link
Member

ras0q commented Dec 3, 2023

compose.ymlに認証情報を書いているのですが、それを公開する許可をほしいです

Docker Composeは自動で.envを読みに行くので.envに

CLIENT_ID=hogehoge

でいけませんか?

@Luftalian
Copy link
Member Author

それでも出来たんですが、#500 (comment) に追記したんですが、今までのClient IDを上書きしただけなので、これでいいと思ったんですが、どうでしょうか?

@ras0q
Copy link
Member

ras0q commented Dec 3, 2023

まあclient idは公開情報なのと(traQの認可エンドポイントに飛ぶときに見える)、今設定されてるやつが乱数なのかfujiさんとかが使ってたやつなのかわからないので変えてもいいと思います

強いて言えば本番環境じゃなくてステージング環境のclient idを使うと良さそうなんですがこっちはBot ConsoleがないのでAPIから直接取得する必要があるのと、デバッグ時にリクエストを飛ばす先をステージングのtraQにする必要があってちょっとめんどくさいのでまあ好きな方でいいです

@Luftalian
Copy link
Member Author

とりあえず

Copy link
Member

@ras0q ras0q left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(動作確認してないです) よさそうです!
手元で動いてるならマージしちゃってください

@Luftalian
Copy link
Member Author

動作確認してます
マージしまーじ

@Luftalian Luftalian merged commit aa368e8 into main Dec 3, 2023
4 checks passed
@Luftalian Luftalian deleted the fix/issue499 branch December 3, 2023 12:02
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

Successfully merging this pull request may close these issues.

compose.ymlのORIGINがhttp://localhost:6006になっている
2 participants