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.

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

Optional IPV6 support for mirror checks #523

Open
pitastrudl opened this issue Aug 4, 2024 · 2 comments
Open

Optional IPV6 support for mirror checks #523

pitastrudl opened this issue Aug 4, 2024 · 2 comments

Comments

@pitastrudl
Copy link

Hi,

When adding or having an existing mirror inside archweb, one cannot control if IPV6 support is added or not. There has been cases where IPV6 is broken via mirror checks but they still use it or a failed ipv6 check is what is causing a bad mirror score for an otherwise healthy mirror (if ipv6 is not intended)

If I understand correctly, authenticated users in archweb can only see them as readonly fields

readonly_fields = ('protocol', 'has_ipv4', 'has_ipv6')

One Idea would be that we keep the same functionality but the fields become auto-ticked checkboxes if the mirror has either A or AAAA record. So one may disable either for whatever reason.

@jelly jelly changed the title Optioanl IPV6 support for mirror checks Optional IPV6 support for mirror checks Aug 19, 2024
@jelly
Copy link
Member

jelly commented Aug 19, 2024

So we have a service called mirrors/management/commands/mirrorresolv.py which sets the has_ipv4 and has_ipv6 fields and this is buggy? Then why don't we fix the issue of resolving if the mirror has ipv4 or ipv6?

Do you know a mirror which was wrongly shown as broken?

@pitastrudl
Copy link
Author

pitastrudl commented Aug 23, 2024

Hey, the issue is with this mirror: https://archlinux.org/mirrors/huongnguyen.dev/. Currently still ongoing.

Then why don't we fix the issue of resolving if the mirror has ipv4 or ipv6?

This could be a good idea, so we keep the same functionality. How would we go about fixing it? This is only doing resolving but not if the mirror endpoint is reachable or not, should we do a port test?

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

No branches or pull requests

2 participants