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

Commit50943c1

Browse files
committed
review improvements
Change-Id: I7868299308cdf4fca36092025bf2692cfe652640Signed-off-by: Thomas Kosiewski <tk@coder.com>
1 parentb5ed0bd commit50943c1

File tree

14 files changed

+67
-76
lines changed

14 files changed

+67
-76
lines changed

‎coderd/coderd.go‎

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -804,11 +804,6 @@ func New(options *Options) *API {
804804
DB:options.Database,
805805
Optional:false,
806806
})
807-
// Same as above but optional
808-
workspaceAgentInfoOptional:=httpmw.ExtractWorkspaceAgentAndLatestBuild(httpmw.ExtractWorkspaceAgentAndLatestBuildConfig{
809-
DB:options.Database,
810-
Optional:true,
811-
})
812807

813808
// API rate limit middleware. The counter is local and not shared between
814809
// replicas or instances of this middleware.
@@ -1029,7 +1024,6 @@ func New(options *Options) *API {
10291024
r.Route("/external-auth",func(r chi.Router) {
10301025
r.Use(
10311026
apiKeyMiddleware,
1032-
workspaceAgentInfoOptional,
10331027
)
10341028
// Get without a specific external auth ID will return all external auths.
10351029
r.Get("/",api.listUserExternalAuths)
@@ -1265,12 +1259,8 @@ func New(options *Options) *API {
12651259
r.Get("/",api.workspaceByOwnerAndName)
12661260
r.Get("/builds/{buildnumber}",api.workspaceBuildByBuildNumber)
12671261
})
1268-
r.With(
1269-
workspaceAgentInfoOptional,
1270-
).Get("/gitsshkey",api.gitSSHKey)
1271-
r.With(
1272-
workspaceAgentInfoOptional,
1273-
).Put("/gitsshkey",api.regenerateGitSSHKey)
1262+
r.Get("/gitsshkey",api.gitSSHKey)
1263+
r.Put("/gitsshkey",api.regenerateGitSSHKey)
12741264
r.Route("/notifications",func(r chi.Router) {
12751265
r.Route("/preferences",func(r chi.Router) {
12761266
r.Get("/",api.userNotificationPreferences)

‎coderd/database/dbauthz/dbauthz_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3988,7 +3988,7 @@ func (s *MethodTestSuite) TestSystemFunctions() {
39883988
check.Args(database.InsertWorkspaceAgentParams{
39893989
ID:uuid.New(),
39903990
Name:"dev",
3991-
APIKeyScope:database.ApiKeyScopeEnumDefault,
3991+
APIKeyScope:database.AgentKeyScopeEnumAll,
39923992
}).Asserts(rbac.ResourceSystem,policy.ActionCreate)
39933993
}))
39943994
s.Run("InsertWorkspaceApp",s.Subtest(func(db database.Store,check*expects) {

‎coderd/database/dbgen/dbgen.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ func WorkspaceAgent(t testing.TB, db database.Store, orig database.WorkspaceAgen
210210
MOTDFile:takeFirst(orig.TroubleshootingURL,""),
211211
DisplayApps:append([]database.DisplayApp{},orig.DisplayApps...),
212212
DisplayOrder:takeFirst(orig.DisplayOrder,1),
213-
APIKeyScope:takeFirst(orig.APIKeyScope,database.ApiKeyScopeEnumDefault),
213+
APIKeyScope:takeFirst(orig.APIKeyScope,database.AgentKeyScopeEnumAll),
214214
})
215215
require.NoError(t,err,"insert workspace agent")
216216
returnagt

‎coderd/database/dump.sql‎

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

‎coderd/database/migrations/000320_add_api_key_scope_to_workspace_agents.down.sql‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ ALTER TABLE workspace_agents
33
DROP COLUMN IF EXISTS api_key_scope;
44

55
-- Drop the enum type for API key scope
6-
DROPTYPE IF EXISTSapi_key_scope_enum;
6+
DROPTYPE IF EXISTSagent_key_scope_enum;
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
-- Create the enum type for API key scope
2-
CREATETYPEapi_key_scope_enumAS ENUM ('default','no_user_data');
2+
CREATETYPEagent_key_scope_enumAS ENUM ('all','no_user_data');
33

44
-- Add the api_key_scope column to the workspace_agents table
5-
-- It defaults to 'default' to maintain existing behavior for current agents.
5+
-- It defaults to 'all' to maintain existing behavior for current agents.
66
ALTERTABLE workspace_agents
7-
ADD COLUMN api_key_scopeapi_key_scope_enumNOT NULL DEFAULT'default';
7+
ADD COLUMN api_key_scopeagent_key_scope_enumNOT NULL DEFAULT'all';
88

99
-- Add a comment explaining the purpose of the column
10-
COMMENT ON COLUMN workspace_agents.api_key_scope IS'Defines the scope of the API key associated with the agent.''default'' allows access to everything,''no_user_data'' restricts it to exclude user data.';
10+
COMMENT ON COLUMN workspace_agents.api_key_scope IS'Defines the scope of the API key associated with the agent.''all'' allows access to everything,''no_user_data'' restricts it to exclude user data.';

‎coderd/database/models.go‎

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

‎coderd/database/queries.sql.go‎

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

‎coderd/externalauth_test.go‎

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -713,7 +713,7 @@ func TestExternalAuthCallback(t *testing.T) {
713713
apiKeyScopestring
714714
expectsErrorbool
715715
}{
716-
{apiKeyScope:"default",expectsError:false},
716+
{apiKeyScope:"all",expectsError:false},
717717
{apiKeyScope:"no_user_data",expectsError:true},
718718
} {
719719
t.Run(tt.apiKeyScope,func(t*testing.T) {
@@ -746,6 +746,15 @@ func TestExternalAuthCallback(t *testing.T) {
746746
token,err:=agentClient.ExternalAuth(t.Context(), agentsdk.ExternalAuthRequest{
747747
Match:"github.com/asd/asd",
748748
})
749+
750+
iftt.expectsError {
751+
require.Error(t,err)
752+
varsdkErr*codersdk.Error
753+
require.ErrorAs(t,err,&sdkErr)
754+
require.Equal(t,http.StatusForbidden,sdkErr.StatusCode())
755+
return
756+
}
757+
749758
require.NoError(t,err)
750759
require.NotEmpty(t,token.URL)
751760

@@ -756,13 +765,8 @@ func TestExternalAuthCallback(t *testing.T) {
756765
Match:"github.com/asd/asd",
757766
Listen:true,
758767
})
759-
iftt.expectsError {
760-
assert.Error(t,err)
761-
close(tokenChan)
762-
}else {
763-
assert.NoError(t,err)
764-
tokenChan<-token
765-
}
768+
assert.NoError(t,err)
769+
tokenChan<-token
766770
}()
767771

768772
time.Sleep(250*time.Millisecond)
@@ -771,9 +775,6 @@ func TestExternalAuthCallback(t *testing.T) {
771775
require.Equal(t,http.StatusTemporaryRedirect,resp.StatusCode)
772776

773777
token=<-tokenChan
774-
iftt.expectsError {
775-
return
776-
}
777778
require.Equal(t,"access_token",token.Username)
778779

779780
token,err=agentClient.ExternalAuth(t.Context(), agentsdk.ExternalAuthRequest{

‎coderd/gitsshkey_test.go‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package coderd_test
22

33
import (
44
"context"
5+
"net/http"
56
"testing"
67

78
"github.com/google/uuid"
@@ -12,6 +13,7 @@ import (
1213
"github.com/coder/coder/v2/coderd/coderdtest"
1314
"github.com/coder/coder/v2/coderd/database"
1415
"github.com/coder/coder/v2/coderd/gitsshkey"
16+
"github.com/coder/coder/v2/codersdk"
1517
"github.com/coder/coder/v2/codersdk/agentsdk"
1618
"github.com/coder/coder/v2/provisioner/echo"
1719
"github.com/coder/coder/v2/testutil"
@@ -134,7 +136,7 @@ func TestAgentGitSSHKey_APIKeyScopes(t *testing.T) {
134136
apiKeyScopestring
135137
expectErrorbool
136138
}{
137-
{apiKeyScope:"default",expectError:false},
139+
{apiKeyScope:"all",expectError:false},
138140
{apiKeyScope:"no_user_data",expectError:true},
139141
} {
140142
t.Run(tt.apiKeyScope,func(t*testing.T) {
@@ -165,6 +167,9 @@ func TestAgentGitSSHKey_APIKeyScopes(t *testing.T) {
165167

166168
iftt.expectError {
167169
require.Error(t,err)
170+
varsdkErr*codersdk.Error
171+
require.ErrorAs(t,err,&sdkErr)
172+
require.Equal(t,http.StatusForbidden,sdkErr.StatusCode())
168173
}else {
169174
require.NoError(t,err)
170175
}

0 commit comments

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