Skip to content

Feature/wtel 8714 sip user agent - #508

Open
suifri wants to merge 2 commits into
mainfrom
feature/WTEL-8714-sip-user-agent
Open

suifri wants to merge 2 commits into
mainfrom
feature/WTEL-8714-sip-user-agent

Conversation

@suifri

@suifri suifri commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@suifri
suifri requested a review from navrotskyj September 14, 2026 15:41
@webitel-review

Copy link
Copy Markdown
Contributor

🤖 Webitel Code Review

Цей комміт додає підтримку передачі User-Agent при створенні дзвінка (додається змінна для SIP-інвайту sip_h_User-Agent з контексту) та розширює gRPC-структуру історії дзвінків HistoryCall новим полем user_agent. Загальний ризик низький, проте інтеграція поля в історію дзвінків виглядає неповною.

📋 Walkthrough (2 файл(и/ів))
Файл Зміни
app/call.go Додано вилучення User-Agent з контексту та його встановлення у змінні інвайту як sip_h_User-Agent.
gen/engine/call.pb.go Оновлено згенеровані protobuf-файли: додано поле UserAgent та метод GetUserAgent() до структури HistoryCall.

Знахідки

  • [medium] gen/engine/call.pb.go:2275 — Поле UserAgent додано до gRPC-структури HistoryCall, проте у пов'язаному коді engine/grpc_api/call.go у функції toEngineHistoryCall відсутнє мапування цього поля з внутрішньої моделі model.HistoryCall (яка також, судячи з engine/model/call.go, наразі не містить поля UserAgent). Через це значення user_agent не буде повертатися в API запитів історії дзвінків. Перевірте, чи планується додавання цього поля до моделі БД та його мапування в gRPC-відповідь.
🔗 Cross-repo callers змінених символів (1)
  • GetUserAgentwebitel.go/app/auth.go:603

Index-grounded review across the Webitel codebase. Знахідки можуть бути неточними — перевіряйте перед застосуванням.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant