See CHANGELOG for v2.0.2

Author: jpnavarro

CHANGELOG

@@ -1,4 +1,7 @@
-v2.0.1 20221206 JP
+v2.0.2 20221206 JP
+  - Improve logging
+
+v2.0.1 20221205 JP
   - Fix new service bug that prevented links and hosts from being added
   - Set btn-access-auth border-radius
   - Include information/help glyphicon in header linking to application information in the operations portal

Operations_ServiceIndex_Django/services/signals.py

@@ -1,6 +1,5 @@
 #from django.db import models
 #import time
-import logging
 from django.contrib.auth.models import User
 from django.dispatch import receiver, Signal
 from allauth.account.signals import user_logged_in
@@ -9,6 +8,9 @@
 from allauth.socialaccount.providers.oauth2.client import OAuth2Error
 from allauth.socialaccount.signals import pre_social_login
 
+import logging
+logger = logging.getLogger(__name__)
+
 @receiver(user_logged_in)
 def set_username(request, user, **kwargs):
     try:
@@ -23,6 +25,9 @@ def set_username(request, user, **kwargs):
 
     user.username = username
     user.save()
+    msg = '{} logged in as {}'.format(subject, request.user.username)
+    logger.info(msg)
+
 
 @receiver(pre_social_login)
 def connect_existing_user(request, sociallogin, **kwargs):
@@ -33,11 +38,14 @@ def connect_existing_user(request, sociallogin, **kwargs):
         email = sociallogin.email_addresses[0]
     except:
         raise OAuth2Error('ACCESS CI Identity required email is missing')
-    
+
     try:    # username is unique
         existing_user = User.objects.get(email=email)
     except: # Let socialaccount handle creation
         return
 
     sociallogin.connect(request, existing_user)
     setup_user_email(request, existing_user, [])
+    msg = 'login as email {} connected to {}'.format(email, existing_user.username)
+    logger.info(msg)
+

Operations_ServiceIndex_Django/services/views.py

@@ -329,6 +329,8 @@ def export(request):
             t = get_template('services/export.txt')
             context = {'services':services}
             response.write(t.render(context, request))
+            msg = '{} exported data'.format(request.user.username)
+            logger.info(msg)
             return response
 
     else:
@@ -705,10 +707,10 @@ def add_event(request):
             e = form.save()     # creates Event
             # make status entries for each host
             for h in Host.objects.filter(service__deprecated=False):
-                hes = HostEventStatus(event=e,host=h,
-                        status=HostEventStatus.UNCHECKED)
+                hes = HostEventStatus(event=e, host=h, status=HostEventStatus.UNCHECKED)
                 hes.save()
-                
+            msg = '{} reported new event: {}'.format(request.user.username, e.name)
+            logger.info(msg)
             return redirect(reverse('services:events'))
     else:
         form = AddEventForm()
@@ -757,6 +759,8 @@ def update_event(request, hes_id):
             log.save()
             hes.status = form.cleaned_data['status']
             hes.save()
+            msg = '{} logged event for \'{}\' host'.format(request.user.username, hes.host.hostname)
+            logger.info(msg)
             return redirect(reverse('services:event', args=[hes.event.id]))
     else:
         #form = UpdateEventForm(instance=hes)