Skip to content

Commit 16c6b3d

Browse files
Merge pull request #10 from ibuildthecloud/master
chore: refactor types
2 parents f87ea82 + 7e3f198 commit 16c6b3d

File tree

7 files changed

+114
-177
lines changed

7 files changed

+114
-177
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Build stage
2-
FROM golang:1.21-alpine AS builder
2+
FROM golang:1.23-alpine AS builder
33

44
# Install build dependencies
55
RUN apk add --no-cache git ca-certificates tzdata

pkg/providers/generic.go

Lines changed: 6 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -9,25 +9,16 @@ import (
99
"os"
1010
"strings"
1111
"time"
12-
)
1312

14-
// OAuthMetadata represents OAuth authorization server metadata
15-
type OAuthMetadata struct {
16-
Issuer string `json:"issuer"`
17-
AuthorizationEndpoint string `json:"authorization_endpoint"`
18-
TokenEndpoint string `json:"token_endpoint"`
19-
UserinfoEndpoint string `json:"userinfo_endpoint,omitempty"`
20-
ScopesSupported []string `json:"scopes_supported,omitempty"`
21-
ResponseTypesSupported []string `json:"response_types_supported,omitempty"`
22-
GrantTypesSupported []string `json:"grant_types_supported,omitempty"`
23-
}
13+
"github.com/obot-platform/mcp-oauth-proxy/pkg/types"
14+
)
2415

2516
// GenericProvider implements a generic OAuth provider
2617
type GenericProvider struct {
2718
clientID string
2819
clientSecret string
2920
authorizeURL string
30-
metadata *OAuthMetadata
21+
metadata *types.OAuthMetadata
3122
httpClient *http.Client
3223
}
3324

@@ -87,7 +78,7 @@ func (p *GenericProvider) discoverEndpoints() error {
8778
}
8879

8980
// If no metadata found, create a basic metadata structure
90-
p.metadata = &OAuthMetadata{
81+
p.metadata = &types.OAuthMetadata{
9182
Issuer: baseURL,
9283
AuthorizationEndpoint: p.authorizeURL,
9384
TokenEndpoint: baseURL + "/token",
@@ -101,7 +92,7 @@ func (p *GenericProvider) discoverEndpoints() error {
10192
}
10293

10394
// fetchMetadata fetches OAuth metadata from a URL
104-
func (p *GenericProvider) fetchMetadata(metadataURL string) (*OAuthMetadata, error) {
95+
func (p *GenericProvider) fetchMetadata(metadataURL string) (*types.OAuthMetadata, error) {
10596
resp, err := p.httpClient.Get(metadataURL)
10697
if err != nil {
10798
return nil, err
@@ -117,7 +108,7 @@ func (p *GenericProvider) fetchMetadata(metadataURL string) (*OAuthMetadata, err
117108
return nil, fmt.Errorf("failed to fetch metadata: %s", resp.Status)
118109
}
119110

120-
var metadata OAuthMetadata
111+
var metadata types.OAuthMetadata
121112
if err := json.NewDecoder(resp.Body).Decode(&metadata); err != nil {
122113
return nil, fmt.Errorf("failed to decode metadata: %w", err)
123114
}

0 commit comments

Comments
 (0)