Skip to content

Conversation

@BillFarber
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings August 8, 2025 15:21
@BillFarber BillFarber requested a review from rjrudin as a code owner August 8, 2025 15:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds security hardening to Docker services by dropping the NET_RAW capability to address Polaris security scanning issues. The NET_RAW capability allows containers to create raw sockets, which Polaris identifies as a security risk.

  • Adds cap_drop: [NET_RAW] configuration to all Docker services in docker-compose.yml
  • Includes explanatory comment for the broker service describing the purpose of removing NET_RAW capability
Comments suppressed due to low confidence (1)

depends_on:
- broker
ports:
- "8081:8081"
Copy link

Copilot AI Aug 8, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider adding a comment explaining the purpose of dropping NET_RAW capability for consistency with the broker service, or alternatively, reference the comment on the broker service to avoid duplication.

Suggested change
- "8081:8081"
- "8081:8081"
# See broker service above for explanation of dropping NET_RAW capability.

Copilot uses AI. Check for mistakes.
@BillFarber BillFarber changed the title Add "cap_drop" with "NET_RAW" to Docker services to prevent Polaris issue. MLE-23437 - Add "cap_drop" with "NET_RAW" to Docker services to prevent Polaris issue. Aug 8, 2025
@BillFarber BillFarber force-pushed the task/fixDockerIssue branch from 578f497 to fe1c518 Compare August 8, 2025 15:37
@BillFarber BillFarber merged commit 2e483fd into develop Aug 8, 2025
2 checks passed
@BillFarber BillFarber deleted the task/fixDockerIssue branch August 8, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants