Skip to content

Fix incorrect time values in the Commvault Security IQ CCF connector - #14990

Open
adutt-commvault wants to merge 1 commit into
Azure:masterfrom
adutt-commvault:fix/commvault-ccf-unix-timestamp
Open

Fix incorrect time values in the Commvault Security IQ CCF connector#14990
adutt-commvault wants to merge 1 commit into
Azure:masterfrom
adutt-commvault:fix/commvault-ccf-unix-timestamp

Conversation

@adutt-commvault

Copy link
Copy Markdown
Contributor

Change(s):

  • Updated the Commvault Security IQ CCF Data Connector to send correctly formatted time values when retrieving security events.
  • Regenerated the solution package using the Microsoft Sentinel solution packaging tool.
  • Updated the solution package version to 3.0.7.
  • Updated the solution release notes.

Reason for Change(s):

  • The connector used an unsupported time format, causing malformed fromTime and toTime values to be sent to the Commvault API.
  • This change ensures that polling time values are sent as valid Unix timestamps.

Version Updated:

  • Not applicable. No Detection/Analytic Rule templates were modified.

Testing Completed:

  • Yes.
  • Successfully regenerated the solution template and version 3.0.7 package using the solution packaging tool.
  • Verified that the source poller configuration, generated template, and packaged template contain the corrected time format.
  • Verified that the unsupported time format is no longer present.

Checked that the validations are passing and have addressed any issues that are present:

  • Yes. Relevant packaging and content validation checks passed.

@adutt-commvault
adutt-commvault requested review from a team as code owners August 26, 2026 21:47
@v-atulyadav
v-atulyadav requested a lite review from Copilot August 27, 2026 04:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 queryTimeFormat to use UnixTimestamp (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",
@v-maheshbh

Copy link
Copy Markdown
Contributor

Hi adutt-commvault

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants