-
Notifications
You must be signed in to change notification settings - Fork 60
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
add: Timestamp checks #87
add: Timestamp checks #87
Conversation
Hi, thanks for the PR. Could you give some context on/reasoning for the new functionality? There should also be some examples added to the README and the code could use some testing. Check out the existing tests on how to implement those. |
So I have an application with an HTTP JSON API. The application has two cron jobs which should run every hour. I can check the status of the last run and I have a Timestamp when the cron job last ran. Status checks work great with this check, but the Timestamp of the last run doesn't. So I implemented a check for the age of the Timestamp. (I also added checks for the opposite - how recent the Timestamp is and also how far in the future...) Thank you for the response. Sorry for my late reply. I will add examples to the README and will add some testing. I hope it won't take me a long time. |
I added some tests and wrote some information + examples to the README. @martialblog could you please review this PR again? |
Thanks for the work. I will have a look at it. |
If the JSON has a Timestamp value you can check the age/youth with --key_time.