Skip to content

Commit c5935ce

Browse files
committed
fix[SSO]: manual tests finalized for cloud
1 parent d554ea5 commit c5935ce

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

structs.go

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -692,7 +692,7 @@ type SSOInfo struct {
692692
Sub string `json:"sub"`
693693
OrgID string `json:"org_id"`
694694
ClientID string `json:"client_id"`
695-
CodeVerifier string `json:"code_verifier"` // PKCE code verifier
695+
CodeVerifier string `json:"code_verifier"` // PKCE code verifier
696696
ChallengeExpiry time.Time `json:"challenge_expiry"`
697697
}
698698

@@ -3091,9 +3091,11 @@ type SSOConfig struct {
30913091
OpenIdAuthorization string `json:"openid_authorization" datastore:"openid_authorization"`
30923092
OpenIdToken string `json:"openid_token" datastore:"openid_token"`
30933093
SSORequired bool `json:"SSORequired" datastore:"SSORequired"`
3094-
AutoProvision bool `json:"auto_provision" datastore:"auto_provision"`
3095-
RoleRequired bool `json:"role_required" datastore:"role_required"`
3096-
SkipSSOForAdmins bool `json:"skip_sso_for_admins" datastore:"skip_sso_for_admins"`
3094+
// i have a distinct hatered for this name.
3095+
// says "auto_provision" but all logic treats it as disable_auto_provision.
3096+
AutoProvision bool `json:"auto_provision" datastore:"auto_provision"`
3097+
RoleRequired bool `json:"role_required" datastore:"role_required"`
3098+
SkipSSOForAdmins bool `json:"skip_sso_for_admins" datastore:"skip_sso_for_admins"`
30973099
}
30983100

30993101
type SamlRequest struct {
@@ -4579,15 +4581,15 @@ type AgentOutput struct {
45794581
// For easy testing
45804582
DecisionString string `json:"decision_string,omitempty" datastore:"decision_string"`
45814583
// For tracking of details parent<->child
4582-
StartedAt int64 `json:"started_at,omitempty" datastore:"started_at"`
4583-
CompletedAt int64 `json:"completed_at,omitempty" datastore:"completed_at"`
4584-
ExecutionId string `json:"execution_id,omitempty" datastore:"execution_id"`
4585-
NodeId string `json:"node_id,omitempty" datastore:"node_id"`
4586-
Memory string `json:"memory,omitempty" datastore:"memory"`
4587-
Input string `json:"input" datastore:"input"`
4588-
OriginalInput string `json:"original_input,omitempty" datastore:"original_input"`
4584+
StartedAt int64 `json:"started_at,omitempty" datastore:"started_at"`
4585+
CompletedAt int64 `json:"completed_at,omitempty" datastore:"completed_at"`
4586+
ExecutionId string `json:"execution_id,omitempty" datastore:"execution_id"`
4587+
NodeId string `json:"node_id,omitempty" datastore:"node_id"`
4588+
Memory string `json:"memory,omitempty" datastore:"memory"`
4589+
Input string `json:"input" datastore:"input"`
4590+
OriginalInput string `json:"original_input,omitempty" datastore:"original_input"`
45894591
AllowedActions []string `json:"allowed_actions,omitempty" datastore:"allowed_actions"`
4590-
Output string `json:"output,omitempty" datastore:"output"`
4592+
Output string `json:"output,omitempty" datastore:"output"`
45914593
}
45924594

45934595
type HTTPWrapper struct {

0 commit comments

Comments
 (0)