YouTube-operational-API icon indicating copy to clipboard operation
YouTube-operational-API copied to clipboard

Merge `protobuf_and_authentication_for_community_post_poll_vote_details_retrieval` branch

Open Benjamin-Loison opened this issue 1 year ago • 3 comments

protobuf_and_authentication_for_community_post_poll_vote_details_retrieval.

As someone asked on Discord if these changes were deployed on the official instance.

Let us investigate the Windows case:

https://community.chocolatey.org/packages/protoc https://github.com/protocolbuffers/protobuf/releases/latest

https://community.chocolatey.org/packages/composer https://github.com/composer/windows-setup/releases/latest

 git clone https://github.com/Benjamin-Loison/YouTube-operational-API --branch protobuf_and_authentication_for_community_post_poll_vote_details_retrieval

https://www.youtube.com/channel/UCQxJsAlqmBPAbR_0syDi9mg/community?lb=UgwSoAm2bGLaJM44UTZ4AaABCQ

Benjamin-Loison avatar Apr 11 '24 13:04 Benjamin-Loison

Facing similar issue as #268:

grep -r '\${'
community.php:    return array_map(fn($k, $v) => "${k}${separator}${v}", array_keys($anArray), array_values($anArray));
community.php:    $SAPISIDHASH = "${currentTime}_" . sha1("$currentTime $SAPISID $ORIGIN");

Benjamin-Loison avatar Apr 12 '24 12:04 Benjamin-Loison

Should leverage includeOnceProto{,s} before merging:

    function includeOnceProto($proto) {
        $COMMON_PATH = 'proto/php';
        include_once "$COMMON_PATH/$proto.php";
        include_once "$COMMON_PATH/GPBMetadata/$proto.php";
    }
    
    function includeOnceProtos($protos) {
        foreach($protos as $proto) {
            includeOnceProto($proto);
        }   
    }

Benjamin-Loison avatar Jul 05 '24 19:07 Benjamin-Loison

git branch -l
  get_video_endscreen
* main
  meta
  protobuf_and_authentication_for_community_post_poll_vote_details_retrieval
  provide_google_abuse_exemption_cookie

Benjamin-Loison avatar Sep 15 '24 13:09 Benjamin-Loison

git branch -r
  origin/HEAD -> origin/main
  origin/get_video_endscreen
  origin/main
  origin/meta
  origin/protobuf_and_authentication_for_community_post_poll_vote_details_retrieval
  origin/provide_google_abuse_exemption_cookie

Benjamin-Loison avatar Sep 15 '24 14:09 Benjamin-Loison

The Stack Overflow question 38206196 does not clearly help, I would like to append commits of the development branch.

Benjamin-Loison avatar Sep 15 '24 14:09 Benjamin-Loison

Let us cherry-pick one commit at a time.

git cherry-pick 91ca212068c80e3ee82240263793f25ba7764ddb
Auto-merging README.md
CONFLICT (content): Merge conflict in README.md
error: could not apply 91ca212... Correct Windows `protoc` command
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

Maybe it is because I forced pushed at the time or was not synchronized locally and remotely and pushed a branch as is.

Benjamin-Loison avatar Sep 15 '24 14:09 Benjamin-Loison

I confirm today pushed commits to correctly be able to load voteRatios locally.

Benjamin-Loison avatar Sep 15 '24 14:09 Benjamin-Loison

On the official instance voteRatio values are null, do not investigate until someone complains, as the only user leveraging this feature in theory uses his own instance.

Benjamin-Loison avatar Sep 15 '24 14:09 Benjamin-Loison