Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use only IPv4 for discovery. IPv6 discovery e.g. does not work from containers. #201

Merged
merged 1 commit into from
Dec 1, 2024
Merged

Use only IPv4 for discovery. IPv6 discovery e.g. does not work from containers. #201

merged 1 commit into from
Dec 1, 2024

Conversation

ptabor
Copy link
Contributor

@ptabor ptabor commented Nov 30, 2024

No description provided.

@sfc-gh-ptabor
Copy link

Error I'm getting without this patch:

Nov 30 21:54:45 plaster-arch.local pilot[578]: time="2024-11-30T21:54:45+01:00" level=info msg="error discovering entries: unable to create new zeroconf resolver: listen udp6 [ff02::]:5353: socket: address family not supported by protocol"
Nov 30 21:54:45 plaster-arch.local pilot[578]: 2024/11/30 21:54:45 main.go:770: RESPONSE Code: 200

vishen
vishen approved these changes Dec 1, 2024
View reviewed changes
Copy link
Owner

@vishen vishen left a comment

Choose a reason for hiding this comment

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

Not sure the ramifications of merging this as i'm not sure if IPv6 discovery works for others, but I think it should be fine.

@vishen vishen merged commit 92f518d into vishen:master Dec 1, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vishen vishen vishen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ptabor @sfc-gh-ptabor @vishen