jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix #3424: /iqdb_queries.json fails for certain urls.

Browse Source 
Fix the HTML page -> image URL download rewrite strategy failing for
https://lohas.nicoseiga.jp/thumb/${id}i URLs.
This commit is contained in:
evazion
2017-12-15 09:27:33 -06:00
parent ec81b87766
commit 4c39783d28
2 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/logical/sources/strategies/nico_seiga.rb
View File

@@ -120,15 +120,15 @@ module Sources
def normalized_url def normalized_url
@normalized_url ||= begin @normalized_url ||= begin
if url =~ %r!\Ahttp://lohas\.nicoseiga\.jp/o/[a-f0-9]+/\d+/(\d+)! if url =~ %r!\Ahttps?://lohas\.nicoseiga\.jp/o/[a-f0-9]+/\d+/(\d+)!
"http://seiga.nicovideo.jp/seiga/im#{$1}" "http://seiga.nicovideo.jp/seiga/im#{$1}"
elsif url =~ %r{\Ahttp://lohas\.nicoseiga\.jp/priv/(\d+)\?e=\d+&h=[a-f0-9]+}i elsif url =~ %r{\Ahttps?://lohas\.nicoseiga\.jp/priv/(\d+)\?e=\d+&h=[a-f0-9]+}i
"http://seiga.nicovideo.jp/seiga/im#{$1}" "http://seiga.nicovideo.jp/seiga/im#{$1}"
elsif url =~ %r{\Ahttp://lohas\.nicoseiga\.jp/priv/[a-f0-9]+/\d+/(\d+)}i elsif url =~ %r{\Ahttps?://lohas\.nicoseiga\.jp/priv/[a-f0-9]+/\d+/(\d+)}i
"http://seiga.nicovideo.jp/seiga/im#{$1}" "http://seiga.nicovideo.jp/seiga/im#{$1}"
elsif url =~ %r{\Ahttp://lohas\.nicoseiga\.jp/priv/(\d+)}i elsif url =~ %r{\Ahttps?://lohas\.nicoseiga\.jp/priv/(\d+)}i
"http://seiga.nicovideo.jp/seiga/im#{$1}" "http://seiga.nicovideo.jp/seiga/im#{$1}"
elsif url =~ %r{\Ahttp://lohas\.nicoseiga\.jp//?thumb/(\d+)}i elsif url =~ %r{\Ahttps?://lohas\.nicoseiga\.jp//?thumb/(\d+)i?}i
"http://seiga.nicovideo.jp/seiga/im#{$1}" "http://seiga.nicovideo.jp/seiga/im#{$1}"
elsif url =~ %r{/seiga/im\d+} elsif url =~ %r{/seiga/im\d+}
url url

8
test/unit/sources/nico_seiga_test.rb
View File

@@ -49,6 +49,14 @@ module Sources
@site_2.to_h @site_2.to_h
end end
end end
should "work for a https://lohas.nicoseiga.jp/thumb/${id}i url" do
site = Sources::Site.new("https://lohas.nicoseiga.jp/thumb/6844226i")
site.get
full_image_url = %r!https?://lohas.nicoseiga.jp/priv/[a-f0-9]{40}/[0-9]+/6844226!
assert_match(full_image_url, site.image_url)
end
end end
end end
end end