-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: simplify exclude list, latest muffet, update redirects #1733
Conversation
oop! hold on - I'm getting false positives on github redirects on older repos that we've moved. Lemme try and resolve that before your review! |
so using the
|
OK!! muffet doesn't send a user agent apparently? A lot of sites don't like that, hence all the Commit b978454 gets only these (below) which I think are all valid. Almost there!
|
When I run this at home I often see Given it runs fine on EC2, I think it should be fine in GH CI then! 🤞
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work on cleaning up the muffet
script and updating it to the latest version. ⚡️
I suggested a small fix (I believe it was a copy paste issue), but otherwise it looks good.
I suggest we merge it, let it run in the CI and fix anything that needs fixing.
Co-authored-by: Andra Blaj <[email protected]>
Description
This PR updates the muffet link checker to:
http[s]*:
.*" \
"http[s]*://.*google.com/.*DOC-ID-HERE.*" \
127*
andlocalhost
are at the end--dns-resolver
flag nowgithub.com/medic/medic/
->github.com/medic/cht-core/
github.com/medic/medic-webapp/
->github.com/medic/cht-core/
github.com/medic/medic-conf/
->github.com/medic/cht-conf/
github.com/medic/medic-android/
->github.com/medic/cht-android/
*Redirect issue
OMG - what in the world!? saw this and just decided to do the repo rename in the links to avoid this case. Two identical calls to
curl
two different responses 🪦License
The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.