When I execute `okteto ctx` command with --insecure-skip-tls-verify flag, the `Insecure mode enabled` message is displayed twice

[ifbyol]

When I execute `okteto ctx` command with --insecure-skip-tls-verify flag, the `Insecure mode enabled` message is displayed twice

[nitishfy]

I'll solve this issue. Please assign this to me!

[jLopezbarb]

Assigned!! I don't know how trivial this one is, but let us know if you face any problem and we'll try to give you some tips

[nitishfy]

Assigned!! I don't know how trivial this one is, but let us know if you face any problem and we'll try to give you some tips

I'm late for this but I've done my research. Here's what I have found after debugging:

The function PersistentPreRun is being called twice, first for the root and then for the context command. Everytime it is called, it is always going to be executing the okteto.SetInsecureSkipTLSVerifyPolicy(ctxOptions.InsecureSkipTlsVerify) function.

if parent := cmd.Parent(); parent != nil {
	if parent.PersistentPreRun != nil {
	   parent.PersistentPreRun(parent, args)
	}
}

When the parent.PersistentPreRun() function is called recursively for the first time, it changes it's value to main.main() due to which we are facing the above issue. For every recursive call, the function okteto.SetInsecureSkipTLSVerifyPolicy(ctxOptions.InsecureSkipTlsVerify) is called.

I think one way we can solve this is by either calling the function before the If statement OR setting up a boolean flag/count pointer.

cc: @jLopezbarb

[andreafalzetti]

The change seems requiring some refactoring in order to make the change testable and avoid any workaround. I am removing the good first issue label because such change should be done and owned by the CLI team. Apologies agian @nitishfy and thanks for all your contributions 🙇

[github-actions]

This issue is stale because it has been open for 60 days with no activity. Comment on this issue or it will be closed in 7 days