Skip to content

headers and Multi-platform fix#2

Open
vandad1901 wants to merge 11 commits into
mitchelljy:masterfrom
vandad1901:master
Open

headers and Multi-platform fix#2
vandad1901 wants to merge 11 commits into
mitchelljy:masterfrom
vandad1901:master

Conversation

@vandad1901

@vandad1901 vandad1901 commented Sep 7, 2021

Copy link
Copy Markdown

Hi. I downloaded this repo for personal use but found it to be broken. I fixed the code and also improved some aspects of it.

  • Fixed headers not being used in the image fetch request which
  • Switched to using pathlib for multi-platform use
  • Fixed file extensions always being .png
  • Switched to using faker for user-agent header
  • The code now respects keyboard interrupts and also doesn't swallow the stack trace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants