-
Notifications
You must be signed in to change notification settings - Fork 24
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
it doesn't work anymore #22
Comments
It's working fine my side. 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. All the best, |
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. 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 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") 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 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 😄 |
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 |
@unaufhaltbar are you still experiencing crashes? If so , let us know what OS and version of python you are using |
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
The text was updated successfully, but these errors were encountered: