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

fix: latency test for selector and ipv6 test #860

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

YetAnotherZephyruso
Copy link
Contributor

Remove useless logic for latency, for the seletor and urltest group we simply use it as a pointer to point the real latency for render
For ipv6 test, this part of code is a nightmare, i try my best, already tested on sing-box for single node latency test and proxy groups latency test, but not provider latencty test since i don't have one

@YetAnotherZephyruso
Copy link
Contributor Author

And a minor fix for default regex of quick filter

@YetAnotherZephyruso
Copy link
Contributor Author

YetAnotherZephyruso commented Jul 25, 2024

Let me know if there is still have problem with latency test or ipv6 test

@arkxfly
Copy link
Contributor

arkxfly commented Sep 6, 2024

After a few months, I synchronized the code today and found that IPv6 had been changed 😔.

The IPv6 icon cannot be displayed because the version, this PR #549 has not been merged.

@YetAnotherZephyruso
Copy link
Contributor Author

YetAnotherZephyruso commented Sep 7, 2024

cannot be displayed

I thought IPv6 test works as expected now ,What do you mean by "Cannot be displayed", any screenshot or log?

Oh i see, You are using legacy version of mihomo

@arkxfly
Copy link
Contributor

arkxfly commented Sep 7, 2024

It is displayed when you click test, but it disappears when you refresh.

@YetAnotherZephyruso
Copy link
Contributor Author

It is displayed when you click test, but it disappears when you refresh.

Oh i see, I have a solution for that

@YetAnotherZephyruso
Copy link
Contributor Author

Should works with #983 now

@arkxfly
Copy link
Contributor

arkxfly commented Sep 8, 2024

I see that the status is returned on the API.

In fact, I would rather read the status from extra, because I use it on multiple devices, sometimes a mobile phone, sometimes a tablet, and sometimes a PC.

@YetAnotherZephyruso
Copy link
Contributor Author

YetAnotherZephyruso commented Sep 8, 2024

I see that the status is returned on the API.

In fact, I would rather read the status from extra, because I use it on multiple devices, sometimes a mobile phone, sometimes a tablet, and sometimes a PC.

But this way it can be compatible with sing-box at the same time because sing-box does not have "extra" at all

As for your situation, You only have to test once for each device in the first time, and i don‘t think that is unbearable?

@arkxfly
Copy link
Contributor

arkxfly commented Sep 8, 2024

So is it possible to use extra if available and only use localstorage if not?

@YetAnotherZephyruso
Copy link
Contributor Author

YetAnotherZephyruso commented Sep 8, 2024

Of course,You can try pr it since i don't have config for mihomo

@YetAnotherZephyruso YetAnotherZephyruso deleted the ipv6-test branch October 14, 2024 02:43
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