You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cr...@apache.org on 2021/06/29 17:10:53 UTC

[sling-org-apache-sling-auth-saml2] branch master updated: catch possible NPE when setting up Saml2UserMgtService

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

cris pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-auth-saml2.git


The following commit(s) were added to refs/heads/master by this push:
     new bb53e39  catch possible NPE when setting up Saml2UserMgtService
bb53e39 is described below

commit bb53e3960f4f0efbe160e00370f3af65a11fbbaa
Author: Cris Rockwell <cm...@umich.edu>
AuthorDate: Tue Jun 29 13:10:31 2021 -0400

    catch possible NPE when setting up Saml2UserMgtService
---
 .../java/org/apache/sling/auth/saml2/impl/Saml2UserMgtServiceImpl.java  | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/main/java/org/apache/sling/auth/saml2/impl/Saml2UserMgtServiceImpl.java b/src/main/java/org/apache/sling/auth/saml2/impl/Saml2UserMgtServiceImpl.java
index 60b7f00..cc27ac7 100644
--- a/src/main/java/org/apache/sling/auth/saml2/impl/Saml2UserMgtServiceImpl.java
+++ b/src/main/java/org/apache/sling/auth/saml2/impl/Saml2UserMgtServiceImpl.java
@@ -67,6 +67,8 @@ public class Saml2UserMgtServiceImpl implements Saml2UserMgtService {
                     "Check mapping org.apache.sling.auth.saml2:{}={}", SERVICE_NAME, SERVICE_USER, e);
         } catch (RepositoryException e) {
             logger.error("RepositoryException", e);
+        } catch (Exception e){
+            logger.error("Could not setup Saml2UserMgtService", e);
         }
         return false;
     }