Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REQUEST] support xhamster.com alternative domain #31023

Closed
afterdelight opened this issue Jun 12, 2022 · 2 comments · May be fixed by #32579
Closed

[REQUEST] support xhamster.com alternative domain #31023

afterdelight opened this issue Jun 12, 2022 · 2 comments · May be fixed by #32579
Labels
site-support-request Add extractor(s) for a new domain

Comments

@afterdelight
Copy link
Contributor

Checklist

  • [x ] I'm reporting a new site support request
  • [x ] I've verified that I'm running youtube-dl version 2021.12.17
  • [x ] I've checked that all provided URLs are alive and playable in a browser
  • [x ] I've checked that none of provided URLs violate any copyrights
  • [ x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

single link:
https://xhday.com/videos/strapless-threesome-xhh7yVf
channle link:
https://xhday.com/users/mobhunter

Description

Add support for xhmaster alternative domain https://xhday.com/

@afterdelight afterdelight added the site-support-request Add extractor(s) for a new domain label Jun 12, 2022
@dirkf
Copy link
Contributor

dirkf commented Jun 12, 2022

This trivial patch does the job:

--- old/youtube_dl/extractor/xhamster.py
+++ new/youtube_dl/extractor/xhamster.py
@@ -23,7 +24,7 @@
 
 
 class XHamsterIE(InfoExtractor):
-    _DOMAINS = r'(?:xhamster\.(?:com|one|desi)|xhms\.pro|xhamster\d+\.com)'
+    _DOMAINS = r'(?:xhamster\.(?:com|one|desi)|xhms\.pro|xhamster\d+\.com|xhday\.com)'
     _VALID_URL = r'''(?x)
                     https?://
                         (?:.+?\.)?%s/

@dirkf dirkf closed this as completed in cc179df Jun 12, 2022
gaming-hacker added a commit to gaming-hacker/youtube-dl that referenced this issue Jun 12, 2022
* commit 'cc179df346abf34c8f77dbb221b839092007f20c':
  [XHamster] Support xhday.com alias, extract `uploader_id` * support xhday.com alias for xhamster.com (resolves ytdl-org#31023)   Authored by: dirkf * extract `uploader_id`:   from yt-dlp/yt-dlp@908b56e   (PR yt-dlp/yt-dlp#844)   Authored by: octotherp
  [utils, etc] Kill child processes when yt-dl is killed
  [YouTube] Support JSON3 subtitle format * subtitle tests updated to match
  [test] Fix workable subtitle tests (except YT) and mark others as skip, broken
  [test] Skip not _WORKING IE in subtitle tests; use unittest.skipTest throughout
github-actions bot added a commit to hellopony/youtube-dl that referenced this issue Jun 12, 2022
* https://github.com/ytdl-org/youtube-dl:
  [XHamster] Support xhday.com alias, extract `uploader_id` * support xhday.com alias for xhamster.com (resolves ytdl-org#31023)   Authored by: dirkf * extract `uploader_id`:   from yt-dlp/yt-dlp@908b56e   (PR yt-dlp/yt-dlp#844)   Authored by: octotherp
  [utils, etc] Kill child processes when yt-dl is killed
  [YouTube] Support JSON3 subtitle format * subtitle tests updated to match
  [test] Fix workable subtitle tests (except YT) and mark others as skip, broken
  [test] Skip not _WORKING IE in subtitle tests; use unittest.skipTest throughout
@afterdelight
Copy link
Contributor Author

ty man

dirkf added a commit to dirkf/youtube-dl that referenced this issue Jun 27, 2022
* support xhday.com alias for xhamster.com (resolves ytdl-org#31023)
  Authored by: dirkf
* extract `uploader_id`:
  from yt-dlp/yt-dlp@908b56e
  (PR yt-dlp/yt-dlp#844)
  Authored by: octotherp
alxlive pushed a commit to alxlive/youtube-dl that referenced this issue Feb 27, 2023
* support xhday.com alias for xhamster.com (resolves ytdl-org#31023)
  Authored by: dirkf
* extract `uploader_id`:
  from yt-dlp/yt-dlp@908b56e
  (PR yt-dlp/yt-dlp#844)
  Authored by: octotherp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-support-request Add extractor(s) for a new domain
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants