[instagram] Add support for stories #371
Conversation
Add support for Instagram user's stories (https://www.instagram.com/stories/<username>/). First the shared_data in instagram.com/stories/<username> is fetched in order to retrieve the user_id that is then passed to fetch the stories via the corresponding graphql query. Please note that fetching stories is supported only when authentication is enabled and the corresponding <username> is followed.
| # If there are stories present but the user is not authenticated or | ||
| # does not have permissions no stories are returned. | ||
| if not shared_data['data']['reels_media']: | ||
| return [] # no stories present |
There was a problem hiding this comment.
Maybe add a warning as to why there aren't any stories?
There was a problem hiding this comment.
Good question! What other extractors are doing in a similar situations? Actually by using the web browser what happens is that no stories could be seen and a «Log in to view this story» form is showed.
There was a problem hiding this comment.
Hmm, now that you mention it: most other extractors also don't show any warnings etc. in such situations, even though they should. I guess it's OK to leave it like that for right now, and I'll go and add some more "authentication required" error messages to all other extractors.
Simplify handling of typename in InstagramExtractor.items() and multi-line string in _extract_stories(). NFCI.
|
Hey, sorry for popping in here, but I thought it would be best to just ask real quick before opening a new [instagram] Add support for IGTV issue in a more formal manner. What are your thoughts on IGTV support? Which is basically video, but not exactly the same video you would find on a normal Instagram feed (i.e. a standard video post). Don't ask me about the idea behind this, I have no clue, and it's not immediately obvious to me. But, well, it's there.. I think here's the official announcement: And from there, an example account on Instagram: Which has a IGTV section (right next to Posts): It seems like there is only one entry though, and the link format for such an IGTV "post" looks like this: |
|
Hello HRXN,
HRXN writes:
Hey, sorry for popping in here, but I thought it would be best to just ask real quick before opening a new *[instagram] Add support for IGTV* issue in a more formal manner.
What are your thoughts on IGTV support? Which is basically video, but not exactly the same video you would find on a normal Instagram feed (i.e. a standard video post). Don't ask me about the idea behind this, I have no clue, and it's not immediately obvious to me. But, well, it's *there*..
I think here's the official announcement:
https://instagram-press.com/blog/2018/06/20/welcome-to-igtv/
And from there, an example account on Instagram:
https://www.instagram.com/laurdiy/
Which has a IGTV section (right next to *Posts*):
https://www.instagram.com/laurdiy/channel/
It seems like there is only one entry though, and the link format for such an IGTV "post" looks like this:
https://www.instagram.com/tv/BkQL48uB_cV/
Sorry for the delay but I have inspected and mitmproxy-ed them only
now.
It seems that all the logic is already in gallery-dl and it needs
only few minor adjustments. I'll share a pull request adding
support for them in the next few minutes!
|
|
Leonardo Taccari writes:
It seems that all the logic is already in gallery-dl and it needs
only few minor adjustments. I'll share a pull request adding
support for them in the next few minutes!
I have just shared them via:
<#373>
|
|
@iamleot Don't worry about any delays, just take all the time you need.. 😄 |
Add support for Instagram user's stories
(https://www.instagram.com/stories//).
First the shared_data in instagram.com/stories/ is fetched in
order to retrieve the user_id that is then passed to fetch the stories
via the corresponding graphql query.
Please note that fetching stories is supported only when authentication
is enabled and the corresponding is followed.