twayback icon indicating copy to clipboard operation
twayback copied to clipboard

it doesn't work anymore

Open unaufhaltbar opened this issue 1 year ago • 4 comments

once I launch the app it closes itself, it was working fine before the last update I tried downloading a pervious version and the same thing happened I tried it on 2 devices and it didn't work on any of them

unaufhaltbar avatar Aug 24 '22 02:08 unaufhaltbar

@unaufhaltbar

It's working fine my side. What devices are you on and using? Try maybe sending screenshots to @Mennaruuk

What Python version do you have? Do you have conflicting Py libraries with Twayback?

Delete the repository, git clone again, work in a virtual environment (venv) to avoid conflicting libs.

Just tested Wayback at this present moment. NUTR (Nothing Unusual to Report)

All the best,

C3n7ral051nt4g3ncy avatar Aug 24 '22 11:08 C3n7ral051nt4g3ncy

Hi, I found this repo from https://github.com/topics/help-wanted?l=python and was just perusing issues. I can't say that I understand the problem or how the script works; however, I had the same issue as @unaufhaltbar when running the command in the screenshot from @C3n7ral051nt4g3ncy.

image

I built a tiny docker image to isolate any portability issues.

from python:3.10

COPY . /app
RUN pip install -r /app/requirements.txt
ENTRYPOINT ["python", "/app/twayback.py"]

It seems like the issue is in the # list of just missing twitter url block.

missing_tweet_list = []
for result in results_list:
    if result[1] == 404:
        missing_tweet_list.append(str(result[0]))
    if result[1] == 429:
        print("Respose Code 429: Too Many Requests. Your traffic to Twitter is being limited and results of this script will not be accurate")

image

Those results were getting a 200 code which wasn't being handled. Modifying this section to be the following returns results, but I don't know if it's the desired solution. It definitely feels wrong. Also, the new result only returns 5 tweets when the original find was 6 tweets 😕

missing_tweet_list = []
for result in results_list:
    if result[1] == 404:
        missing_tweet_list.append(str(result[0]))
    if result[1] == 200:
        missing_tweet_list.append(str(result[0]))
    if result[1] == 429:
        print("Respose Code 429: Too Many Requests. Your traffic to Twitter is being limited and results of this script will not be accurate")

Just leaving this here in case anyone stumbles here again 😄

image

nickumia avatar Oct 09 '22 03:10 nickumia

Thanks for the comment @nickumia . We don't want to capture the urls with 200 codes. We're interested in the URLs that are returning a 404 code as that would mean they were once there but are not there any longer

humandecoded avatar Oct 14 '22 14:10 humandecoded

@unaufhaltbar are you still experiencing crashes? If so , let us know what OS and version of python you are using

humandecoded avatar Oct 14 '22 23:10 humandecoded