Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The "poll_incidents.py" in PR #10 fails if there is no checkpoint file (last_datestring_indexed isn't set, but is used later) #13

Open
BobBarnesMN opened this issue Sep 12, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@BobBarnesMN
Copy link

Describe the bug

"last_datestring_indexed" is not defined in #10 if there is no checkpoint file (i.e. on a first run). Just add

  last_datestring_indexed=""

after line 69

Expected behavior

Expect code to work if there is no checkpoint file

Current behavior

The "else" condition when there is no checkpoint file does not set "last_datestring_indexed", so we get an error when it is accessed a few lines later

Possible solution

Just add

  last_datestring_indexed=""

after line 69

Steps to reproduce

Screenshots

Context

Your Environment

  • Version used:
  • Environment name and version (e.g. Chrome 59, node.js 5.4, python 3.7.3):
  • Operating System and version (desktop or mobile):
  • Link to your project:
@BobBarnesMN BobBarnesMN added the bug Something isn't working label Sep 12, 2022
@nterl0k
Copy link

nterl0k commented Sep 18, 2022

Added corrective fix for this issue in latest commit. Thanks!

https://github.com/nterl0k/prisma-cloud-compute-splunk/tree/nterl0k-No-serialNum-patch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants