Merge pull request #25789 from gastaldi/icon

Remove `quarkus.webauthn.relying-party.icon`
quarkusio · May 25, 2022 · 1d923f6 · 1d923f6
2 parents 748908f + f65d35a
commit 1d923f6
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 9 deletions.
diff --git a/...ty-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java b/...ty-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java
@@ -181,12 +181,6 @@ public static class RelyingPartyConfig {
          */
         @ConfigItem(defaultValue = "Quarkus server")
         public String name;
-
-        /**
-         * A URL location for an icon
-         */
-        @ConfigItem
-        public Optional<String> icon;
     }
 
     // FIXME: merge with form config?

diff --git a/...ecurity-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnSecurity.java b/...ecurity-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnSecurity.java
@@ -41,9 +41,6 @@ public WebAuthnSecurity(WebAuthnRunTimeConfig config, Vertx vertx, WebAuthnAuthe
         }
         // this is required
         relyingParty.setName(config.relyingParty.name);
-        if (config.relyingParty.icon.isPresent()) {
-            relyingParty.setIcon(config.relyingParty.icon.get());
-        }
         options.setRelyingParty(relyingParty);
         if (config.attestation.isPresent()) {
             options.setAttestation(config.attestation.get());