vim-jp / slacklog-generator

Archive generator for Slack
https://vim-jp.org/slacklog/
Creative Commons Attribution 4.0 International
26 stars 9 forks source link

Fix detection way of auth error while downloading #93

Closed thinca closed 4 years ago

thinca commented 4 years ago

ファイルのダウンロード時、token が無効等の理由で認証が通っていない場合にログインページ(200)にリダイレクトされてしまうため、リダイレクト = エラーとして扱っていたが、なぜか .svg ファイルをダウンロードする際にも正常なリダイレクトが発生することが判明しました。

参考: https://gist.github.com/thinca/c2156a848f2038fd8f250e7d3e3894da

仕方ないので、リダイレクト先の URL を見て、たぶん認証ページに飛んでそう、というやつだけをエラーにするようにしてみました。サンプルが少ないのでこれで正しく判別できるかは不明ですが、ひとまず現段階のログの範囲では正しく処理できていそうです。