-
Notifications
You must be signed in to change notification settings - Fork 0
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
wrc104 Handle S3 403 #5
Conversation
@mixmixmix - are we dropping py3.7 support? What have they done upstream with that? |
They moved to 3.12.... The docker image in this repo uses 3.7, but our image in fjelltopp-base-images uses 3.8. |
they changed the way how project is built among other things, but there are not changes in documentation - so it is unfinished & out of data in the upstream repo. |
Let's update the image in this repo to 3.8 and remove the 3.7 tests then? I think that should really be part of this PR so it doesn't just hang forever |
Why part of this PR? It was broken before and not sure there is any point in trying to tidy it up now. |
done! |
Adds handling of Access Denied errors from s3 backend and passes them back in a response.
Needed by https://github.com/fjelltopp/who-romania-ckan/issues/104
Testing
Tested with who-romania-ckan deployment, it is build as image v0.5.3c
Checklist
Put an
x
in the boxes that apply to this pull request (you can also fill these out after opening the pull request).You may not need to check all boxes.