Skip to content

enh(apps::monitoring::zscaler::zdx::api): adapt to OneAPI authentication#6094

Merged
omercier merged 11 commits intodevelopfrom
CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode
Apr 10, 2026
Merged

enh(apps::monitoring::zscaler::zdx::api): adapt to OneAPI authentication#6094
omercier merged 11 commits intodevelopfrom
CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode

Conversation

@omercier
Copy link
Copy Markdown
Contributor

@omercier omercier commented Apr 2, 2026

Description

Authentication to Zscaler APIs now all go through OneAPI (read heare for more information).

Since this is now the only way to access this API, the plugin had to be adapted.

Refs: CTOR-2251

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Functionality enhancement or optimization (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

⚡ Enhancements

  • Replaced legacy auth with OneAPI client_credentials flow and options
  • Updated packaging and Dockerfiles to add dependencies and enable repos

🔧 Refactors

  • Changed token caching and retrieval to SHA256-based cache key
  • Hardened API responses with safe defaults and missing-result handling

More info

@omercier omercier requested review from a team as code owners April 2, 2026 16:06
@omercier omercier force-pushed the CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode branch from ed066e9 to e38260f Compare April 2, 2026 16:07
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
Comment thread tests/apps/monitoring/zscaler/zdx/api/application.robot Outdated
Comment thread tests/apps/monitoring/zscaler/zdx/api/discovery.robot Outdated
Comment thread tests/apps/monitoring/zscaler/zdx/api/list-locations.robot Outdated
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
@omercier omercier marked this pull request as draft April 3, 2026 07:53
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
Comment thread src/apps/monitoring/zscaler/zdx/api/custom/api.pm Outdated
@omercier omercier force-pushed the CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode branch from 84fc0fb to b8371ef Compare April 3, 2026 16:30
@omercier omercier force-pushed the CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode branch from b8371ef to b3873f9 Compare April 9, 2026 07:43
@omercier omercier marked this pull request as ready for review April 9, 2026 07:43
@omercier omercier requested a review from a team as a code owner April 9, 2026 07:43
@omercier omercier requested a review from scresto31 April 9, 2026 07:43
kduret
kduret previously approved these changes Apr 9, 2026
@omercier omercier force-pushed the CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode branch from 224c511 to bf73e02 Compare April 9, 2026 16:12
@omercier omercier requested a review from kduret April 10, 2026 07:19
@omercier omercier merged commit e6ee0d3 into develop Apr 10, 2026
284 checks passed
@omercier omercier deleted the CTOR-2251-applications-monitoring-zscaler-zdx-api-handle-new-oneAPI-authentication-mode branch April 10, 2026 07:25
crsuser pushed a commit to crsuser/centreon-plugins that referenced this pull request Apr 21, 2026
…ion (centreon#6094)

Co-authored-by: Sylvain Cresto <scresto@centreon.com>
Refs: CTOR-2251
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants