From 6997c388ddea17cefc2d012116d96a6a947adfdb Mon Sep 17 00:00:00 2001 From: Sparow199 Date: Sun, 17 Nov 2019 19:06:44 +0100 Subject: [PATCH] doc(*): Clean loggers fix writing extensions format --- docs/src/main/asciidoc/writing-extensions.adoc | 12 ++++++------ .../client/deployment/InfinispanClientProcessor.java | 2 +- .../runtime/QuarkusIdentityProviderManagerImpl.java | 2 +- .../io/quarkus/it/elytron/KeyFactoryEndpoint.java | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/docs/src/main/asciidoc/writing-extensions.adoc b/docs/src/main/asciidoc/writing-extensions.adoc index b3bf6e1ea..52334084b 100644 --- a/docs/src/main/asciidoc/writing-extensions.adoc +++ b/docs/src/main/asciidoc/writing-extensions.adoc @@ -1768,7 +1768,7 @@ An example of parsing a XML config file using JAXB is shown in the `TestProcesso Unmarshaller unmarshaller = context.createUnmarshaller(); InputStream is = getClass().getResourceAsStream("/config.xml"); <1> if (is != null) { - log.infof("Have XmlConfig, loading"); + log.info("Have XmlConfig, loading"); XmlConfig config = (XmlConfig) unmarshaller.unmarshal(is); <2> ... } @@ -1854,7 +1854,7 @@ You can use the `io.quarkus.arc.runtime.BeanContainer` interface to interact wit public void configureBeans(BeanContainer beanContainer, Class beanClass, TestBuildAndRunTimeConfig buildTimeConfig, TestRunTimeConfig runTimeConfig) { - log.infof("Begin BeanContainerListener callback\n"); + log.info("Begin BeanContainerListener callback\n"); IConfigConsumer instance = beanContainer.instance(beanClass); <4> instance.loadConfig(buildTimeConfig, runTimeConfig); <5> log.infof("configureBeans, instance=%s\n", instance); @@ -1881,9 +1881,9 @@ A common purpose for an extension is to integrate a non-CDI aware service into t Unmarshaller unmarshaller = context.createUnmarshaller(); InputStream is = getClass().getResourceAsStream("/config.xml"); if (is != null) { - log.infof("Have XmlConfig, loading"); + log.info("Have XmlConfig, loading"); XmlConfig config = (XmlConfig) unmarshaller.unmarshal(is); - log.infof("Loaded XmlConfig, creating service"); + log.info("Loaded XmlConfig, creating service"); RuntimeValue service = recorder.initRuntimeService(config); //<1> serviceBuildItem = new RuntimeServiceBuildItem(service); //<3> } @@ -2139,7 +2139,7 @@ Objects created during the build phase that are passed into the runtime need to There is a `io.quarkus.runtime.ObjectSubstitution` interface that can be implemented to tell Quarkus how to handle such classes. An example implementation for the `DSAPublicKey` is shown here: .DSAPublicKeyObjectSubstitution Example -[source,bash] +[source,java] ---- package io.quarkus.extest.runtime.subst; @@ -2199,7 +2199,7 @@ An extension registers this substitution by producing an `ObjectSubstitutionBuil ObjectSubstitutionBuildItem keysub = new ObjectSubstitutionBuildItem(holder); substitutions.produce(keysub); - log.infof("loadDSAPublicKey run"); + log.info("loadDSAPublicKey run"); return new PublicKeyBuildItem(publicKey); } ---- diff --git a/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java b/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java index 17c5ee868..8907d6135 100644 --- a/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java +++ b/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java @@ -97,7 +97,7 @@ class InfinispanClientProcessor { if (stream == null) { properties = new Properties(); if (log.isTraceEnabled()) { - log.tracef("There was no hotrod-client.properties file found - using defaults"); + log.trace("There was no hotrod-client.properties file found - using defaults"); } } else { try { diff --git a/extensions/security/runtime/src/main/java/io/quarkus/security/runtime/QuarkusIdentityProviderManagerImpl.java b/extensions/security/runtime/src/main/java/io/quarkus/security/runtime/QuarkusIdentityProviderManagerImpl.java index 29a08c94c..5f50f88af 100644 --- a/extensions/security/runtime/src/main/java/io/quarkus/security/runtime/QuarkusIdentityProviderManagerImpl.java +++ b/extensions/security/runtime/src/main/java/io/quarkus/security/runtime/QuarkusIdentityProviderManagerImpl.java @@ -112,7 +112,7 @@ public class QuarkusIdentityProviderManagerImpl implements IdentityProviderManag List> providers, T request, AuthenticationRequestContext context) { if (pos == providers.size()) { //we failed to authentication - log.debugf("Authentication failed as providers would authenticate the request"); + log.debug("Authentication failed as providers would authenticate the request"); CompletableFuture cf = new CompletableFuture<>(); cf.completeExceptionally(new AuthenticationFailedException()); return cf; diff --git a/integration-tests/elytron-security/src/main/java/io/quarkus/it/elytron/KeyFactoryEndpoint.java b/integration-tests/elytron-security/src/main/java/io/quarkus/it/elytron/KeyFactoryEndpoint.java index c5c0a55e4..af6137ecb 100644 --- a/integration-tests/elytron-security/src/main/java/io/quarkus/it/elytron/KeyFactoryEndpoint.java +++ b/integration-tests/elytron-security/src/main/java/io/quarkus/it/elytron/KeyFactoryEndpoint.java @@ -57,7 +57,7 @@ public class KeyFactoryEndpoint { log.infof("Loaded SHA256withRSA: %s", sha256withRSA); //log.infof("Loaded SHA256withRSA: %s, %s", sha256withRSA, sha256withRSA.getProvider()); sha256withRSA.initVerify(pk); - log.infof("Initialized SHA256withRSA"); + log.info("Initialized SHA256withRSA"); return true; }