github/gallery-dl
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[urlgalleries] do not strip query parameters (#7888)

Browse Source 
fixes regression introduced in fc06243b89
This commit is contained in:
Mike Fährmann
2025-07-24 08:23:23 +02:00
parent cf2e5a1619
commit b2a354f0d2
2 changed files with 26 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gallery_dl/extractor/urlgalleries.py
View File

@@ -42,7 +42,7 @@ class UrlgalleriesGalleryExtractor(GalleryExtractor):
for data["num"], img in enumerate(imgs, 1): for data["num"], img in enumerate(imgs, 1):
page = self.request(root + img).text page = self.request(root + img).text
url = text.extr(page, "window.location.href = '", "'") url = text.extr(page, "window.location.href = '", "'")
yield Message.Queue, url.partition("?")[0], data yield Message.Queue, url, data
def metadata(self, page): def metadata(self, page):
extr = text.extract_from(page) extr = text.extract_from(page)

31
test/results/urlgalleries.py
View File

@@ -14,9 +14,9 @@ __tests__ = (
"#class" : urlgalleries.UrlgalleriesGalleryExtractor, "#class" : urlgalleries.UrlgalleriesGalleryExtractor,
"#range" : "1-3", "#range" : "1-3",
"#results" : ( "#results" : (
"https://fappic.com/x207mqkn2463/4gq1yv.jpg", "https://fappic.com/x207mqkn2463/4gq1yv.jpg?gname=Clarice+window+8",
"https://fappic.com/q684ua2rp0j9/4gq1xv.jpg", "https://fappic.com/q684ua2rp0j9/4gq1xv.jpg?gname=Clarice+window+8",
"https://fappic.com/8vf3n8fgz9po/4gq1ya.jpg", "https://fappic.com/8vf3n8fgz9po/4gq1ya.jpg?gname=Clarice+window+8",
), ),
"blog" : "photos2q", "blog" : "photos2q",
@@ -33,9 +33,9 @@ __tests__ = (
"#class" : urlgalleries.UrlgalleriesGalleryExtractor, "#class" : urlgalleries.UrlgalleriesGalleryExtractor,
"#range" : "1-3", "#range" : "1-3",
"#results" : ( "#results" : (
"https://fappic.com/x207mqkn2463/4gq1yv.jpg", "https://fappic.com/x207mqkn2463/4gq1yv.jpg?gname=Clarice+window+8",
"https://fappic.com/q684ua2rp0j9/4gq1xv.jpg", "https://fappic.com/q684ua2rp0j9/4gq1xv.jpg?gname=Clarice+window+8",
"https://fappic.com/8vf3n8fgz9po/4gq1ya.jpg", "https://fappic.com/8vf3n8fgz9po/4gq1ya.jpg?gname=Clarice+window+8",
), ),
"blog" : "photos2q", "blog" : "photos2q",
@@ -65,4 +65,23 @@ __tests__ = (
"title" : "Angelika - Rustic Charm - AmourAngels 2016-09-27 7722lhrmys", "title" : "Angelika - Rustic Charm - AmourAngels 2016-09-27 7722lhrmys",
}, },
{
"#url" : "https://urlgalleries.net/b/xarchivesx/porn-gallery-6722560/caroline",
"#comment" : "image host URLs with query parameters (#7888)",
"#class" : urlgalleries.UrlgalleriesGalleryExtractor,
"#range" : "1-3",
"#results" : (
"http://img272.imagevenue.com/img.php?image=63353_qedf2jsd4j_123_376lo.jpg",
"http://img220.imagevenue.com/img.php?image=63140_hl2kkhv0n4_123_621lo.jpg",
"http://img217.imagevenue.com/img.php?image=63140_z2edqlkpkz_123_986lo.jpg",
),
"blog" : "The Archives Blog",
"count" : 141,
"date" : "dt:2016-06-11 00:00:00",
"gallery_id": "6722560",
"num" : range(1, 3),
"title" : "Caroline q5c75eeybn",
},
) )