Fix incorrect time values in the Commvault Security IQ CCF connector - #14990
Fix incorrect time values in the Commvault Security IQ CCF connector#14990adutt-commvault wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Commvault Security IQ CCF connector packaging to fix the time format used for polling, ensuring fromTime/toTime are sent as valid Unix timestamps, and bumps the solution version to 3.0.7.
Changes:
- Changed CCF poller
queryTimeFormatto useUnixTimestamp(vs. the prior unsupported format). - Regenerated/updated solution package artifacts and version strings to 3.0.7.
- Added 3.0.7 release notes entry describing the fix.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| Solutions/Commvault Security IQ/ReleaseNotes.md | Adds 3.0.7 entry documenting the timestamp formatting fix. |
| Solutions/Commvault Security IQ/Package/mainTemplate.json | Bumps package versions to 3.0.7 and updates connector poller time format in the packaged template. |
| Solutions/Commvault Security IQ/Data/Solution_Commvault Security IQ.json | Updates the solution version to 3.0.7 in the solution metadata. |
| Solutions/Commvault Security IQ/Data Connectors/CommvaultSecurityIQ_CCF/CommvaultSecurityIQ_PollerConfig.json | Updates the source poller configuration time format to UnixTimestamp. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "retryCount": 3, | ||
| "timeoutInSeconds": 120, | ||
| "queryTimeFormat": "UnixTimestampInSeconds", | ||
| "queryTimeFormat": "UnixTimestamp", |
| "retryCount": 3, | ||
| "timeoutInSeconds": 120, | ||
| "queryTimeFormat": "UnixTimestampInSeconds", | ||
| "queryTimeFormat": "UnixTimestamp", |
| @@ -17,7 +17,7 @@ | |||
| ], | |||
| "Metadata": "SolutionMetadata.json", | |||
| "BasePath": "/Users/akhildutt/Documents/GitHub/Azure-Sentinel/Solutions/Commvault Security IQ", | |||
|
Kindly review the Copilot comments and address any applicable suggestions. Once completed, please mark the conversations as resolved. and attach CCF connector testing screenshots, including the connected state for the changes introduced in this PR. Thanks! |
Change(s):
Reason for Change(s):
fromTimeandtoTimevalues to be sent to the Commvault API.Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: