PAYARA-1501-Illegal-non-String-type-errors-when-JMX-accessed (payara#…

…1460)
Pandrex247 · Apr 3, 2017 · 0fe298c · 0fe298c
1 parent 807b0dd
commit 0fe298c
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/...tion-snmp-core/src/main/java/fish/payara/notification/snmp/SnmpNotifierConfiguration.java b/...tion-snmp-core/src/main/java/fish/payara/notification/snmp/SnmpNotifierConfiguration.java
@@ -73,7 +73,7 @@ public interface SnmpNotifierConfiguration extends NotifierConfiguration {
     String getHost();
     void setHost(String value) throws PropertyVetoException;
 
-    @Attribute(defaultValue = "162")
-    Integer getPort();
+    @Attribute(defaultValue = "162", dataType = Integer.class)
+    String getPort();
     void setPort(String value) throws PropertyVetoException;
 }
diff --git a/.../java/fish/payara/notification/snmp/SnmpNotifierConfigurationExecutionOptionsFactory.java b/.../java/fish/payara/notification/snmp/SnmpNotifierConfigurationExecutionOptionsFactory.java
@@ -68,7 +68,7 @@ public SnmpNotifierConfigurationExecutionOptions build(SnmpNotifierConfiguration
         executionOptions.setOid(notifierConfiguration.getOid());
         executionOptions.setVersion(notifierConfiguration.getVersion());
         executionOptions.setHost(notifierConfiguration.getHost());
-        executionOptions.setPort(notifierConfiguration.getPort());
+        executionOptions.setPort(Integer.parseInt(notifierConfiguration.getPort()));
 
         return executionOptions;
     }

diff --git a/...tion-xmpp-core/src/main/java/fish/payara/notification/xmpp/XmppNotifierConfiguration.java b/...tion-xmpp-core/src/main/java/fish/payara/notification/xmpp/XmppNotifierConfiguration.java
@@ -61,7 +61,7 @@ public interface XmppNotifierConfiguration extends NotifierConfiguration {
     void host(String host) throws PropertyVetoException;
 
     @Attribute(dataType = Integer.class, defaultValue = "5222")
-    Integer getPort();
+    String getPort();
     void port(String port) throws PropertyVetoException;
 
     @Attribute(required = true)
@@ -77,7 +77,7 @@ public interface XmppNotifierConfiguration extends NotifierConfiguration {
     void password(String password) throws PropertyVetoException;
 
     @Attribute(defaultValue = "false")
-    Boolean getSecurityDisabled();
+    String getSecurityDisabled();
     void securityDisabled(String securityDisabled) throws PropertyVetoException;
 
     @Attribute(required = true)

diff --git a/.../java/fish/payara/notification/xmpp/XmppNotifierConfigurationExecutionOptionsFactory.java b/.../java/fish/payara/notification/xmpp/XmppNotifierConfigurationExecutionOptionsFactory.java
@@ -67,11 +67,11 @@ public XmppNotifierConfigurationExecutionOptions build(XmppNotifierConfiguration
         executionOptions.setEnabled(Boolean.parseBoolean(notifierConfiguration.getEnabled()));
 
         executionOptions.setHost(notifierConfiguration.getHost());
-        executionOptions.setPort(notifierConfiguration.getPort());
+        executionOptions.setPort(Integer.parseInt(notifierConfiguration.getPort()));
         executionOptions.setServiceName(notifierConfiguration.getServiceName());
         executionOptions.setUsername(notifierConfiguration.getUsername());
         executionOptions.setPassword(notifierConfiguration.getPassword());
-        executionOptions.setSecurityDisabled(notifierConfiguration.getSecurityDisabled());
+        executionOptions.setSecurityDisabled(Boolean.valueOf(notifierConfiguration.getSecurityDisabled()));
         executionOptions.setRoomId(notifierConfiguration.getRoomId());
 
         return executionOptions;