Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Thumbor 7 #103
Thumbor 7 #103
Changes from 18 commits
51000af
87ba78c
354a14f
701c060
f5d9b72
2c08977
531483d
d0e0935
bd0a981
f8d4f41
0f4f151
0eb9938
2ddabaf
0c65988
5654566
117f688
4f5ed01
c241912
1e6bf20
6819c56
c7d48f0
d29863c
209ebe0
6a2a3b0
c1801a2
813cee4
d337145
c1dd14e
338f36a
13384d6
a1bc573
e0cb21e
7d244c9
b57595c
e520fa5
319e988
2dbff17
d2a44f7
971699d
da93faf
fe1fd8e
d334975
d461962
8a6381d
1196f56
263740c
4f63c79
945aba6
b6b95b5
5224474
e273779
53f564e
2112da5
5152050
f8610f5
a95da14
6e148cd
4a31dc4
39fe365
a54933e
f40ef14
5f59538
de1c5b4
5efb523
2e691f4
b1bba12
b5b0fe7
82ab38a
354ad6c
3e573cc
3247b6d
b402d76
ad2d62d
94715bc
1b5a064
4ec37b5
79441f1
35570f2
2a4b61b
10a70ac
9da0893
4e06a62
1cafbfd
8478102
5171f81
6987d1d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@heynemann I guess we still need some of the libraries here with thumbor 7 ? which ones? and which versions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created a new library called thumbor-aws we can use until tc-aws gets updated to python3. https://github.com/thumbor/thumbor-aws
The problem with urllib2 is that tc-aws is not python3 compliant. https://stackoverflow.com/questions/2792650/import-error-no-module-name-urllib2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so should we add thumbor-aws to requirements.txt ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also, correct me if I'm wrong, but wouldn't this require a change to the configs? e.g. if someone uses
LOADER=tc_aws.loaders.s3_loaders
they would need to change it? if so, is there a way to make thumbor-aws backwards compatible, so it accepts the "old" config directives?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, I meant as an additional lib to the docker image :) That way people who want can use it while tc-aws is not compatible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is :) will work on making it more compatible later...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great. What's the next step here then? I'm not sure I completely follow :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@heynemann is all this resolved since tc_aws 7.0.2 works with Thumbor 7?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated tc-aws to 7.0.2. There's a new thumbor 7.4.7 image on docker hub. Can you test it? ideally with tc-aws. I think this should solve the urllib2 issue.