Skip to content

Replace imghdr image validation#69

Draft
Mirochill wants to merge 1 commit into
YoongiKim:masterfrom
Mirochill:fix-67-remove-imghdr
Draft

Replace imghdr image validation#69
Mirochill wants to merge 1 commit into
YoongiKim:masterfrom
Mirochill:fix-67-remove-imghdr

Conversation

@Mirochill

Copy link
Copy Markdown

Fixes #67.

Summary:

  • remove the imghdr import from main.py
  • validate downloaded image files using local header checks for common image formats
  • preserve the existing None result for unreadable files so the caller still removes invalid downloads

Validation:

  • git diff --check HEAD~1..HEAD
  • Static search confirmed imghdr is no longer imported or used.
  • Not run locally.

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.

how to solve this problem?

1 participant