Merge pull request keitaroinc#78 from GSA/logging-events

Audit authentication events
GSA · Jan 12, 2023 · 9943fb2 · 9943fb2
2 parents 43d6510 + 1203f68
commit 9943fb2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -31,6 +31,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         python-version: [ '3.7', '3.8', '3.9']
         ckan-version: ["2.9", "2.10"]

diff --git a/ckanext/saml2auth/plugin.py b/ckanext/saml2auth/plugin.py
@@ -108,6 +108,11 @@ def logout(self):
                 # CKAN <= 2.9.x also sets auth_tkt cookie
                 response.set_cookie('auth_tkt', domain=domain, expires=0)
 
+        if g.userobj:
+            log.info(u'User {0}<{1}> logged out successfully'.format(g.userobj.name, g.userobj.email))
+        else:
+            log.info(u'No user was logged in!')
+
         return response
 
 

diff --git a/ckanext/saml2auth/views/saml2auth.py b/ckanext/saml2auth/views/saml2auth.py
@@ -295,6 +295,8 @@ def _log_user_into_ckan(resp):
         user_id = g.userobj.name
     set_repoze_user(user_id, resp)
 
+    log.info(u'User {0}<{1}> logged in successfully'.format(g.userobj.name, g.userobj.email))
+
 
 def saml2login():
     u'''Redirects the user to the