Skip to content

fix: Fix properly selecting workspace apps by agent #3684

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 12 commits into from
Aug 29, 2022
Merged

fix: Fix properly selecting workspace apps by agent #3684

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
433705d
fix: Fix properly naming workspace agents
Emyrk Aug 24, 2022
7d0bc56
chore: Make workspace and agent extract a middleware
Emyrk Aug 24, 2022
0bb51b4
chore: Write unit tests for agent selector
Emyrk Aug 24, 2022
8b3f46d
Add unit test cases
Emyrk Aug 24, 2022
f44f5c0
Linting
Emyrk Aug 24, 2022
712906e
Fix authorize unit test
Emyrk Aug 24, 2022
c5bacc6
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 25, 2022
4058665
Add test cases
Emyrk Aug 25, 2022
5401e70
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 25, 2022
48ed5db
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 26, 2022
991448d
Fix auth test on all http method types
Emyrk Aug 26, 2022
2a30b23
Rename url variable
Emyrk Aug 29, 2022
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
Next Next commit
fix: Fix properly naming workspace agents
  • Loading branch information
@Emyrk
Emyrk committed Aug 24, 2022
commit 433705d03fc57c3808b96f4ed167f865d96ca76c
1 change: 1 addition & 0 deletions coderd/httpmw/chiparams.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
package httpmw
23 changes: 20 additions & 3 deletions coderd/workspaceapps.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,14 +85,31 @@ func (api *API) workspaceAppsProxyPath(rw http.ResponseWriter, r *http.Request)
return
}

agent := agents[0]
if len(workspaceParts) > 1 {
// If we have more than 1 workspace agent, we need to specify which one to use.
if len(agents) > 1 && len(workspaceParts) <= 1 {
httpapi.Write(rw, http.StatusBadRequest, codersdk.Response{
Message: "More than one agent exists, but no agent specified.",
})
return
}

// If we have more than 1 workspace agent, we need to specify which one to use.
var agent *database.WorkspaceAgent
if len(agents) > 1 {
for _, otherAgent := range agents {
if otherAgent.Name == workspaceParts[1] {
agent = otherAgent
agent = &otherAgent
break
}
}
if agent == nil {
httpapi.Write(rw, http.StatusBadRequest, codersdk.Response{
Message: fmt.Sprintf("No agent exists with the name %s", workspaceParts[1]),
})
return
}
} else {
agent = &agents[0]
}

app, err := api.Database.GetWorkspaceAppByAgentIDAndName(r.Context(), database.GetWorkspaceAppByAgentIDAndNameParams{
Expand Down