imperva_cloud_waf: handle case where cursor state cannot be found in … #14298
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
efd6 added bugfix 
…log file index The previous code assumes that it is possible to determine the remainder of work to be done by splitting on the last log file that was collected. This fails when the last log file collected is not in the log files that the index returns for whatever reason. In that case, use the complete index to start again on the basis that the cursor log file state has gone stale relative to the index returned by the API.
efd6 force-pushed the s6128-imperva_cloud_waf branch from 14721c9 to db3793a Compare 
🚀 Benchmarks reportTo see the full report comment with |
|
💚 Build Succeeded
cc @efd6 |
| Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
| Package imperva_cloud_waf - 1.12.2 containing this change is available at https://epr.elastic.co/package/imperva_cloud_waf/1.12.2/ |
shmsr pushed a commit to shmsr/integrations that referenced this pull request Jun 30, 2025
…log file index (elastic#14298) The previous code assumes that it is possible to determine the remainder of work to be done by splitting on the last log file that was collected. This fails when the last log file collected is not in the log files that the index returns for whatever reason. In that case, use the complete index to start again on the basis that the cursor log file state has gone stale relative to the index returned by the API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Proposed commit message
Note
Due to testing infrastructure limitations, the change here is not testable in CI.
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots