Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Add Slack Provider #608

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add Slack Provider #608

wants to merge 2 commits into from

Conversation

hensur
Copy link

@hensur hensur commented May 31, 2018

slack can return the user's team id and email address.

To restrict authentication to a slack team, I added the slack-team settings/cmdline option.

Slack authentication can be restricted by email address and team id
@martin-loetzsch
Copy link

This works quite well for us and as soon as there is an official fork (#628), then I'd love to see it integrated.

The older https://github.com/tappleby/slack_auth_proxy did not work for us, because it doesn't support the nginx auth_request directive.

This is how we install it: first install from master branch, then check out the PR and then install it again:

mkdir /path/to/oauth2_proxy
GOPATH=/path/to/oauth2_proxy go get github.com/bitly/oauth2_proxy
cd /path/to/oauth2_proxy/src/github.com/bitly/oauth2_proxy; git fetch origin pull/608/head:slack-auth-provider; git checkout slack-auth-provider
GOPATH=/path/to/oauth2_proxy go get github.com/bitly/oauth2_proxy

Thanks @hensur for creating this PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants