Add Django 3.2 trove classifier

[jschneier]

No description provided.

[amureki]

@jschneier hey Josh!
I was debugging some issue with one of the latest Django releases (3.2.1) and noticed that django-storages is not yet have Django 3.2+ support (CI).

Seeing this PR issue - I wonder if those were some flaky problems with pip/wheel setup and recent cryptography releases.
I think, we might want to try to restart this build to see if it was resolved.

Is there anything I can assist you with to get this going?
Many thanks for maintaining this project!

Best,
Rust

[adamzap]

@jschneier Any update on Django v3.2 support? Is there any way your users can help push it forward? Thanks!

[jschneier]

@adamzap CI needs to work, we're dead in the water without it.

[jschneier]

CI is fixed but tests are broken by latest Django changes.

[adamzap]

I want to help, but I'm having trouble getting a development environment running for this project. Do you have instructions somewhere?

[allanlewis]

@adamzap Try tox -e py38-django32 --develop, you should just need Python 3.8 and tox for that to work. tox -a will list the available test configurations.

It seems that tests are failing on Django 2.2 as well as 3.2 😕

[mariajgrimaldi]

Hey, @adamzap @jschneier @amureki @allanlewis! I hope you're doing great!

I'm reaching out to help as part of the Open edX community effort to upgrade Django to Django 3.2 in the Open edX platform. That includes, among other things, updating its dependencies. Of course, django-storages is one of them. Here is the issue!

I'm thrilled to help you in whatever you need to finish this issue. Please let me know what's its status!

[adamzap]

@jschneier I was able to fix this, but then I realized that #1042 is already open. That's almost the same fix that I came up with.

Django fixed a security issue, and it changed which exception gets thrown in the failing code from ValueError to SuspiciousFileOperation. The only issue I see with #1042 is that it's asserting that SuspiciousOperation is raised instead of SuspiciousFileOperation, which is more specific.

So, I think if you get something like #1042 in and then get this PR in, Django v3.2 will be supported. I can open a new PR that does everything, if you want.

[jschneier]

Annoyingly, CI is not running because Travis no longer supports open source in the same way.

[adamzap]

@jschneier Are you interested in switching to GitHub Actions?

[jschneier]

@adamzap yup

[mariajgrimaldi]

Hi there! @adamzap @jschneier let me know if you need help!

[jschneier]

Please do!

…

On Monday, September 6, 2021, Maria Grimaldi ***@***.***> wrote: Hi there! @adamzap <https://github.com/adamzap> @jschneier <https://github.com/jschneier> can I help with this? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1005 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAREDWBYPFA32GWALL67UWLUASZ7FANCNFSM43HKF5DA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[mariajgrimaldi]

Hi there! @adamzap @jschneier

I opened this PR: #1051. Any suggestions are welcome!