Skip to content
This repository was archived by the owner on Apr 19, 2022. It is now read-only.

player respond should go correct method#6

Merged
LinPolly merged 2 commits intoLinPolly:masterfrom
HFAsif:patch-4
Jun 24, 2021
Merged

player respond should go correct method#6
LinPolly merged 2 commits intoLinPolly:masterfrom
HFAsif:patch-4

Conversation

@HFAsif
Copy link

@HFAsif HFAsif commented Jun 23, 2021

so i used a try catch to go second method ,

HFAsif added 2 commits June 24, 2021 03:36
so i used a try catch to go second method ,
@LinPolly LinPolly merged commit b07db42 into LinPolly:master Jun 24, 2021
@HFAsif
Copy link
Author

HFAsif commented Jun 24, 2021

i have changed this (@"""jsUrl""\s*:\s*""([^""]+)"""); buy following your fix
and JObject.Parse also changed by following but still same any video cant load exception at same please , but try catch only works to skip this stepp
i really want ur suggession and fix

LinPolly added a commit that referenced this pull request Jun 24, 2021
This reverts commit b07db42, reversing
changes made to 34a99d4.
@LinPolly
Copy link
Owner

I refer to the content of youtube-dl, but I cannot search for ytplayer.config.
In fact, the ytplayer.config js syntax is the same as the next Regex answer after execution, so delete the Regex of ytplayer.config.

@HFAsif
Copy link
Author

HFAsif commented Jun 24, 2021

delete dataRegex = new Regex(@"ytplayer.config\s*=\s*({.+?});", RegexOptions.Multiline);? ?

@HFAsif
Copy link
Author

HFAsif commented Jun 24, 2021

ahh you mean remove this code , yes it works without this code
"
//if (dataMatch.Success)
//{
// string extractedJson = dataMatch.Result("$1");
// if (!extractedJson.Contains("raw_player_response:ytInitialPlayerResponse")) //https://www.youtube.com/watch?v=9Y7TRMISkGE
// {
// player_response = Newtonsoft.Json.Linq.JObject.Parse(extractedJson)["args"]["player_response"].ToString();
// return YoutubeModel.FromJson(player_response);
// }

        //    }
        //}

"

??

yes remove this code now works , but i think if yt some video use this resonce to see video ,
thank you

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.

2 participants