search

kubernetes-client / c

Official C client library for Kubernetes
Apache License 2.0
141 stars 45 forks source link

Remove printf from internal function #225

Closed homer6 closed 3 months ago

homer6 commented 4 months ago

The generic client is working well, however there's a quick tweak that I think would be useful.

Using the generic client, I'm seeing printf to stdout in the 200-level case and the 400-level case.

Would prefer if it didn't write to stdout. The response code is retrievable from the api_client, so I can handle it in the application.

https://github.com/kubernetes-client/c/blob/d0763b643f5da72604dc3097f69c2a67245d84c5/kubernetes/src/generic.c#L59

brendandburns commented 4 months ago

Yeah, I think it is fine to send a PR to remove those printfs.