Skip to content

Improve error messages for authentication setup and credentials#6098

Merged
cgoetz-inovex merged 3 commits intostackitcloud:mainfrom
avanderbergh:issue/6097
Apr 1, 2026
Merged

Improve error messages for authentication setup and credentials#6098
cgoetz-inovex merged 3 commits intostackitcloud:mainfrom
avanderbergh:issue/6097

Conversation

@avanderbergh
Copy link
Copy Markdown
Contributor

@avanderbergh avanderbergh commented Mar 31, 2026

Description

relates to #6097

Fix two incorrect error messages in the core auth package:

  • Correct the Workload Identity Federation setup error so it no longer reports no auth
  • Correct the corrupted credentials unmarshalling error message

Also add regression tests for both error-message paths and update the core changelog/version for the next core patch release.

Note: this PR updates the core changelog and core VERSION file, not a service changelog or service VERSION file.

Checklist

  • Issue was linked above
  • No generated code was adjusted manually (check comments in file header)
  • Changelogs
    • Changelog in the root directory was adjusted (see here)
    • Changelog(s) of the service(s) were adjusted (see e.g. here)
  • VERSION file(s) of the service(s) were adjusted
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • 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)

Core module note:core/CHANGELOG.md and [x] core/VERSION were adjusted

@avanderbergh avanderbergh requested a review from a team as a code owner March 31, 2026 14:40
Copy link
Copy Markdown
Contributor

@marceljk marceljk left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Looks good to me, only one nitpick regarding the changelog

@cgoetz-inovex cgoetz-inovex merged commit b5e9323 into stackitcloud:main Apr 1, 2026
12 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.

3 participants