Merge pull request #1603 from CartoDB/jarroyo/ch64739/send-user-id-wh…

…en-a-user-creates-a-map

Use default credentials. Add checks to avoid exceptions

cartoframes/auth/credentials.py

@@ -118,7 +118,9 @@ def user_id(self):
 
             try:
                 user_me = api_key_auth_client.send(ME_SERVICE, 'get').json()
-                self._user_id = user_me.get('user_data', {}).get('id')
+                user_data = user_me.get('user_data')
+                if user_data:
+                    self._user_id = user_data.get('id')
 
             except ValueError:  # When the response isn't a JSON
                 pass

cartoframes/utils/metrics.py

@@ -114,10 +114,7 @@ def build_extra_metrics_data(decorated_function, *args, **kwargs):
             'credentials', decorated_function, *args, **kwargs)
         credentials = get_credentials(credentials)
         return {'user_id': credentials.user_id} if credentials and credentials.user_id else {}
-
-    # ValueError: When the decorated function doesn't contain `credentials`, e.g. creating a map
-    # AttributeError: When no user is set, e.g., reading a public table
-    except (ValueError, AttributeError):
+    except Exception:
         return {}
 
 

cartoframes/utils/utils.py

@@ -529,15 +529,17 @@ def fn(*args, **kw):
 
 
 def get_parameter_from_decorator(parameter_name, decorated_function, *args, **kwargs):
+    parameter = None
+
     try:
         parameter = kwargs[parameter_name]
-
     except KeyError:
         try:
-            parameter_arg_index = inspect.getargspec(decorated_function).args.index(parameter_name)
-            parameter = args[parameter_arg_index]
-
+            parameter_args = inspect.getargspec(decorated_function).args
+            if parameter_name in parameter_args:
+                parameter_arg_index = parameter_args.index(parameter_name)
+                parameter = args[parameter_arg_index]
         except IndexError:
-            parameter = None
+            pass
 
     return parameter