Skip to content

Generate ScanAgent token with aictl#27

Draft
sliusardanya wants to merge 7 commits into
mainfrom
feature/agent-token-generate
Draft

Generate ScanAgent token with aictl#27
sliusardanya wants to merge 7 commits into
mainfrom
feature/agent-token-generate

Conversation

@sliusardanya

Copy link
Copy Markdown
Collaborator

Добавлен единственный возможный сейчас вариант получения API токена для ScanAgent - через авторизацию по username / password.

Поэтому реализовано вне рамок стандартного формата работы aictl через ctx и с переопределением конфигурации.

Протестировано командой:

go run ./cmd/run/main.go create agent-token test-agent-cli --login 'xxxx' --password 'xxxxx' -u https://some.aie.com --tls-skip

@sliusardanya sliusardanya marked this pull request as draft December 29, 2025 11:29
@sliusardanya sliusardanya force-pushed the feature/agent-token-generate branch from 45c0026 to 0579eab Compare December 30, 2025 07:15
@sliusardanya sliusardanya self-assigned this Dec 30, 2025
@sliusardanya sliusardanya added the enhancement New feature or request label Dec 30, 2025
@sliusardanya sliusardanya marked this pull request as ready for review December 30, 2025 08:15
@sliusardanya sliusardanya marked this pull request as draft December 30, 2025 10:20
…erate-remake

fix: make agent-token generate in proper way
Comment thread internal/adapter/ai/ai.go
return "", fmt.Errorf("ai adapter create agent token: %w", err)
}

return *response.JSON200.Token, nil

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Token может оказаться nil, нужна проверка

Comment thread pkg/clientai/client_ai.go
return fmt.Errorf("marshal request: %w", err)
}

url := cfg.UriString() + "/api/auth/userLogin?scopeType=Web"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Путь лучше в константы вынести, мне кажется.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants