is `null` valid header field value? about sttp

[adamw]

null values will probably either not work, or throw NullPointerException throughout sttp and other Scala libraries, as null is very rarely used (if at all) when writing Scala code. Alternatively, you can use an Option, which is very commonly used. So, your example would become basicRequest.header("name", map.get("field"))

[wenhoujx]

@adamw thanks for the reply. it makes sense most scala libs use Option instead of risking having NPE.
Occasionally we use java class in scala and some outdated conventions may introduce null.
Our case was we get the http request header from log4j ThreadContext.get("correlction-id") , which returns null if missing.

Do you think it's the library caller's responsibility to check the null or is should be done in the library?

[adamw]

I very rarely see any questions / bug reports concerning nulls, so I think if you're interfacing with Java it's mainly the responsiblity of the wrapper layer - in this case, that would be in your app. You can always do Option(ThreadContext.get(...)) which correctly handles null results

[wenhoujx]

sound good, thanks for the answers.

[adamw]

No problem, I'll close this then :)