[Bug] Reenable Broken link found (404 response).

[vidorteg]

🐞 Bug report

Details

Some unknown recent update seem to have broke a single test in hint-no-broken-links package and this is causing the PR CI pipeline to fail. With the spirit of keeping the main branch green and stable I disabled the test and this bug should be used to track the work required to reenable it. Link to PR : #5753

{
        name: `This test should fail as it has an img with 404 src value(absolute)`,
        reports: [{
            message: `Broken link found (404 response).`,
            severity: Severity.error
        }],
        serverConfig: generateHTMLPage('', bodyWithBrokenImageSource),
        skip: true // temporary disabling to investigate
    },

[shikhar13012001]

Hey @vidorteg I would like to work on this issue? Can you assign this to me? Thanks!

[Bhumit9416]

Hi! 👋
Thanks for documenting the root cause and temporary skip for the failing test. I’ve reviewed the related PR and it seems like the test can now be safely re-enabled after accounting for recent updates in how 404s and rel="nofollow" are handled.

If helpful, I’d be happy to test the fix locally and submit a patch to re-enable the test and confirm it’s passing with updated behavior. Please let me know if this is still open for contribution!

— Bhumit Mittal