Actions
バグ #1379
closed
mastodonの認証トークンがデバッグ出力で見えてしまっている
プラグイン名:
mastodon
クラッシュする:
No
Description
mikutterを--debugで起動すると，以下の例のようなデバッグログが流れます．
notice: {MIKUTTER_DIR}/core/plugin/mastodon/api.rb:181:in `send_request': Mastodon::API.call get https://mstdn.maud.io/api/v1/timelines/home [["Authorization", "Bearer XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"]] [["limit", 40]]
デバッグログに認証トークンが含まれるのは適切ではないと考え，チケットを作成しました．以下の場合に問題になり得ます．
- バグの情報を得るために，mikutterの利用者にデバッグ出力をWebにアップロードするようにお願いした場合
- デバッグログを表示した画面のスクリーンショットをうっかりWebにアップロードしてしまった場合
再現手順
- mikutterを
--debugで起動 - mastodon worldを追加
- mastodonのホームタイムラインを抽出タブで表示する
Updated by Shibafu Midorino almost 5 years ago
ある程度上手く動作できている現状では、APIリクエスト毎にデバッグ用のログを出力しなくても良いように思います。
noticeメソッドの呼出自体を削除 or コメントアウトして良いのでは？
Updated by あひる 家鴨 over 4 years ago
- Status changed from 実装待ち to レビュー待ち
- ブランチ set to topic/1379-remove-header-from-notice
ヘッダをログから削除してみました。
Updated by toshi_a 初音 over 4 years ago
- Status changed from レビュー待ち to まだダメ
通信を全てログに表示するのは意味がないので、noticeの呼び出し自体を消してしまおうと思っていました。
Updated by あひる 家鴨 over 4 years ago
- Status changed from まだダメ to レビュー待ち
Updated by あひる 家鴨 over 4 years ago
- Status changed from マージ待ち to 終了
- Target version changed from 3.9 to 4.0
これもdevelopからブランチを生やしてしまっていたので、masterからブランチを作り直した上でcherry-pickしてマージしました。
Actions