Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commita2607a8

Browse files
committed
feat(oauth2): remove unique constraint on app names for RFC 7591 compliance
Change-Id: Iae7a1a06546fbc8de541a52e291f8a4510d57e8aSigned-off-by: Thomas Kosiewski <tk@coder.com>
1 parentf4614fd commita2607a8

File tree

6 files changed

+7
-27
lines changed

6 files changed

+7
-27
lines changed

‎coderd/database/dbmem/dbmem.go‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8971,12 +8971,6 @@ func (q *FakeQuerier) InsertOAuth2ProviderApp(_ context.Context, arg database.In
89718971
q.mutex.Lock()
89728972
deferq.mutex.Unlock()
89738973

8974-
for_,app:=rangeq.oauth2ProviderApps {
8975-
ifapp.Name==arg.Name {
8976-
return database.OAuth2ProviderApp{},errUniqueConstraint
8977-
}
8978-
}
8979-
89808974
//nolint:gosimple // Go wants database.OAuth2ProviderApp(arg), but we cannot be sure the structs will remain identical.
89818975
app:= database.OAuth2ProviderApp{
89828976
ID:arg.ID,

‎coderd/database/dump.sql‎

Lines changed: 0 additions & 3 deletions
Some generated files are not rendered by default. Learn more aboutcustomizing how changed files appear on GitHub.
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
-- Restore unique constraint on oauth2_provider_apps.name for rollback
2+
-- Note: This rollback may fail if duplicate names exist in the database
3+
ALTERTABLE oauth2_provider_apps ADDCONSTRAINT oauth2_provider_apps_name_key UNIQUE (name);
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
-- Remove unique constraint on oauth2_provider_apps.name to comply with RFC 7591
2+
-- RFC 7591 does not require unique client names, only unique client IDs
3+
ALTERTABLE oauth2_provider_apps DROPCONSTRAINT oauth2_provider_apps_name_key;

‎coderd/database/unique_constraint.go‎

Lines changed: 0 additions & 1 deletion
Some generated files are not rendered by default. Learn more aboutcustomizing how changed files appear on GitHub.

‎coderd/oauth2_test.go‎

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -64,13 +64,6 @@ func TestOAuth2ProviderApps(t *testing.T) {
6464
CallbackURL:"http://localhost:3000",
6565
},
6666
},
67-
{
68-
name:"NameTaken",
69-
req: codersdk.PostOAuth2ProviderAppRequest{
70-
Name:"taken",
71-
CallbackURL:"http://localhost:3000",
72-
},
73-
},
7467
{
7568
name:"URLMissing",
7669
req: codersdk.PostOAuth2ProviderAppRequest{
@@ -135,17 +128,8 @@ func TestOAuth2ProviderApps(t *testing.T) {
135128
},
136129
}
137130

138-
// Generate an application for testing name conflicts.
139-
req:= codersdk.PostOAuth2ProviderAppRequest{
140-
Name:"taken",
141-
CallbackURL:"http://coder.com",
142-
}
143-
//nolint:gocritic // OAauth2 app management requires owner permission.
144-
_,err:=client.PostOAuth2ProviderApp(ctx,req)
145-
require.NoError(t,err)
146-
147131
// Generate an application for testing PUTs.
148-
req= codersdk.PostOAuth2ProviderAppRequest{
132+
req:= codersdk.PostOAuth2ProviderAppRequest{
149133
Name:fmt.Sprintf("quark-%d",time.Now().UnixNano()%1000000),
150134
CallbackURL:"http://coder.com",
151135
}

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp