search

ciderpunx / twitrssme

TwitRSS.me: Tool to make Twittter timelines and searches into RSS feeds
https://twitrss.me
GNU General Public License v2.0
359 stars 59 forks source link

Enable cookies because Twitter now requires them. #76

Closed dperelman closed 4 years ago

dperelman commented 4 years ago

I believe this fixes #75. At least, Twitter also broke for me a few days ago and this change fixed it.

I discovered the issue by changing

    err('Can’t screenscrape Twitter',404);

to

    err(Dumper($response) . 'Can’t screenscrape Twitter',404);

and seeing that the issue was a redirect loop to the same URL. This StackOverflow answer mentioned that redirect loops are sometimes used to set and check cookies and the LWP::UserAgent docs on cookies gave the code for the minimal way to enable cookies.

j-f1 commented 4 years ago

Any update on getting this merged @ciderpunx?

ciderpunx commented 4 years ago

Sorry folks, I've been AFK for a while.

Thanks for the patch, merging now