Skip to content

feat: Implement experiment gated CRUD for workspace proxies #6928

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Apr 4, 2023
Merged

feat: Implement experiment gated CRUD for workspace proxies #6928

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
243f837
feat: Implement basic moon crud
Emyrk Mar 30, 2023
5bd1392
chore: Implement enterprise endpoints for moons
Emyrk Mar 31, 2023
4b5e219
Make gen
Emyrk Mar 31, 2023
a593520
Add unit test for moons
Emyrk Mar 31, 2023
8613b13
Linting
Emyrk Mar 31, 2023
f1da2f1
Add url validation
Emyrk Mar 31, 2023
dff4fc8
Add unit tests
Emyrk Mar 31, 2023
aa41907
Make gen
Emyrk Mar 31, 2023
fa5ad31
Linting
Emyrk Mar 31, 2023
d4260b5
Fix unit test
Emyrk Mar 31, 2023
f689381
Fix unit test
Emyrk Mar 31, 2023
bace1cf
Make gen
Emyrk Apr 3, 2023
6ba9079
Merge remote-tracking branch 'origin/main' into stevenmasley/moon_crud
Emyrk Apr 3, 2023
a8a5be1
Bump migrations
Emyrk Apr 3, 2023
cce8a7d
wildcard hostname vs url
Emyrk Apr 4, 2023
d84b46c
Use tabs
Emyrk Apr 4, 2023
a1a31e4
Remove org scoped proxies
Emyrk Apr 4, 2023
440645c
Fix unit tests
Emyrk Apr 4, 2023
c7b6b7c
make fmt
Emyrk Apr 4, 2023
9824f2e
Fix route gen
Emyrk Apr 4, 2023
c7c52f3
Fix unit test
Emyrk Apr 4, 2023
0f3af22
Merge remote-tracking branch 'origin/main' into stevenmasley/moon_crud
Emyrk Apr 4, 2023
ac3f912
Merge remote-tracking branch 'origin/main' into stevenmasley/moon_crud
Emyrk Apr 4, 2023
3145d5f
Make gen
Emyrk Apr 4, 2023
8a5119e
Bump migrations
Emyrk Apr 4, 2023
7daf459
Only return non deleted
Emyrk Apr 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix unit test
  • Loading branch information
@Emyrk
Emyrk committed Mar 31, 2023
commit d4260b5300d5dd52d3f170a8017e4e0c7caff1c6
10 changes: 7 additions & 3 deletions coderd/database/dbauthz/querier_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -464,9 +464,13 @@ func (s *MethodTestSuite) TestWorkspaceProxy() {
}).Asserts(p, rbac.ActionDelete)
}))
s.Run("GetWorkspaceProxies", s.Subtest(func(db database.Store, check *expects) {
p1 := dbgen.WorkspaceProxy(s.T(), db, database.WorkspaceProxy{})
p2 := dbgen.WorkspaceProxy(s.T(), db, database.WorkspaceProxy{})
check.Args().Asserts(p1, rbac.ActionRead, p2, rbac.ActionRead).Returns(slice.New(p1, p2))
p1 := dbgen.WorkspaceProxy(s.T(), db, database.WorkspaceProxy{
OrganizationID: uuid.New(),
})
p2 := dbgen.WorkspaceProxy(s.T(), db, database.WorkspaceProxy{
OrganizationID: p1.OrganizationID,
})
check.Args(p1.OrganizationID).Asserts(p1, rbac.ActionRead, p2, rbac.ActionRead).Returns(slice.New(p1, p2))
}))
}

Expand Down
1 change: 1 addition & 0 deletions coderd/database/dbgen/generator.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,7 @@ func WorkspaceProxy(t testing.TB, db database.Store, orig database.WorkspaceProx
ID: takeFirst(orig.ID, uuid.New()),
OrganizationID: takeFirst(orig.OrganizationID, uuid.New()),
Name: takeFirst(orig.Name, namesgenerator.GetRandomName(1)),
DisplayName: takeFirst(orig.DisplayName, namesgenerator.GetRandomName(1)),
Icon: takeFirst(orig.Icon, namesgenerator.GetRandomName(1)),
Url: takeFirst(orig.Url, fmt.Sprintf("https://%s.com", namesgenerator.GetRandomName(1))),
WildcardUrl: takeFirst(orig.WildcardUrl, fmt.Sprintf("https://*.%s.com", namesgenerator.GetRandomName(1))),
Expand Down
2 changes: 1 addition & 1 deletion enterprise/coderd/workspaceproxy.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ func (api *API) postWorkspaceProxyByOrganization(rw http.ResponseWriter, r *http
httpapi.Write(ctx, rw, http.StatusCreated, convertProxy(proxy))
}

// nolint:flag-parameter
// nolint:revive
func validateProxyURL(u string, wildcard bool) error {
p, err := url.Parse(u)
if err != nil {
Expand Down