Only accept HTML `<link rel="alternate">` on successful HTTP statuses (!633) · Merge requests · TransFem.org / Sharkey

Julia requested to merge feature/only-accept-html-alternate-on-success into develop Sep 22, 2024

What does this PR do? This PR makes it such that ApRequestService no longer attempts to parse HTML responses to extract a <link rel="alternate" type="application/activity+json"> tag from the response unless the HTTP status is successful (200-299). This avoids incurring a performance penalty when calling ApRequestService.signedGet with the url of a server that is currently down, for one reason or another.

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

I agree to follow this project's Contribution Guidelines
I have made sure to test this pull request

Only accept HTML `<link rel="alternate">` on successful HTTP statuses

Merge request reports