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

fix: do not expose empty cipassword to API #364

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jay7x
Copy link

@jay7x jay7x commented Sep 12, 2024

@Tinyblargon
Copy link
Collaborator

Tinyblargon commented Sep 12, 2024

@jay7x thank you for your contribution, could you add a test case to cement this behavior.

This is the test I'm referring to:

func Test_ConfigQemu_mapToAPI(t *testing.T) {

Can you add the test case in the following category:

{category: `CloudInit`, // Create CloudInit no need for update as update and create behave the same. will be changed in the future

Looking at it it's probably only a create case. If you want me to add the test case that's also possible.

@jay7x
Copy link
Author

jay7x commented Sep 13, 2024

I'll try to add it this weekend :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: A password is being set for the default cloud-init user when cipassword is not-defined or null
2 participants