Skip to content

#739 Logging#749

Merged
lprichar merged 9 commits intomainfrom
logging
Aug 17, 2022
Merged

#739 Logging#749
lprichar merged 9 commits intomainfrom
logging

Conversation

@lprichar
Copy link
Copy Markdown
Contributor

Issue

Supports #739

Description

Couldn't repro the 739 bug, so this adds several features that will help us if it occurs again:

  • If an error occurs during ballot decryption the system will now log it as an error and continue processing
  • This updates GUI logging to combine it with the regular EG logging
  • This updates GUI logging to send logs to the filesystem

Testing

Run the GUI, you should see file system logs at egui_mnt\data\logs

Comment thread src/electionguard/logs.py
Comment thread src/electionguard/logs.py
Comment thread src/electionguard/logs.py
Copy link
Copy Markdown
Contributor

@SteveMaier-IRT SteveMaier-IRT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@lprichar lprichar merged commit ffde9a7 into main Aug 17, 2022
@lprichar lprichar deleted the logging branch August 17, 2022 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants