Skip to content

Commit

Permalink
fix: stop show credential threads to users
Browse files Browse the repository at this point in the history
The threads used to authorize agents and workflows should not be shown
to users. This change ensures that is the case.

Signed-off-by: Donnie Adams <[email protected]>
  • Loading branch information
thedadams committed Jan 7, 2025
1 parent 35572d2 commit f33f651
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions pkg/api/handlers/agent.go
Original file line number Diff line number Diff line change
Expand Up @@ -953,6 +953,8 @@ func runAuthForAgent(ctx context.Context, c kclient.WithWatch, invoker *invoke.I
agent.Spec.Manifest.AvailableThreadTools = nil
agent.Spec.Manifest.DefaultThreadTools = nil
agent.Spec.Credentials = credentials
agent.Spec.CredentialContextID = agent.Name
agent.Name = ""

return invoker.Agent(ctx, c, agent, "", invoke.Options{
Synchronous: true,
Expand Down
2 changes: 1 addition & 1 deletion pkg/controller/handlers/toolinfo/toolinfo.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ func (h *Handler) SetToolInfoStatus(req router.Request, resp router.Response) (e

// Get all the credentials that exist in the expected context.
creds, err := h.gptscript.ListCredentials(req.Ctx, gptscript.ListCredentialsOptions{
CredentialContexts: []string{req.Name},
CredentialContexts: []string{req.Name, req.Namespace},
})
if err != nil {
return err
Expand Down

0 comments on commit f33f651

Please sign in to comment.