認証を Signin with Slack に変える?
Opened this issue · 4 comments
5t111111 commented
omniauth-slack に signin with slack の承認フローを追加する PR がマージされたので GitHub の HEAD を使えば identity.basic
などのスコープを使うことができます。
よいこと
- アバター画像の URL 取れる
- ログイン時の Authorize のときの画面遷移が1つ減る
- Slack 的にはこれが正攻法?
悪いこと
- 結構な breaking changes で uid が変わる (でもこれは一意性を担保する点ではよい?)
アカウント名が取れない (でもこれはストラテジ側の問題かもなので raw response を見てみないと)- なんかまた変わりそう
5t111111 commented
アカウント名が取れない (でもこれはストラテジ側の問題かもなので raw response を見てみないと)
raw response みたら identity の scope 指定しててもそのユーザーの user_info は取ってこれてて、よく見たらストラテジ側でも raw_info に突っ込んでくれてるようでした。
つまり深いところにあるけど auth.dig(:extra, :raw_info, :user_info, :user, :name)
でアカウント名もちゃんと取れます。
yucao24hours commented
💭全然どうでもいいことだけど、 auth
は Hashie::Mash
の子どもにあたるから auth.info.nickname
みたいにドットでアクセスできるって、死んだばあちゃんが言ってました。(お好みで)
yucao24hours commented
ここに書かなくてもよかった