This is an automated email from the ASF dual-hosted git repository.

brondsem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/allura.git


The following commit(s) were added to refs/heads/master by this push:
     new 44fda66f4 avoid error in audit-logging if a tool is missing an option 
field
44fda66f4 is described below

commit 44fda66f4e65eda3d32b8c5907d29f5b5d697aa1
Author: Dave Brondsema <[email protected]>
AuthorDate: Fri Feb 7 15:55:54 2025 -0500

    avoid error in audit-logging if a tool is missing an option field
---
 Allura/allura/app.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Allura/allura/app.py b/Allura/allura/app.py
index 5515cccfd..02a9a81bd 100644
--- a/Allura/allura/app.py
+++ b/Allura/allura/app.py
@@ -1016,11 +1016,11 @@ def configure(self, **kw):
                 except fev.Invalid as e:
                     flash(f'{opt.name}: {str(e)}', 'error')
                     continue
-                if self.app.config.options[opt.name] != val:
+                if self.app.config.options.get(opt.name) != val:
                     M.AuditLog.log('{}: set option "{}" {} => {}'.format(
                         self.app.config.options['mount_point'],
                         opt.name,
-                        self.app.config.options[opt.name],
+                        self.app.config.options.get(opt.name, '(not set)'),
                         val
                     ))
                 self.app.config.options[opt.name] = val

Reply via email to