Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New-OctopusKubernetesTarget does not validate that only one authentication mechanism has been provided #7580

Closed
1 task done
shaun-od opened this issue Jun 2, 2022 · 2 comments
Assignees
Labels
kind/bug This issue represents a verified problem we are committed to solving

Comments

@shaun-od
Copy link

shaun-od commented Jun 2, 2022

Team

  • I've assigned a team label to this issue

Severity

0

Version

Earliest tried 2022.3.1032 but would be in earlier versions

Latest Version

I could reproduce the problem in the latest build

What happened?

When both -octopusAccountIdOrName and -octopusClientCertificateIdOrName parameters are provided no error is thrown and certificate authentication is picked.

Reproduction

  1. Add a script steps that has a powershell scripts that calls the New-OctopusKubernetesTarget command and provides both the -octopusAccountIdOrName and -octopusClientCertificateIdOrName parameters.
  2. The Deployment target that is created will be using certificate authentication.

Error and Stacktrace

No response

More Information

No response

Workaround

Only provide a single parameter of either -octopusAccountIdOrName or -octopusClientCertificateIdOrName

@shaun-od shaun-od added kind/bug This issue represents a verified problem we are committed to solving state/triage labels Jun 2, 2022
@shaun-od shaun-od closed this as completed Jun 3, 2022
@octoreleasebot
Copy link

Release Note: New-OctopusKubernetesTarget command will now validate that only one of the following parameters have been provided -octopusAccountIdOrName and -octopusClientCertificateIdOrName

@Octobob
Copy link
Member

Octobob commented Aug 29, 2022

🎉 The fix for this issue has been released in:

Release stream Release
2022.3 2022.3.1270
2022.4+ all releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This issue represents a verified problem we are committed to solving
Projects
None yet
Development

No branches or pull requests

3 participants