Skip to content

Remove logResponseContent attribute from Polling Config - #15036

Open
nitsan-tzur wants to merge 1 commit into
Azure:masterfrom
nitsan-tzur:hotfix-logResponseContent
Open

Remove logResponseContent attribute from Polling Config#15036
nitsan-tzur wants to merge 1 commit into
Azure:masterfrom
nitsan-tzur:hotfix-logResponseContent

Conversation

@nitsan-tzur

Copy link
Copy Markdown
Contributor

Required items, please complete

Change(s):

  • Remove logResponseContent from CyberArkAudit_PollingConfig.json

Reason for Change(s):

  • Security patch to prevent sensitive response data to be captured in telemetry

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.

Note

Copilot was unable to run its full agentic suite in this review.

Pull request overview

Removes logResponseContent from polling configuration to avoid capturing sensitive response payloads in telemetry.

Changes:

  • Removed logResponseContent from the polling config used by the CyberArkAudit CCP data connector.
  • Removed logResponseContent from the packaged ARM template equivalent.
  • Added a release note entry documenting the security patch.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
Solutions/CyberArkAudit/ReleaseNotes.md Documents the security patch as version 3.2.1.
Solutions/CyberArkAudit/Package/mainTemplate.json Removes logResponseContent from the packaged template polling/request config.
Solutions/CyberArkAudit/Data Connectors/CyberArkAudit_CCP/CyberArkAudit_PollingConfig.json Removes logResponseContent from the connector’s polling configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Solutions/CyberArkAudit/Package/mainTemplate.json
@v-maheshbh

v-maheshbh (v-maheshbh) commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Hi nitsan-tzur

Kindly review the Copilot comments and address them where applicable.
and attach a screenshot of the CCF connector in the Connected state for validation.

Thanks!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants