Could you guys help with what behaviour should be implemented for empty values?

Options:

• fail on empty value (throw IllegalArgumentException)
• ignore field with empty value & log -> return null;
• ignore field with empty value & log -> return defaultValue;

FYI: I explicitly fail test until this behaviour will be defined and implemented

Hi @NikitaZhevnitskiy
Thanks for offering to help!

When the credsStore string is blank, I think the right behaviour would be to return null from the authConfigUsingStore method. This effectively behaves as if the credsStore key is just not present at all.

I'd agree that it's a good idea to log, though - perhaps at WARN level?

Thanks!
Richard

Releasing this in 1.11.0 🎉