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

Link checker is temperamental and apparently deprecated #1990

Open
tghosth opened this issue Jul 10, 2024 · 8 comments
Open

Link checker is temperamental and apparently deprecated #1990

tghosth opened this issue Jul 10, 2024 · 8 comments
Assignees
Labels
GH_ACTIONS MAKEFILE _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine.

Comments

@tghosth
Copy link
Collaborator

tghosth commented Jul 10, 2024

The link checker seems to sometimes fail, even when it should not.

For example:
https://github.com/OWASP/ASVS/actions/runs/9854671316/job/27207915942

The link seems to work even through the link checker said it didn't.

Also, it says in the README that it is deprecated.
https://github.com/gaurav-nelson/github-action-markdown-link-check?tab=readme-ov-file

@ike do you think you could upgrade to the newer tool he suggests?

@tghosth tghosth added MAKEFILE _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine. labels Jul 10, 2024
@ike
Copy link
Collaborator

ike commented Jul 10, 2024

Yeah! Looks like he's also created a new github action.

@tghosth
Copy link
Collaborator Author

tghosth commented Jul 11, 2024

Ok so do you want to try and upgrade it?

@tghosth
Copy link
Collaborator Author

tghosth commented Aug 26, 2024

Hi @ike, any update on this?

arkid15r added a commit to arkid15r/owasp-asvs that referenced this issue Aug 27, 2024
tghosth pushed a commit that referenced this issue Aug 27, 2024
@tghosth
Copy link
Collaborator Author

tghosth commented Aug 29, 2024

It would also be good to only include 5.0 in the link checker and not 4.0 because we don't change 4.0 any more but we do add translations so it makes a mess:
https://github.com/OWASP/ASVS/actions/runs/10573577586/job/29345695930?pr=1973

@ike @arkid15r

@arkid15r
Copy link
Contributor

It would also be good to only include 5.0 in the link checker and not 4.0 because we don't change 4.0 any more but we do add translations so it makes a mess: https://github.com/OWASP/ASVS/actions/runs/10573577586/job/29345695930?pr=1973

@ike @arkid15r

Alright, I'll work on migrating the workflow to a proper replacement and running it for 5.0 only.
I'm going to look around first as https://github.com/UmbrellaDocs/action-linkspector looks pretty young action

ERROR: 2 dead links found!
[✖] https://owasp.org/www-project-security-knowledge-framework/ → Status: 404
[✖] https://cheatsheetseries.owasp.org/cheatsheets/IndexASVS.html → Status: 404

The findings from https://github.com/OWASP/ASVS/actions/runs/10573577586/job/29345695930?pr=1973 look legit to me as they are 404. My understanding is they just irrelevant as 4.0 doesn't require any updates @tghosth ?

@tghosth
Copy link
Collaborator Author

tghosth commented Aug 30, 2024

My understanding is they just irrelevant as 4.0 doesn't require any updates

Correct

Alright, I'll work on migrating the workflow to a proper replacement and running it for 5.0 only.

Great!

@arkid15r
Copy link
Contributor

arkid15r commented Sep 2, 2024

Just a small update:

As #2035 has been merged please let me know if you notice something is broken or works not as expected. I'll keep an eye on the workflow too.

The only part left to resolve this issue is addressing URL checker action deprecated status. I'm going to work on it this week.

@tghosth
Copy link
Collaborator Author

tghosth commented Sep 2, 2024

Thanks so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GH_ACTIONS MAKEFILE _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine.
Projects
None yet
Development

No branches or pull requests

3 participants