git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2102: ARTEMIS-1882 verify PKCS12 keystores wo...


Github user jbertram commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2102#discussion_r190585655
  
    --- Diff: tests/compatibility-tests/src/main/resources/servers/artemisServer.groovy ---
    @@ -44,7 +44,7 @@ try {
         if (!type.startsWith("ARTEMIS-1")) {
             configuration.addAddressesSetting("#", new AddressSettings().setAutoCreateAddresses(true));
             if (globalMaxSize != null) {
    -            configuration.getAddressesSettings().get("#").setPageSizeBytes(globalMaxSize).setAddressFullMessagePolicy(AddressFullMessagePolicy.PAGE)
    +            configuration.getAddressesSettings().get("#").setPageSizeBytes(Long.parseLong(globalMaxSize)).setAddressFullMessagePolicy(AddressFullMessagePolicy.PAGE)
    --- End diff --
    
    I only added this fix because the PR build was failing.  I'm not sure how or where it broke.  It's not related to the previous commit.


---