Netflix / hubcommander

A Slack bot for GitHub organization management -- and other things too
Apache License 2.0
1.28k stars 156 forks source link

Duo support for multiple keys #77

Closed mikegrima closed 6 years ago

mikegrima commented 6 years ago

The DUO plugin needs to better support multiple keys depending on the domain.

For example, users with @somedomain.com should be authenticated separately from users that may be members of @someotherdomain.com.

Right now, there is 1 set of credentials that are passed in. This should be easy to fix by making the credentials a dict for the domain. Then, it should check the domain a user is a member of to pick the correct credentials.

mikegrima commented 6 years ago

Implemented in #80