Only accept HTML `<link rel="alternate">` on successful HTTP statuses
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
Merge request reports
Activity
added enhancement label
- Resolved by Julia
added 2 commits
- Resolved by Julia
- Resolved by Julia
while you're here, also
.close()
the windows inMfmService
: if these happy-dom structures are the cause of leak,MfmService
creates many many more of them! (one for each activity we send!)
added 1 commit
- e4cbd588 - Remove superfluous `DetachedWindowAPI.close` call
mentioned in merge request !634 (merged)
mentioned in commit c94f6994