Skip to content

fix: use Log Analytics API for delegated consent - #435

Merged
Paulo Lacerda (placerda) merged 2 commits into
developfrom
placerda-specify-issue-433
Aug 21, 2026
Merged

fix: use Log Analytics API for delegated consent#435
Paulo Lacerda (placerda) merged 2 commits into
developfrom
placerda-specify-issue-433

Conversation

@placerda

Copy link
Copy Markdown
Contributor

Summary

  • use the actual Log Analytics API resource app (ca7f3f0b-7d91-482c-8e09-c5d840d0eac5) during hosted Cockpit consent preflight
  • preserve delegated scope resolution by value for the exact Data.Read grant
  • add regression coverage for the corrected resource and delegated scope

Context

A real Entra consent attempt for issue #433 failed because the previous resource app ID did not resolve to the Log Analytics API in the tenant. Tenant discovery and Microsoft documentation identify the corrected resource app ID. Tenant admin consent remains an external prerequisite.

Validation

  • python -m pytest tests/unit/test_cockpit_deployment_preview.py -q (77 passed)
  • python -m pytest tests/ -x -q (1886 passed, 1 skipped)
  • Ruff on changed files passed; repository-wide Ruff reports two unrelated pre-existing E2E/script findings
  • mypy on the changed service passed; repository-wide mypy reports four unrelated pre-existing cloud_runner.py errors
  • all repository Bicep files compiled successfully
  • git diff --check passed

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: be09d4cd-c070-4714-8e1e-56e193076d69
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@placerda
Paulo Lacerda (placerda) merged commit bd1d85e into develop Aug 21, 2026
13 checks passed
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