You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by go...@apache.org on 2017/11/17 16:06:57 UTC

[cxf-fediz] branch 1.4.x-fixes updated: IDP maxParameter size is set only if positive

This is an automated email from the ASF dual-hosted git repository.

gonzalad pushed a commit to branch 1.4.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf-fediz.git


The following commit(s) were added to refs/heads/1.4.x-fixes by this push:
     new 874b0f8  IDP maxParameter size is set only if positive
874b0f8 is described below

commit 874b0f8ecb2c9ccf25733550d69cba350f7a5354
Author: gonzalad <ad...@yahoo.fr>
AuthorDate: Fri Nov 17 01:27:38 2017 +0100

    IDP maxParameter size is set only if positive
    
    maxParameter size should be set only if positive,
    otherwise, we'll use default value.
---
 .../org/apache/cxf/fediz/service/idp/service/jpa/IdpDAOJPAImpl.java  | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/service/jpa/IdpDAOJPAImpl.java b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/service/jpa/IdpDAOJPAImpl.java
index 360755a..ee5b052 100644
--- a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/service/jpa/IdpDAOJPAImpl.java
+++ b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/service/jpa/IdpDAOJPAImpl.java
@@ -333,7 +333,10 @@ public class IdpDAOJPAImpl implements IdpDAO {
         idp.setRpSingleSignOutCleanupConfirmation(entity.isRpSingleSignOutCleanupConfirmation());
         idp.setAutomaticRedirectToRpAfterLogout(entity.isAutomaticRedirectToRpAfterLogout());
         idp.setDisableLogoutAddressValidation(entity.isDisableLogoutAddressValidation());
-        idp.setMaxParameterSize(entity.getMaxParameterSize());
+
+        if (entity.getMaxParameterSize() > 0) {
+            idp.setMaxParameterSize(entity.getMaxParameterSize());
+        }
 
         if (expandList != null && (expandList.contains("all") || expandList.contains("applications"))) {
             for (ApplicationEntity item : entity.getApplications()) {

-- 
To stop receiving notification emails like this one, please contact
['"commits@cxf.apache.org" <co...@cxf.apache.org>'].