From be44328ae286c876127edabf2993c746da1b6535 Mon Sep 17 00:00:00 2001
From: Mauro Mugnaini <themaxx76@gmail.com>
Date: Thu, 13 Aug 2020 17:27:20 +0200
Subject: [PATCH] Getting avatar in try catch block to avoid auto-login
 interruption when avatar is not found

---
 .../org/gcube/portal/oidc/lr62/OpenIdConnectAutoLogin.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/gcube/portal/oidc/lr62/OpenIdConnectAutoLogin.java b/src/main/java/org/gcube/portal/oidc/lr62/OpenIdConnectAutoLogin.java
index 94badf0..e7bf8c0 100644
--- a/src/main/java/org/gcube/portal/oidc/lr62/OpenIdConnectAutoLogin.java
+++ b/src/main/java/org/gcube/portal/oidc/lr62/OpenIdConnectAutoLogin.java
@@ -118,7 +118,12 @@ public class OpenIdConnectAutoLogin extends BaseAutoLogin {
                 UserLocalServiceUtil.updateUser(user);
             }
 
-            byte[] userAvatar = OpenIdConnectRESTHelper.getUserAvatar(configuration.getAvatarURL(), token);
+            byte[] userAvatar = null;
+            try {
+                userAvatar = OpenIdConnectRESTHelper.getUserAvatar(configuration.getAvatarURL(), token);
+            } catch (Exception e) {
+                log.info("Can't retrieve user's avatar from Keycloak: " + e.getLocalizedMessage());
+            }
             if (userAvatar != null) {
                 UserServiceUtil.updatePortrait(user.getUserId(), userAvatar);
             } else {