Skip to content

Fix embed ID extraction and update request headers#277

Merged
mhdzumair merged 1 commit into
mhdzumair:mainfrom
GLlgGL:patch-3
May 17, 2026
Merged

Fix embed ID extraction and update request headers#277
mhdzumair merged 1 commit into
mhdzumair:mainfrom
GLlgGL:patch-3

Conversation

@GLlgGL

@GLlgGL GLlgGL commented May 12, 2026

Copy link
Copy Markdown
Contributor

https://vidmoly.biz/embed-qu2swicnn9j6.html

tested working

Summary by CodeRabbit

  • Bug Fixes
    • Improved video extraction and playback from Vidmoly sources with enhanced stream URL normalization, optimized request preparation, and better compatibility with platform security requirements. These changes ensure more reliable video delivery and consistent performance when accessing Vidmoly content through the service.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 12, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: beaa8bb8-1c12-49e3-9c4d-9ec1420beafb

📥 Commits

Reviewing files that changed from the base of the PR and between 0ee1fac and 8e6209f.

📒 Files selected for processing (1)
  • mediaflow_proxy/extractors/vidmoly.py

Walkthrough

The VidmolyExtractor now derives an embed_id from the URL path and uses it to configure Cloudflare turnstile cookies and enhanced Sec-Fetch headers in outgoing requests. After extracting the master playlist URL from the embed HTML, the extractor normalizes relative URLs to absolute form via urljoin.

Changes

Vidmoly Extractor Enhancements

Layer / File(s) Summary
Embed ID extraction and request header configuration
mediaflow_proxy/extractors/vidmoly.py
ExtractorError import added. URL-path embed_id is extracted via regex and validated. Request headers are expanded to include a Cloudflare turnstile cookie (cf_turnstile_demo_pass_{embed_id}=1) and multiple Sec-Fetch-* header values.
Stream URL normalization
mediaflow_proxy/extractors/vidmoly.py
Extracted master_url is normalized to an absolute URL using urljoin when it does not already start with http.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • mhdzumair/mediaflow-proxy#169: Both PRs modify the same VidmolyExtractor with overlapping changes to request headers, master URL extraction, and stream URL resolution.
  • mhdzumair/mediaflow-proxy#217: Both PRs update mediaflow_proxy/extractors/vidmoly.py and affect how the Vidmoly extractor obtains and processes the master playlist URL.
  • mhdzumair/mediaflow-proxy#195: Both PRs modify mediaflow_proxy/extractors/vidmoly.py with overlapping code-level changes to request headers and master_url normalization.

Poem

🐰 A turnstile cookie, embed IDs gleam,
Security headers flow like a stream,
URLs now absolute, no paths astray,
Vidmoly extractor hops out to play! 🎬

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly relates to the main changes: extracting embed IDs from URLs and updating request headers with Cloudflare Turnstile cookies and Sec-Fetch headers.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mhdzumair mhdzumair merged commit a6fc2a6 into mhdzumair:main May 17, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants