Skip to content

fix(execd): default listener to IPv4 only#801

Merged
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:hotfix/execd/v4only
Apr 27, 2026
Merged

fix(execd): default listener to IPv4 only#801
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:hotfix/execd/v4only

Conversation

@Pangjiping
Copy link
Copy Markdown
Collaborator

Summary

  • Use net.Listen("tcp4", addr) instead of engine.Run(addr) to avoid binding to IPv6 dual-stack socket.

Testing

  • Not run (explain why)
  • Unit tests
  • Integration tests
  • e2e / manual verification

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

Use net.Listen("tcp4", addr) instead of engine.Run(addr) to avoid
binding to IPv6 dual-stack socket.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@ninan-nn ninan-nn left a comment

Choose a reason for hiding this comment

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

LGTM

@ninan-nn ninan-nn merged commit 6607863 into alibaba:main Apr 27, 2026
12 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component/execd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants