False positive : apache commons StringUtils::isNotBlank should be considered a valid "null check"

Rules and Languages Report False-positive / False-negative...
,
1

java:S2259
SonarQube * Enterprise Edition* Version 9.2.4 (build 50792)

    private List<String> getEnvPropertyList(String propName) {
        List<String> propList = emptyList();
        String propValue = env.getProperty(propName);
        if (isNotBlank(propValue)) {
            propList = List.of(propValue.split(","));
        }
        return propList;
    }

image
image860×222 12.9 KB

2

Hey there.

Take a look at this post here.

4

I found the problem.
The static import was referencing and obsolete version of StringUtils

Thanks