diff options
author | Claire <claire.github-309c@sitedethib.com> | 2024-02-22 19:12:57 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-22 19:12:57 +0100 |
commit | f0541adbd44bc031fb8b070e24a605e0a0e853e4 (patch) | |
tree | 8c63f39137a1befdc0320d3ebe1aacaee7ef49e7 | |
parent | 3fecb3673970c4495a3757cecc33c64c5af793d1 (diff) |
Fix link verifications when page size exceeds 1MB (#29362)
-rw-r--r-- | app/services/verify_link_service.rb | 2 | ||||
-rw-r--r-- | spec/services/verify_link_service_spec.rb | 13 |
2 files changed, 8 insertions, 7 deletions
diff --git a/app/services/verify_link_service.rb b/app/services/verify_link_service.rb index 707aeb4e08b..b317fc31a85 100644 --- a/app/services/verify_link_service.rb +++ b/app/services/verify_link_service.rb @@ -19,7 +19,7 @@ class VerifyLinkService < BaseService def perform_request! @body = Request.new(:get, @url).add_headers('Accept' => 'text/html').perform do |res| - res.code == 200 ? res.body_with_limit : nil + res.code == 200 ? res.truncated_body : nil end end diff --git a/spec/services/verify_link_service_spec.rb b/spec/services/verify_link_service_spec.rb index 8f65f3a8462..0d08ec14eb5 100644 --- a/spec/services/verify_link_service_spec.rb +++ b/spec/services/verify_link_service_spec.rb @@ -78,24 +78,25 @@ RSpec.describe VerifyLinkService, type: :service do " <!doctype html> <body> - <a rel=\"me\" href=\"#{ActivityPub::TagManager.instance.url_for(account)}\" + <a rel=\"me\" href=\"#{ActivityPub::TagManager.instance.url_for(account)}\"> " end - it 'marks the field as not verified' do - expect(field.verified?).to be false + it 'marks the field as verified' do + expect(field.verified?).to be true end end - context 'when a link back might be truncated' do + context 'when a link tag might be truncated' do let(:html) do " <!doctype html> <body> - <a rel=\"me\" href=\"#{ActivityPub::TagManager.instance.url_for(account)}" + <a rel=\"me\" href=\"#{ActivityPub::TagManager.instance.url_for(account)}\" + " end - it 'does not mark the field as verified' do + it 'marks the field as not verified' do expect(field.verified?).to be false end end |