Skip to content

Conversation

Benjosh95
Copy link
Contributor

@Benjosh95 Benjosh95 commented Sep 10, 2025

Description

Use existing GetValidAccessToken in curl commands to automatically refresh expired tokens for user token flows

Jira: https://jira.schwarz/browse/STACKITCLI-22

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see e.g. here)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@Benjosh95 Benjosh95 requested a review from a team as a code owner September 10, 2025 07:54
@Benjosh95 Benjosh95 force-pushed the bf/fix-curl-refresh-tokens branch from fa8134a to 4f55873 Compare September 12, 2025 07:02
Copy link

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/stackitcloud/stackit-cli/internal/cmd/auth/get-access-token 0.00% (ø)
github.com/stackitcloud/stackit-cli/internal/cmd/curl 51.02% (+2.00%) 👍
github.com/stackitcloud/stackit-cli/internal/pkg/auth 56.70% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/stackitcloud/stackit-cli/internal/cmd/auth/get-access-token/get_access_token.go 0.00% (ø) 30 (+1) 0 30 (+1)
github.com/stackitcloud/stackit-cli/internal/cmd/curl/curl.go 51.02% (+2.00%) 98 (-4) 50 48 (-4) 👍
github.com/stackitcloud/stackit-cli/internal/pkg/auth/auth.go 54.17% (ø) 96 52 44

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@Benjosh95 Benjosh95 merged commit d1e9eca into main Sep 12, 2025
5 checks passed
@Benjosh95 Benjosh95 deleted the bf/fix-curl-refresh-tokens branch September 12, 2025 07:47
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.

2 participants