mirror of
https://github.com/oauth2-proxy/oauth2-proxy.git
synced 2026-10-03 21:24:33 +02:00
* fix: #3428 bybitbucket auth failure by changing token n validation flow to use Authentication header instead of query param Co-authored-by: aviralgarg05 <gargaviral99@gmail.com> Signed-off-by: Mohammad Hassan <m8fouad@gmail.com> Signed-off-by: Jan Larwig <jan.larwig@digits.schwarz> * fix: added changelog entry and added tests to verify bitbucket auth header fix Signed-off-by: Mohammad Hassan <m8fouad@gmail.com> * fix(bitbucket): aded support for limiting login for workspace members and handled deprecated team api and added test cases Signed-off-by: Mohammad Hassan <m8fouad@gmail.com> * docs(bitbucket): applied missing docs changes Signed-off-by: Jan Larwig <jan.larwig@digits.schwarz> --------- Signed-off-by: Mohammad Hassan <m8fouad@gmail.com> Signed-off-by: Jan Larwig <jan.larwig@digits.schwarz> Co-authored-by: aviralgarg05 <gargaviral99@gmail.com> Co-authored-by: Jan Larwig <jan.larwig@digits.schwarz>
293 lines
9.7 KiB
Go
293 lines
9.7 KiB
Go
package providers
|
|
|
|
import (
|
|
"context"
|
|
"log"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"net/url"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/oauth2-proxy/oauth2-proxy/v7/pkg/apis/options"
|
|
"github.com/oauth2-proxy/oauth2-proxy/v7/pkg/apis/sessions"
|
|
. "github.com/onsi/gomega"
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func testBitbucketProvider(hostname, team string, repository string) *BitbucketProvider {
|
|
p := NewBitbucketProvider(
|
|
&ProviderData{
|
|
ProviderName: "",
|
|
LoginURL: &url.URL{},
|
|
RedeemURL: &url.URL{},
|
|
ProfileURL: &url.URL{},
|
|
ValidateURL: &url.URL{},
|
|
Scope: ""},
|
|
options.BitbucketOptions{
|
|
Team: team,
|
|
Workspace: team,
|
|
Repository: repository,
|
|
},
|
|
)
|
|
|
|
if hostname != "" {
|
|
updateURL(p.Data().LoginURL, hostname)
|
|
updateURL(p.Data().RedeemURL, hostname)
|
|
updateURL(p.Data().ProfileURL, hostname)
|
|
updateURL(p.Data().ValidateURL, hostname)
|
|
}
|
|
return p
|
|
}
|
|
|
|
func testBitbucketBackend(payload string) *httptest.Server {
|
|
paths := map[string]bool{
|
|
"/2.0/user/emails": true,
|
|
}
|
|
|
|
return httptest.NewServer(http.HandlerFunc(
|
|
func(w http.ResponseWriter, r *http.Request) {
|
|
url := r.URL
|
|
if !paths[url.Path] {
|
|
log.Printf("%s not in %+v\n", url.Path, paths)
|
|
w.WriteHeader(404)
|
|
} else if !IsAuthorizedInHeader(r.Header) {
|
|
w.WriteHeader(403)
|
|
} else {
|
|
w.WriteHeader(200)
|
|
w.Write([]byte(payload))
|
|
}
|
|
}))
|
|
}
|
|
|
|
func testBitbucketBackendWithWorkspace(emailPayload, workspacePayload string) *httptest.Server {
|
|
return testBitbucketBackendFull(emailPayload, workspacePayload, "")
|
|
}
|
|
|
|
func testBitbucketBackendFull(emailPayload, workspacePayload, repoPayload string) *httptest.Server {
|
|
return httptest.NewServer(http.HandlerFunc(
|
|
func(w http.ResponseWriter, r *http.Request) {
|
|
if !IsAuthorizedInHeader(r.Header) {
|
|
w.WriteHeader(403)
|
|
return
|
|
}
|
|
switch {
|
|
case r.URL.Path == "/2.0/user/emails":
|
|
w.WriteHeader(200)
|
|
w.Write([]byte(emailPayload))
|
|
case r.URL.Path == "/2.0/user/workspaces":
|
|
w.WriteHeader(200)
|
|
w.Write([]byte(workspacePayload))
|
|
case strings.HasPrefix(r.URL.Path, "/2.0/repositories/"):
|
|
w.WriteHeader(200)
|
|
w.Write([]byte(repoPayload))
|
|
default:
|
|
log.Printf("%s not handled\n", r.URL.Path)
|
|
w.WriteHeader(404)
|
|
}
|
|
}))
|
|
}
|
|
|
|
func TestNewBitbucketProvider(t *testing.T) {
|
|
g := NewWithT(t)
|
|
|
|
// Test that defaults are set when calling for a new provider with nothing set
|
|
providerData := NewBitbucketProvider(&ProviderData{}, options.BitbucketOptions{}).Data()
|
|
g.Expect(providerData.ProviderName).To(Equal("Bitbucket"))
|
|
g.Expect(providerData.LoginURL.String()).To(Equal("https://bitbucket.org/site/oauth2/authorize"))
|
|
g.Expect(providerData.RedeemURL.String()).To(Equal("https://bitbucket.org/site/oauth2/access_token"))
|
|
g.Expect(providerData.ProfileURL.String()).To(Equal(""))
|
|
g.Expect(providerData.ValidateURL.String()).To(Equal("https://api.bitbucket.org/2.0/user/emails"))
|
|
g.Expect(providerData.Scope).To(Equal("email"))
|
|
}
|
|
|
|
func TestBitbucketProviderScopeAdjustForTeam(t *testing.T) {
|
|
p := testBitbucketProvider("", "test-team", "")
|
|
assert.NotEqual(t, nil, p)
|
|
assert.Equal(t, "email account", p.Data().Scope)
|
|
}
|
|
|
|
func TestBitbucketProviderScopeAdjustForRepository(t *testing.T) {
|
|
p := testBitbucketProvider("", "", "rest-repo")
|
|
assert.NotEqual(t, nil, p)
|
|
assert.Equal(t, "email repository", p.Data().Scope)
|
|
}
|
|
|
|
func TestBitbucketProviderOverrides(t *testing.T) {
|
|
p := NewBitbucketProvider(
|
|
&ProviderData{
|
|
LoginURL: &url.URL{
|
|
Scheme: "https",
|
|
Host: "example.com",
|
|
Path: "/oauth/auth"},
|
|
RedeemURL: &url.URL{
|
|
Scheme: "https",
|
|
Host: "example.com",
|
|
Path: "/oauth/token"},
|
|
ValidateURL: &url.URL{
|
|
Scheme: "https",
|
|
Host: "example.com",
|
|
Path: "/api/v3/user"},
|
|
Scope: "profile"},
|
|
options.BitbucketOptions{})
|
|
assert.NotEqual(t, nil, p)
|
|
assert.Equal(t, "Bitbucket", p.Data().ProviderName)
|
|
assert.Equal(t, "https://example.com/oauth/auth",
|
|
p.Data().LoginURL.String())
|
|
assert.Equal(t, "https://example.com/oauth/token",
|
|
p.Data().RedeemURL.String())
|
|
assert.Equal(t, "https://example.com/api/v3/user",
|
|
p.Data().ValidateURL.String())
|
|
assert.Equal(t, "profile", p.Data().Scope)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddress(t *testing.T) {
|
|
b := testBitbucketBackend("{\"values\": [ { \"email\": \"michael.bland@gsa.gov\", \"is_primary\": true } ] }")
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "michael.bland@gsa.gov", email)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddressAndGroup(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
workspacePayload := `{"values": [ { "type": "workspace_access", "workspace": { "slug": "bioinformatics" } } ] }`
|
|
b := testBitbucketBackendWithWorkspace(emailPayload, workspacePayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "bioinformatics", "")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "michael.bland@gsa.gov", email)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddressMultipleWorkspaces(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
// target workspace is the last entry to exercise the full loop
|
|
workspacePayload := `{"values": [
|
|
{ "type": "workspace_access", "workspace": { "slug": "other-team-1" } },
|
|
{ "type": "workspace_access", "workspace": { "slug": "other-team-2" } },
|
|
{ "type": "workspace_access", "workspace": { "slug": "bioinformatics" } }
|
|
]}`
|
|
b := testBitbucketBackendWithWorkspace(emailPayload, workspacePayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "bioinformatics", "")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "michael.bland@gsa.gov", email)
|
|
}
|
|
|
|
func TestBitbucketProviderWorkspaceNotInList(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
workspacePayload := `{"values": [
|
|
{ "type": "workspace_access", "workspace": { "slug": "other-team-1" } },
|
|
{ "type": "workspace_access", "workspace": { "slug": "other-team-2" } }
|
|
]}`
|
|
b := testBitbucketBackendWithWorkspace(emailPayload, workspacePayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "bioinformatics", "")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "", email)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddressAndRepository(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
repoPayload := `{"values": [ { "full_name": "bioinformatics/myrepo" } ] }`
|
|
b := testBitbucketBackendFull(emailPayload, "", repoPayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "bioinformatics/myrepo")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "michael.bland@gsa.gov", email)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddressMultipleRepositories(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
// target repo is the last entry to exercise the full loop
|
|
repoPayload := `{"values": [
|
|
{ "full_name": "bioinformatics/other-repo-1" },
|
|
{ "full_name": "bioinformatics/other-repo-2" },
|
|
{ "full_name": "bioinformatics/myrepo" }
|
|
]}`
|
|
b := testBitbucketBackendFull(emailPayload, "", repoPayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "bioinformatics/myrepo")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "michael.bland@gsa.gov", email)
|
|
}
|
|
|
|
func TestBitbucketProviderRepositoryNotInList(t *testing.T) {
|
|
emailPayload := `{"values": [ { "email": "michael.bland@gsa.gov", "is_primary": true } ] }`
|
|
repoPayload := `{"values": [
|
|
{ "full_name": "bioinformatics/other-repo-1" },
|
|
{ "full_name": "bioinformatics/other-repo-2" }
|
|
]}`
|
|
b := testBitbucketBackendFull(emailPayload, "", repoPayload)
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "bioinformatics/myrepo")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, nil, err)
|
|
assert.Equal(t, "", email)
|
|
}
|
|
|
|
// Note that trying to trigger the "failed building request" case is not
|
|
// practical, since the only way it can fail is if the URL fails to parse.
|
|
func TestBitbucketProviderGetEmailAddressFailedRequest(t *testing.T) {
|
|
b := testBitbucketBackend("unused payload")
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "")
|
|
|
|
// We'll trigger a request failure by using an unexpected access
|
|
// token. Alternatively, we could allow the parsing of the payload as
|
|
// JSON to fail.
|
|
session := &sessions.SessionState{AccessToken: "unexpected_access_token"}
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.NotEqual(t, nil, err)
|
|
assert.Equal(t, "", email)
|
|
}
|
|
|
|
func TestBitbucketProviderGetEmailAddressEmailNotPresentInPayload(t *testing.T) {
|
|
b := testBitbucketBackend("{\"foo\": \"bar\"}")
|
|
defer b.Close()
|
|
|
|
bURL, _ := url.Parse(b.URL)
|
|
p := testBitbucketProvider(bURL.Host, "", "")
|
|
|
|
session := CreateAuthorizedSession()
|
|
email, err := p.GetEmailAddress(context.Background(), session)
|
|
assert.Equal(t, "", email)
|
|
assert.Equal(t, nil, err)
|
|
}
|