Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Move off of grequests #486

Closed
Closed

Description

Problem

The API currently uses the grequests library for handling some of the head requests it completes when returning results (specifically in validate_images.py).

Description

The README for the project says:

Note: You should probably use requests-threads or requests-futures instead.

The project has also not had a release since 2020. We might want to consider using one of those proposed alternatives.

Alternatives

Leave as is and hope for the best!

Additional context

Some more context in Sentry: https://sentry.io/share/issue/061ba99fc3df4c23bdb7643d337bbda0/

Implementation

  • 🙋 I would be interested in implementing this feature.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Labels

help wantedOpen to participation from the community✨ goal: improvementImprovement to an existing user-facing feature🐍 tech: pythonRequires familiarity with Python💻 aspect: codeConcerns the software code in the repository🟨 priority: mediumNot blocking but should be addressed soon

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions