EltonChou / TwitterMediaHarvest

Download twitter media with only one-click.
https://chrome.google.com/webstore/detail/media-harvest-twitter-med/hpcgabhdlnapolkkjpejieegfpehfdok
MIT License
374 stars 32 forks source link

[BUG] unable to download "Forbidden Your login session might be expired" #149

Closed MinamikoAAAA closed 1 month ago

MinamikoAAAA commented 1 month ago

After the 4.2.5 update, the download button shows up again. However, the extension is still unable to download any media. The console warning is shown below: error Log file: x.com-1715903685086.log Occasionally, a pop-up window will show the below message: error2 Clicking the "View" button will copy the current tab, and the download button remains broken.

Chrome 125.0.6422.61, I'm living in New Zealand

EltonChou commented 1 month ago

Can you go to options page's console to check Network and Console tab ? I think they may deprecate some api endpoints.

Remember to mask the sensitive data.

brgm0 commented 1 month ago

The same symptoms occur on mac Chrome. Cannot download. . .

MinamikoAAAA commented 1 month ago

Can you go to options page's console to check Network and Console tab ? I think they may deprecate some api endpoints.

Remember to mask the sensitive data.

I don't understand your instructions completely... Could you specify the "option page" for me?

EltonChou commented 1 month ago

I think I have found the root cause. So you don't need to do this.

FYI image

I don't understand your instructions completely... Could you specify the "option page" for me?

MinamikoAAAA commented 1 month ago

v4.2.6 works now. Thank you very much!

globalgoldgetter commented 1 month ago

I think I have found the root cause. So you don't need to do this.

FYI image

I don't understand your instructions completely... Could you specify the "option page" for me?

Hi, I'm still unclear about what are we to do once checking the settings. I dont see any "Network and Console tab." Also, you mentioned: " I think I have found the root cause. So you don't need to do this." What is the fix to download files again?

Thanks!

Screen Shot 2024-05-19 at 1 09 12 AM

MinamikoAAAA commented 1 month ago

I think I have found the root cause. So you don't need to do this. FYI image

I don't understand your instructions completely... Could you specify the "option page" for me?

Hi, I'm still unclear about what are we to do once checking the settings. I dont see any "Network and Console tab." Also, you mentioned: " I think I have found the root cause. So you don't need to do this." What is the fix to download files again?

Thanks!

Screen Shot 2024-05-19 at 1 09 12 AM

the V4.2.6 update is available in the "Releases" session. Go download it and check if it solves the issue.

globalgoldgetter commented 1 month ago

Working perfectly after downloading the update. Is there an option to download the title of the file posted?

Thanks!

EltonChou commented 1 month ago

Working perfectly after downloading the update. Is there an option to download the title of the file posted?

Thanks!

If you can defined what is a meaningful title in all situations.

globalgoldgetter commented 1 month ago

Working perfectly after downloading the update. Is there an option to download the title of the file posted? Thanks!

If you can defined what is a meaningful title in all situations.

Yes, for instance see the attached images. The area I have circled. Screen Shot 2024-05-19 at 9 44 10 AM Screen Shot 2024-05-19 at 9 46 31 AM

globalgoldgetter commented 1 month ago

Doh! I forget you developers speak differently . Just realized you meant something else when you replied "If you can defined what is a meaningful title in all situations."

Meaning what? How would one define "what is a meaningful title in all situations?" And where specifically in the settings?

Thanks

EltonChou commented 1 month ago

If you want the title to be in filename, you will need to solve these problems.

  1. Not all characters can be used in filename.
  2. If the character is invalid in filename, how do you process it? Remove or swap?
  3. Tweet's content length could be up to 4000 characters. Is it possible to be included in filename?

Another possible solution is storing the content in Indexed DB like history data. When you need it, just search it. But the performance would not be good. If you want to improve performance, I will need to host a service to serve those data and it will not be a free feature.

0x2E16CF0F commented 1 month ago

On the most recent release for Firefox, the same problem occurs when using Multi-Account Containers. image

There is no error or warning logged by the Dev Tools when this happens, as I think that the issue happens outside of the container Twitter is running in.

For example, https://github.com/Flkalas/TwitterMediaAssist 's extension does work with Multi-Account Containers.

EltonChou commented 1 month ago

On the most recent release for Firefox, the same problem occurs when using Multi-Account Containers. There is no error or warning logged by the Dev Tools when this happens, as I think that the issue happens outside of the container Twitter is running in.

For example, https://github.com/Flkalas/TwitterMediaAssist 's extension does work with Multi-Account Containers.

https://github.com/EltonChou/TwitterMediaHarvest/issues/99#issuecomment-1713755005

EltonChou commented 1 month ago

Should be fixed in v4.2.6