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

myip4.ipip.net 似乎也挂了,修改了 ua(#397)。 #418

Closed
wants to merge 2 commits into from
Closed

myip4.ipip.net 似乎也挂了,修改了 ua(#397)。 #418

wants to merge 2 commits into from

Conversation

char-46
Copy link
Contributor

@char-46 char-46 commented Feb 3, 2024

No description provided.

@NewFuture
Copy link
Owner

#420 fix了 URL的问题,
UA再开一个PR?

@char-46
Copy link
Contributor Author

char-46 commented Feb 14, 2024 via email

@laozhoubuluo
Copy link
Contributor

myip6.ipip.net 现在也遇到了证书过期的问题,现在 IP 接口的问题需要合并且发版才能解决了。

@NewFuture
Copy link
Owner

http://myip4.ipip.net/似乎也不靠谱

1 similar comment
@NewFuture
Copy link
Owner

http://myip4.ipip.net/似乎也不靠谱

@laozhoubuluo
Copy link
Contributor

http://myip4.ipip.net/似乎也不靠谱

是的,所以不知道原提交作者什么时候有时间修复问题,或者您这边直接改完了写共同作者?

@char-46
Copy link
Contributor Author

char-46 commented Apr 16, 2024

http://myip4.ipip.net/似乎也不靠谱

是的,所以不知道原提交作者什么时候有时间修复问题,或者您这边直接改完了写共同作者?

上一次修改查询 ip 的 URL 是 #367#368。commit 和 PR 是我提的。当时解决的是 soho 提供的 api 全部指向上海机房的问题。当时有考虑过是否选择 ip.sb 作为默认的查询网址,但当时 ip.sb 的可用性远低于 ipip.net,因此选择了 ipip.net。现在 ip.sb 的可用性有所提高,且 ipip.net 在一定情况下不可靠,因此该 PR 建议将 默认使用的 api 调整到 ip.sb。

@laozhoubuluo
Copy link
Contributor

http://myip4.ipip.net/似乎也不靠谱

是的,所以不知道原提交作者什么时候有时间修复问题,或者您这边直接改完了写共同作者?

上一次修改查询 ip 的 URL 是 #367#368。commit 和 PR 是我提的。当时解决的是 soho 提供的 api 全部指向上海机房的问题。当时有考虑过是否选择 ip.sb 作为默认的查询网址,但当时 ip.sb 的可用性远低于 ipip.net,因此选择了 ipip.net。现在 ip.sb 的可用性有所提高,且 ipip.net 在一定情况下不可靠,因此该 PR 建议将 默认使用的 api 调整到 ip.sb。

lint 没过,所以没法合并。您看下面有写,所有 check 都得过了才行。

@char-46
Copy link
Contributor Author

char-46 commented Apr 18, 2024

lint 没过的是 run.py,我没修改这部分代码来着……

@laozhoubuluo
Copy link
Contributor

lint 没过的是 run.py,我没修改这部分代码来着……

要不您重新开一个 PR ?估计是之前的问题。

@char-46
Copy link
Contributor Author

char-46 commented Apr 18, 2024

也行

@char-46
Copy link
Contributor Author

char-46 commented Apr 18, 2024

这个我就先 close 了。

@char-46 char-46 closed this Apr 18, 2024
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