diff options
author | Thibaut Girka <thib@sitedethib.com> | 2018-09-20 18:36:00 +0200 |
---|---|---|
committer | Thibaut Girka <thib@sitedethib.com> | 2018-09-20 18:36:00 +0200 |
commit | 343acd96cccb16b1ecdc3a52a8733359d4a92633 (patch) | |
tree | ecd2a077b038b864ac7912cba72dd59990fb6258 /app/services | |
parent | fba0200765c14d0ada47a44b5a8603001c92e069 (diff) | |
parent | 50c90a909e400628febd53c805165f1da5ecbfbd (diff) |
Merge branch 'master' into glitch-soc/merge-upstream
Diffstat (limited to 'app/services')
-rw-r--r-- | app/services/verify_link_service.rb | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/app/services/verify_link_service.rb b/app/services/verify_link_service.rb index 846751cd5..7d53bc255 100644 --- a/app/services/verify_link_service.rb +++ b/app/services/verify_link_service.rb @@ -27,6 +27,22 @@ class VerifyLinkService < BaseService def link_back_present? return false if @body.empty? - Nokogiri::HTML(@body).xpath('//a[contains(concat(" ", normalize-space(@rel), " "), " me ")]|//link[contains(concat(" ", normalize-space(@rel), " "), " me ")]').any? { |link| link['href'] == @link_back } + links = Nokogiri::HTML(@body).xpath('//a[contains(concat(" ", normalize-space(@rel), " "), " me ")]|//link[contains(concat(" ", normalize-space(@rel), " "), " me ")]') + + if links.any? { |link| link['href'] == @link_back } + true + elsif links.empty? + false + else + link_redirects_back?(links.first['href']) + end + end + + def link_redirects_back?(test_url) + redirect_to_url = Request.new(:head, test_url, follow: false).perform do |res| + res.headers['Location'] + end + + redirect_to_url == @link_back end end |