Skip to content

Fix python_packages built for Linux#13746

Merged
rahul0216 merged 3 commits intoAzure:masterfrom
nitsan-tzur:cybr-epm-fix-binary
Mar 18, 2026
Merged

Fix python_packages built for Linux#13746
rahul0216 merged 3 commits intoAzure:masterfrom
nitsan-tzur:cybr-epm-fix-binary

Conversation

@nitsan-tzur
Copy link
Contributor

Required items, please complete

Change(s):

  • CyberArkEPMSentinelConn.zip

Reason for Change(s):

  • Reported errors running the data connector

@nitsan-tzur nitsan-tzur requested review from a team as code owners March 4, 2026 19:49
@v-shukore v-shukore added the Solution Solution specialty review needed label Mar 5, 2026
Copy link
Contributor

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@v-maheshbh
Copy link
Contributor

Hi @nitsan-tzur

Could you please share the running data connector screenshots and invocation logs here?

Thanks!

@nitsan-tzur
Copy link
Contributor Author

Hi @nitsan-tzur

Could you please share the running data connector screenshots and invocation logs here?

Thanks!

Hi @v-maheshbh ,
Attaching screenshot
image (1)

@v-maheshbh
Copy link
Contributor

Hi @nitsan-tzur

Kindly check the failing validation error, as it appears to be caused by a missing configuration.

Thanks!

@nitsan-tzur
Copy link
Contributor Author

Hi @nitsan-tzur

Kindly check the failing validation error, as it appears to be caused by a missing configuration.

Thanks!

Hi @v-maheshbh
This is an expected error. We aimed to demonstrate that previously observed errors, due to packaging, were resolved

@nitsan-tzur
Copy link
Contributor Author

Hi @nitsan-tzur
Kindly check the failing validation error, as it appears to be caused by a missing configuration.
Thanks!

Hi @v-maheshbh This is an expected error. We aimed to demonstrate that previously observed errors, due to packaging, were resolved

Hi @v-maheshbh a kind reminder. Can we proceed here?

v-maheshbh
v-maheshbh previously approved these changes Mar 18, 2026
v-atulyadav
v-atulyadav previously approved these changes Mar 18, 2026
v-dvedak
v-dvedak previously approved these changes Mar 18, 2026
@rahul0216
Copy link
Collaborator

Hi @nitsan-tzur, I would recommend removing .python_packages and subfolder. These python module files are not mandatory, as zip file is self contianed. These files create multiple security alerts if not maintained regularly.

@nitsan-tzur nitsan-tzur dismissed stale reviews from v-dvedak, v-atulyadav, and v-maheshbh via 05c97ce March 18, 2026 15:26
@nitsan-tzur
Copy link
Contributor Author

Hi @nitsan-tzur, I would recommend removing .python_packages and subfolder. These python module files are not mandatory, as zip file is self contianed. These files create multiple security alerts if not maintained regularly.

Good point @rahul0216, package removed

@rahul0216 rahul0216 merged commit ac9e480 into Azure:master Mar 18, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Solution Solution specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants