Skip to content

Fix: Setting Env Var, when it is not used#2154

Merged
Kraemii merged 2 commits intomainfrom
fix-krb5-conf-per-target
Mar 10, 2026
Merged

Fix: Setting Env Var, when it is not used#2154
Kraemii merged 2 commits intomainfrom
fix-krb5-conf-per-target

Conversation

@Kraemii
Copy link
Member

@Kraemii Kraemii commented Mar 10, 2026

When instead of the environment variable a given config path or the default is used, set the environment variable accordingly. This is necessary, as the GSSAPI does get the path via the env variable only.

@Kraemii Kraemii requested a review from a team as a code owner March 10, 2026 12:29
@github-actions
Copy link

github-actions bot commented Mar 10, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 8ff6b46.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions github-actions bot added the patch_release creates a patch release label Mar 10, 2026
Kraemii added 2 commits March 10, 2026 13:34
When instead of the environment variable a given config path or the default is used, set the environment variable accordingly. This is necessary, as the GSSAPI does get the path via the env variable only.
@Kraemii Kraemii force-pushed the fix-krb5-conf-per-target branch from c8e1ac3 to 8ff6b46 Compare March 10, 2026 12:37
@github-actions github-actions bot added patch_release creates a patch release and removed patch_release creates a patch release labels Mar 10, 2026
@Kraemii Kraemii merged commit 14088f2 into main Mar 10, 2026
33 of 35 checks passed
@Kraemii Kraemii deleted the fix-krb5-conf-per-target branch March 10, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch_release creates a patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants