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

Token is unsuited for purpose - develop approach and handling #75

Closed
Tracked by #71
cd-rite opened this issue Jan 8, 2024 · 0 comments · Fixed by #109
Closed
Tracked by #71

Token is unsuited for purpose - develop approach and handling #75

cd-rite opened this issue Jan 8, 2024 · 0 comments · Fixed by #109
Assignees

Comments

@cd-rite
Copy link
Collaborator

cd-rite commented Jan 8, 2024

Current behavior:
preflight auth failure - leaves a log entry, does not retry on failure, watcher stops.
subsequent auth failures - keeps processing, adds log entries

subsequent auth failure modes:

  • user does not have required grant for collection

    • throw error, stop processing, retry
  • user can't create assets or assign stigs, when Watcher is configured to do so.

    • throw error, stop processing, retry
  • user can't approve reviews, when watcher is configured to do so.

    • note: double-check API behavior when given status that does not conform with Collection grants/settings
    • log error, continue processing, mark file complete?

retry behavior:

  • scan mode: suspend scanning and file parsing while retrying
  • event mode: continue capturing file add/remove events, suspend file parsing
  • retry every 60 seconds
    • Log initial failure
    • Log every (hour? day? thereafter)
    • Log success
    • log: every attempt at debug level
    • continue trying indefinitely
@cd-rite cd-rite changed the title token is unsuited for purpose - develop approach Token is unsuited for purpose - develop approach Jan 8, 2024
@cd-rite cd-rite changed the title Token is unsuited for purpose - develop approach Token is unsuited for purpose - develop approach and handling Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants