Skip to content

Expose P2P only in K8s#2402

Merged
skudasov merged 5 commits intomainfrom
do-not-expose-p2p
Feb 13, 2026
Merged

Expose P2P only in K8s#2402
skudasov merged 5 commits intomainfrom
do-not-expose-p2p

Conversation

@skudasov
Copy link
Contributor

@skudasov skudasov commented Feb 13, 2026


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes remove the exposure of the P2P port from a component, focusing only on the HTTP and custom port configurations. This simplification could be due to security concerns, a change in network architecture, or the realization that P2P port exposure is unnecessary for this component's operation.

What

  • framework/components/clnode/clnode.go
    • Removed the P2P port from both exposedPorts and portBindings. This means the component will no longer expose the P2P port by default.
    • The removal of P2P port-related code simplifies the generatePortBindings function, potentially reducing security risks and unnecessary network exposure.

@github-actions
Copy link

👋 skudasov, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@skudasov skudasov changed the title do not expose p2p Expose P2P only in K8s Feb 13, 2026
@skudasov skudasov enabled auto-merge (squash) February 13, 2026 17:24
@skudasov skudasov merged commit 41c0b19 into main Feb 13, 2026
83 of 84 checks passed
@skudasov skudasov deleted the do-not-expose-p2p branch February 13, 2026 17:42
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