Skip to content
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

chore: added opencv-python-headless to requirements.txt #148

Merged

Conversation

RaphaelVRossi
Copy link
Contributor

As reported here, there is a need to pin opencv-python-headless version to 4.8.1.78

opencv error

@gingerlime
Copy link
Contributor

Thank you @RaphaelVRossi. What tool did you use to scan the image? I wonder if it's something that can be incorporated into Github actions or something?

I'm totally fine merging this, but shouldn't this be fixed by upstream? or simply rebuilding the image would pick the latest non-vulnerable version? we already have a bunch of explicit versions in requirements.txt but not really sure if they're absolutely needed and maintaining more and more over time can be a, erm, chore :) what do you think?

@gingerlime gingerlime merged commit 9dff63b into MinimalCompact:master Nov 17, 2023
1 check passed
@RaphaelVRossi
Copy link
Contributor Author

Hey @dpramani, can you help here please?!

@RaphaelVRossi
Copy link
Contributor Author

@gingerlime this is the tool they used to test https://app.snyk.io/

I agree with you! We just need to have all mandatory dependencies, eg. Thumbor / Thumbor-wand

lachesis pushed a commit to zincio/docker-thumbor that referenced this pull request Feb 26, 2024
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