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

認証を Signin with Slack に変える? #50

Open
5t111111 opened this issue Jun 14, 2016 · 4 comments
Open

認証を Signin with Slack に変える? #50

5t111111 opened this issue Jun 14, 2016 · 4 comments

Comments

@5t111111
Copy link
Collaborator

5t111111 commented Jun 14, 2016

omniauth-slack に signin with slack の承認フローを追加する PR がマージされたので GitHub の HEAD を使えば identity.basic などのスコープを使うことができます。

よいこと

  • アバター画像の URL 取れる
  • ログイン時の Authorize のときの画面遷移が1つ減る
  • Slack 的にはこれが正攻法?

悪いこと

  • 結構な breaking changes で uid が変わる (でもこれは一意性を担保する点ではよい?)
  • アカウント名が取れない (でもこれはストラテジ側の問題かもなので raw response を見てみないと)
  • なんかまた変わりそう
@5t111111
Copy link
Collaborator Author

WIP #51

@5t111111
Copy link
Collaborator Author

アカウント名が取れない (でもこれはストラテジ側の問題かもなので raw response を見てみないと)

raw response みたら identity の scope 指定しててもそのユーザーの user_info は取ってこれてて、よく見たらストラテジ側でも raw_info に突っ込んでくれてるようでした。

つまり深いところにあるけど auth.dig(:extra, :raw_info, :user_info, :user, :name) でアカウント名もちゃんと取れます。

@yucao24hours
Copy link
Member

💭全然どうでもいいことだけど、 authHashie::Mash の子どもにあたるから auth.info.nickname みたいにドットでアクセスできるって、死んだばあちゃんが言ってました。(お好みで)

@yucao24hours
Copy link
Member

ここに書かなくてもよかった

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

2 participants