-
Notifications
You must be signed in to change notification settings - Fork 48
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
deleting images #17
deleting images #17
Comments
I guess it could be added but there would need to be a way to authenticate users for that to happen |
Could a system with an app_key work? Which could also be required for uploading. |
I think authentication should be managed by the API or whatever service using imgpush. |
Yes, you are right. Maybe then add a feature flag in ENV to disable the deletion route, for users who haven't implemented authentication. |
Good idea ! I'm currently working on it, and will test if for some time.
For the authentication, I'm experimenting with Traefik (auth + block call from outside the stack) and will share the working configuration. |
could a feature be added for deleting existing images by their filename?
The text was updated successfully, but these errors were encountered: