Release 1.2.5 (#118)

Release 1.2.5

.rubocop.yml

@@ -3,3 +3,7 @@ Layout/LineLength:
 
 Layout/EndOfLine:
   EnforcedStyle: lf
+
+ChefCorrectness/PropertyWithoutType:
+  Exclude:
+    - "resources/*.rb"

CHANGELOG.md

@@ -1,3 +1,10 @@
+## v.1.2.5:
+* Fix : Revert Azul JDK specific JVM options from 1.2.2
+* Enhance : Improve JVM option parsing
+
+## v.1.2.3 - v.1.2.4
+* Unreleased
+
 ## v.1.2.2:
 * Enhance : Better support for Payara 5.194+.
 

libraries/asadmin.rb

@@ -44,17 +44,17 @@ def transform_jvm_options(options, withoutversions = false)
         min = ''
         max = ''
         unless withoutversions
-          capture = line.match(/min\(([\d\.]+)\)/)
+          capture = line.match(/min\(([\w\-\d\.]+)\)/)
           min = capture.captures.first unless capture.nil?
 
-          capture = line.match(/max\(([\d\.]+)\)/)
+          capture = line.match(/max\(([\w\-\d\.]+)\)/)
           max = capture.captures.first unless capture.nil?
 
           min = min.gsub(/(\d+)\.(\d+)\.(\d+)\.(\d+)/, '\1.\2.\3u\4')
           max = max.gsub(/(\d+)\.(\d+)\.(\d+)\.(\d+)/, '\1.\2.\3u\4')
         end
 
-        base = line.gsub(/   --> JDK versions: [A-Za-z\(\d\.\), ]+/, '')
+        base = line.gsub(/   --> JDK versions: [A-Za-z\-\(\d\.\), ]+/, '')
 
         if min != '' || max != ''
           "[#{min}|#{max}]#{base}"

metadata.rb

@@ -3,8 +3,7 @@
 maintainer_email '[email protected]'
 license 'Apache-2.0'
 description 'Installs/Configures GlassFish Application Server'
-version '1.2.2'
-
+version '1.2.5'
 chef_version '>= 13.0'
 
 issues_url 'https://github.com/realityforge/chef-glassfish'

providers/iiop_listener.rb

@@ -38,7 +38,7 @@
   end
 
   properties = new_resource.properties.dup.merge(
-    'address' => new_resource.listeneraddress ? new_resource.listeneraddress : '0.0.0.0',
+    'address' => (new_resource.listeneraddress || '0.0.0.0'),
     'enabled' => new_resource.enabled,
     'port' => new_resource.iiopport,
     'security-enabled' => new_resource.securityenabled

recipes/default.rb

@@ -24,12 +24,11 @@
 def gf_scan_existing_binary_endorsed_jars(install_dir)
   jar_extensions = ['.jar']
   gf_binary_endorsed_dir = install_dir + '/glassfish/lib/endorsed'
-  existing_binary_endorsed_jars = if Dir.exist?(gf_binary_endorsed_dir)
-                                    Dir.entries(gf_binary_endorsed_dir).reject { |f| File.directory?(f) || !jar_extensions.include?(File.extname(f)) }
-                                  else
-                                    []
-                                  end
-  existing_binary_endorsed_jars
+  if Dir.exist?(gf_binary_endorsed_dir)
+    Dir.entries(gf_binary_endorsed_dir).reject { |f| File.directory?(f) || !jar_extensions.include?(File.extname(f)) }
+  else
+    []
+  end
 end
 
 include_recipe 'glassfish::derive_version'

resources/domain.rb

@@ -162,19 +162,14 @@ def osgi_jvm_options
 end
 
 def security_jvm_options
-  opts = [
+  [
     '-Dcom.sun.enterprise.security.httpsOutboundKeyAlias=s1as',
     "-Djavax.net.ssl.keyStore=#{domain_dir_path}/config/keystore.jks",
     "-Djava.security.policy=#{domain_dir_path}/config/server.policy",
     "-Djavax.net.ssl.trustStore=#{domain_dir_path}/config/cacerts.jks",
     '-Dcom.sun.enterprise.security.httpsOutboundKeyAlias=s1as',
     "-Djava.security.auth.login.config=#{domain_dir_path}/config/login.conf",
   ]
-
-  if node['glassfish']['variant'] == 'payara' && node['glassfish']['version'].split('.')[0].to_i >= 5 && node['glassfish']['version'].split('.')[1].to_i >= 194
-    opts << '[Azul-1.8.0u222|1.8.0u260]-XX:+UseOpenJSSE'
-  end
-  opts
 end
 
 def grizzly_options