[infrastructure] bump SAT to 0.7.0 (#1007)

* Bump SAT version; Fixed high priority findings
* Defined path to feature file to avoid warnings

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
This commit is contained in:
Christoph Weitkamp 2019-08-29 09:17:35 +02:00 committed by Kai Kreuzer
parent 1afff93e48
commit b3802d565d
5 changed files with 5 additions and 4 deletions

View File

@ -52,7 +52,7 @@ public class AuthenticationHandler implements Handler {
private static final String DEFAULT_LOGIN_URI = "/login"; private static final String DEFAULT_LOGIN_URI = "/login";
static final String REDIRECT_PARAM_NAME = "redirect"; static final String REDIRECT_PARAM_NAME = "redirect";
private final Logger logger = LoggerFactory.getLogger(AuthenticationManager.class); private final Logger logger = LoggerFactory.getLogger(AuthenticationHandler.class);
private final List<CredentialsExtractor<HttpServletRequest>> extractors = new CopyOnWriteArrayList<>(); private final List<CredentialsExtractor<HttpServletRequest>> extractors = new CopyOnWriteArrayList<>();

View File

@ -167,7 +167,7 @@ public class ExtensibleTrustManagerImpl extends X509ExtendedTrustManager impleme
.orElse(null); .orElse(null);
// @formatter:on // @formatter:on
} catch (CommonNameNotFoundException e) { } catch (CommonNameNotFoundException e) {
logger.debug("CN not found", e.getMessage()); logger.debug("CN not found", e);
return null; return null;
} catch (CertificateParsingException e) { } catch (CertificateParsingException e) {
logger.debug("Problem while parsing certificate", e); logger.debug("Problem while parsing certificate", e);

View File

@ -99,7 +99,7 @@ public class ReadyServiceImpl implements ReadyService {
notifyTracker(readyTracker, marker -> readyTracker.onReadyMarkerAdded(marker)); notifyTracker(readyTracker, marker -> readyTracker.onReadyMarkerAdded(marker));
} }
} catch (RuntimeException e) { } catch (RuntimeException e) {
logger.error("Registering tracker '" + readyTracker + "' failed!", e); logger.error("Registering tracker '{}' failed!", readyTracker, e);
} finally { } finally {
rwlTrackers.writeLock().unlock(); rwlTrackers.writeLock().unlock();
} }

View File

@ -71,7 +71,7 @@
<bnd.version>4.2.0</bnd.version> <bnd.version>4.2.0</bnd.version>
<karaf.compile.version>4.2.1</karaf.compile.version> <karaf.compile.version>4.2.1</karaf.compile.version>
<karaf.tooling.version>4.2.6</karaf.tooling.version> <karaf.tooling.version>4.2.6</karaf.tooling.version>
<sat.version>0.6.1</sat.version> <sat.version>0.7.0</sat.version>
<slf4j.version>1.7.21</slf4j.version> <slf4j.version>1.7.21</slf4j.version>
<xtext.version>2.17.0</xtext.version> <xtext.version>2.17.0</xtext.version>
</properties> </properties>

View File

@ -5,3 +5,4 @@ checkstyle.headerCheck.content=^/\\*\\*$\\n^ \\* Copyright \\(c\\) {0}-{1} Contr
checkstyle.headerCheck.values=2010,2019 checkstyle.headerCheck.values=2010,2019
checkstyle.pomXmlCheck.currentVersionRegex=^2\.5\.0 checkstyle.pomXmlCheck.currentVersionRegex=^2\.5\.0
checkstyle.requiredFilesCheck.files=pom.xml checkstyle.requiredFilesCheck.files=pom.xml
checkstyle.karafFeatureCheck.featureXmlPath=features/karaf/openhab-core/src/main/feature/feature.xml