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

[Instagram] skip analysis of private profiles? #3045

Closed
throwaway242685 opened this issue Oct 12, 2022 · 10 comments
Closed

[Instagram] skip analysis of private profiles? #3045

throwaway242685 opened this issue Oct 12, 2022 · 10 comments

Comments

@throwaway242685
Copy link

is there an option to skip the analysis of private Instagram accounts?

@mikf
Copy link
Owner

mikf commented Oct 13, 2022

No, there isn't.

Why? Is gallery-dl reporting a private profile even though you should be able to access its posts?

@throwaway242685
Copy link
Author

if you try to download a private Instagram profile with -o include=all, it will take several minutes to try and (inevitably) fail the download.

it looks like gallery-dl tries to analyze the URL multiple times, and doesn't recognize that it's a private URL with the first analysis.

maybe the first thing that gallery-dl should do is detect if an Instagram profile is private or not? and then proceed (or not) with the rest of the analysis.

and no, I don't have access to it, that's why I prefer to skip it.

@mikf
Copy link
Owner

mikf commented Oct 14, 2022

This feature has already been added in commit fce6642, I think.

You will get multiple [instagram][error] USER's posts are private messages, but they all (except /tagged) happen immediately and only cost 1 HTTP request in total.

@throwaway242685
Copy link
Author

oh, thank you!

@afterdelight
Copy link

pls update the program

@mikf
Copy link
Owner

mikf commented Oct 15, 2022

Some additional recommendation:
You really should not use -o include=all for Instagram, main reason being duplicate media. The files returned from /reels and /channel, which is this point is just another name for /reels, get returned with the main /posts feed as well, I think, but all 3 of those are part of include=all.

A better selection would be just include=stories,highlights,posts and maybe tagged.

@mikf mikf closed this as completed Oct 15, 2022
@throwaway242685
Copy link
Author

ohh... do posts also include the reels?

@mikf
Copy link
Owner

mikf commented Oct 15, 2022

Yes, from what I've seen reels are just the videos from posts.

I've just downloaded the first few files from https://www.instagram.com/instagram/reels/ and https://www.instagram.com/instagram/posts/ and everything from reels plus some extra was also downloaded with posts and all duplicate files are identical.

$ diff posts/ reels/ 
Only in posts/: 2941726983239585531_2941726415882424109.mp4
Only in posts/: 2941726983239585531_2941726422266056732.mp4
Only in posts/: 2941726983239585531_2941726594702450170.mp4
Only in posts/: 2941726983239585531_2941726602788904645.mp4
Only in posts/: 2941726983239585531_2941726608182985063.mp4
Only in posts/: 2941726983239585531_2941726614180702748.mp4
Only in posts/: 2946050323449731568_2946050316269243912.jpg
Only in posts/: 2946050323449731568_2946050316277528820.jpg
Only in posts/: 2946050323449731568_2946050316344729055.jpg
Only in posts/: 2946774028971265897_2946773225097226015.mp4
Only in posts/: 2946774028971265897_2946774025187907850.jpg
Only in posts/: 2946774028971265897_2946774025582272465.jpg
Only in posts/: 2947510331606204451_2947510324115138977.jpg
Only in posts/: 2947510331606204451_2947510324148581716.jpg
Only in posts/: 2947510331606204451_2947510324643547844.jpg

@throwaway242685
Copy link
Author

oh, that's good to know :o

thank you so much!

@afterdelight
Copy link

afterdelight commented Oct 16, 2022

Some additional recommendation: You really should not use -o include=all for Instagram, main reason being duplicate media. The files returned from /reels and /channel, which is this point is just another name for /reels, get returned with the main /posts feed as well, I think, but all 3 of those are part of include=all.

A better selection would be just include=stories,highlights,posts and maybe tagged.

Isnt channel used to be for IGTV?

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

No branches or pull requests

3 participants