Skip to content

Conversation

@Buqeta
Copy link
Contributor

@Buqeta Buqeta commented Dec 15, 2025

Description

Changed the logic for which API token and which team should be used on the commands. The priority has changed as follows:
1. If the profile is sent, it has precedence over any environment variable.
2. If the profile isn't sent, then the API_TOKEN and TEAM_URL environment variables are used.
3. If API_TOKEN and TEAM_URL environment variables aren't set, then CELONIS_API_TOKEN and CELONIS_URL are used.
- The difference here is that if the CELONIS_URL doesn't start with https:// or http://, then we add https:// to the beginning of it.

The reasoning for this is that the CELONIS_API_TOKEN environment variable seems to not be an admin token and not all the CLI commands work if we give more priority to CELONIS_API_TOKEN and CELONIS_URL env vars.

As for the API_TOKEN and TEAM_URL, they're used on ML Workbenches and the existing scripts could break if we give more precedence to CELONIS_API_TOKEN and CELONIS_URL, due to the fact that CELONIS_API_TOKEN is not an admin token.

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@Buqeta Buqeta requested a review from a team as a code owner December 15, 2025 15:10
@Buqeta Buqeta marked this pull request as draft December 15, 2025 15:22
@Buqeta Buqeta changed the title TA-4767: Append protocol to CELONIS_URL environment variable TA-4767: Fix API token and team URL selection on commands Dec 17, 2025
@Buqeta Buqeta marked this pull request as ready for review December 17, 2025 08:03
@sonarqubecloud
Copy link

@Buqeta Buqeta requested a review from promeris December 18, 2025 14:32
@Buqeta Buqeta merged commit 3f8b3c3 into master Dec 19, 2025
5 checks passed
@Buqeta Buqeta deleted the TA-4767-fix-protocol-on-celonis-url-env-var branch December 19, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants