Merge pull request #47 from kevgliss/keys

Fixing issue with a certificate with no role not being viewable

lemur/certificates/views.py

@@ -446,14 +446,13 @@ class CertificatePrivateKey(AuthenticatedResource):
 
         role = role_service.get_by_name(cert.owner)
 
-        if role:
-            permission = ViewKeyPermission(certificate_id, role.name)
+        permission = ViewKeyPermission(certificate_id, getattr(role, 'name', None))
 
-            if permission.can():
-                response = make_response(jsonify(key=cert.private_key), 200)
-                response.headers['cache-control'] = 'private, max-age=0, no-cache, no-store'
-                response.headers['pragma'] = 'no-cache'
-                return response
+        if permission.can():
+            response = make_response(jsonify(key=cert.private_key), 200)
+            response.headers['cache-control'] = 'private, max-age=0, no-cache, no-store'
+            response.headers['pragma'] = 'no-cache'
+            return response
 
         return dict(message='You are not authorized to view this key'), 403
 

lemur/utils.py

@@ -17,4 +17,5 @@ def get_key():
     try:
         return current_app.config.get('LEMUR_ENCRYPTION_KEY').strip()
     except RuntimeError:
+        print("No Encryption Key Found")
         return ''