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

Okta 4.2.1 Release: Improved Monitor Logs Task #2015

Merged
merged 2 commits into from
Oct 11, 2023
Merged

Conversation

joneill-r7
Copy link
Collaborator

@joneill-r7 joneill-r7 commented Oct 9, 2023

Release of Okta 4.2.1 to include the following changes:

  • dedupe events that match the since query parameter
  • only update this state when new event logs are returned to prevent loss of data when pagination has occurred.
  • if user has paused the task, on resuming we don't use the saved timestamp if it is greater than 24 hours ago.

PR:

Testing on staging:

  • Staging configured to run Okta on 5 minute interval
  • Checked platform logs and can see this new logic/logs being hit
  • removing events previously returned
  • not moving checkpoint forward if no events
staging_logs

Local testing:

  • Ran one larger period of time and returned the same number of events (39)

local_logs

@joneill-r7 joneill-r7 added the Do Not Merge ⛔ PR should not be merged label Oct 9, 2023
* PLGN-398: filter out any previously returned log events/update time format

* PLGN-398: only update collection timestamp when new events returned.
@joneill-r7 joneill-r7 removed the Do Not Merge ⛔ PR should not be merged label Oct 10, 2023
@joneill-r7 joneill-r7 marked this pull request as ready for review October 10, 2023 14:57
@ablakley-r7 ablakley-r7 merged commit af86daa into master Oct 11, 2023
6 checks passed
@joneill-r7 joneill-r7 deleted the okta-4.2.1-release branch February 29, 2024 10:30
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.

4 participants