Skip to content

Commit

Permalink
Merge branch 'main' into fe-oauth-impl
Browse files Browse the repository at this point in the history
  • Loading branch information
Alder Whiteford authored and Alder Whiteford committed Jun 18, 2024
2 parents ddc91c6 + 401eac8 commit 79e2a66
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 16 deletions.
9 changes: 1 addition & 8 deletions backend/config/oauth_microsoft.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ type MicrosoftOAuthSettings struct {
}

type intermediateMicrosoftOAuthSetting struct {
Key string `env:"KEY"`
Secret string `env:"SECRET"`
Key string `env:"KEY"`
}

func (i *intermediateMicrosoftOAuthSetting) into() (*MicrosoftOAuthSettings, error) {
Expand All @@ -23,14 +22,8 @@ func (i *intermediateMicrosoftOAuthSetting) into() (*MicrosoftOAuthSettings, err
return nil, err
}

secretSecret, err := m.NewSecret(i.Secret)
if err != nil {
return nil, err
}

return &MicrosoftOAuthSettings{
Key: secretKey,
Secret: secretSecret,
Tenant: tenantID,
}, nil
}
9 changes: 3 additions & 6 deletions backend/integrations/oauth/soth/msft/msft.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,9 @@ var defaultScopes = []string{"openid", "offline_access", "user.read", "calendars
// New creates a new microsoftonline Provider, and sets up important connection details.
// You should always call `msft.New` to get a new Provider. Never try to create
// one manually.
func New(clientKey *m.Secret[string], secret *m.Secret[string], callbackURL string, tenant string, scopes ...string) *Provider {
func New(clientKey *m.Secret[string], callbackURL string, tenant string, scopes ...string) *Provider {
p := &Provider{
ClientKey: clientKey,
Secret: secret,
CallbackURL: callbackURL,
ProviderName: "microsoftonline",
tenant: tenant,
Expand All @@ -47,7 +46,6 @@ func New(clientKey *m.Secret[string], secret *m.Secret[string], callbackURL stri
// Provider is the implementation of `soth.Provider` for accessing microsoftonline.
type Provider struct {
ClientKey *m.Secret[string]
Secret *m.Secret[string]
CallbackURL string
config *oauth2.Config
ProviderName string
Expand Down Expand Up @@ -138,9 +136,8 @@ func newConfig(provider *Provider, scopes []string) *oauth2.Config {
}

c := &oauth2.Config{
ClientID: provider.ClientKey.Expose(),
ClientSecret: provider.Secret.Expose(),
RedirectURL: provider.CallbackURL,
ClientID: provider.ClientKey.Expose(),
RedirectURL: provider.CallbackURL,
Endpoint: oauth2.Endpoint{
AuthURL: authURL,
TokenURL: tokenURL,
Expand Down
2 changes: 1 addition & 1 deletion backend/server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ func Init(db *gorm.DB, stores *store.Stores, integrations integrations.Integrati

applicationURL := settings.Application.ApplicationURL()

msftProvider := msft.New(settings.Microsft.Key, settings.Microsft.Secret, "myapp://auth/callback", settings.Microsft.Tenant)
msftProvider := msft.New(settings.Microsft.Key, "myapp://auth/callback", settings.Microsft.Tenant)
googProvider := goog.New(settings.Google.Key, settings.Google.Secret, fmt.Sprintf("%s/api/v1/auth/google/callback", applicationURL))

authMiddleware := authMiddleware.New(
Expand Down
1 change: 0 additions & 1 deletion config/.env.template
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,5 @@ SAC_GOOGLE_OAUTH_SECRET=GOOGLE_OAUTH_CLIENT_SECRET
SAC_GOOGLE_API_KEY=GOOGLE_API_KEY

SAC_MICROSOFT_OAUTH_KEY=test
SAC_MICROSOFT_OAUTH_SECRET=test

SAC_SEARCH_URI="http://127.0.0.1:9200"

0 comments on commit 79e2a66

Please sign in to comment.