Skip to content
Snippets Groups Projects

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

Merged Julia requested to merge feature/only-accept-html-alternate-on-success into develop

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading