Skip to content
This repository has been archived by the owner on Sep 28, 2020. It is now read-only.

Changes to DownloadUrlResolver #333

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

anilgit90
Copy link

  1. Changed signature resolving.
  2. Changed player regex url resolver.

1. Changed signature resolving.
2. Changed player regex url resolver.
@ac-lap
Copy link

ac-lap commented Dec 18, 2018

Nice job dude! Will try and code-review/test your changes sometime next week or post Christmas.
Given the main repo is not being actively maintained, let's see if we can maintain your forked repo.

1. Added Progress bar to update the download progress of the video.
2. Updated readme to include feature list of Sample GUI.
@HelgeStiemerling
Copy link

Hello,

self problem with: https://www.youtube.com/watch?v=G4mlA65wsbE

public static string DecipherWithVersion(string cipher, string cipherVersion)
{
string jsUrl = string.Format("http://s.ytimg.com/yts/jsbin/player-{0}.js", cipherVersion);

        string js = HttpHelper.DownloadString(jsUrl);

404 file not found jsUrl

ex

1. Changed Decipherer.cs to Include changes to Youtube Player Url.
2. Updated Readme to include badge in new line.
Added Analysis of code by BetterCode.
@rcabreu
Copy link

rcabreu commented Jan 29, 2019

This did not download https://www.youtube.com/watch?v=7rwW1ErcVu8

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants