Skip to content

Exclude oauth token endpoint from HLAPI enabled requirement#21288

Merged
cedric-anne merged 2 commits intoglpi-project:11.0/bugfixesfrom
cconard96:fix/inventory_oauth
Oct 8, 2025
Merged

Exclude oauth token endpoint from HLAPI enabled requirement#21288
cedric-anne merged 2 commits intoglpi-project:11.0/bugfixesfrom
cconard96:fix/inventory_oauth

Conversation

@cconard96
Copy link
Contributor

Checklist before requesting a review

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.

Description

Fixes #21238

Copy link
Contributor

@orthagh orthagh left a comment

Choose a reason for hiding this comment

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

I was about to say we may check the claims (and see only inventory is asked) when requesting the token, but I guess this should be sufficient as the claims are verified later anyway.

@cedric-anne cedric-anne added this to the 11.0.1 milestone Oct 6, 2025
@cedric-anne cedric-anne added the bug label Oct 6, 2025
Copy link
Member

@cedric-anne cedric-anne left a comment

Choose a reason for hiding this comment

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

Tests are failing.

@cedric-anne cedric-anne merged commit 010f639 into glpi-project:11.0/bugfixes Oct 8, 2025
13 of 18 checks passed
@github-actions
Copy link
Contributor

This PR has been is closed and has been automatically locked. Please open a new issue with all requested information for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GLPI-Agent inventory task with oauth token authentication is refused (403)

5 participants