Naming of config variables about flask-s3 HOT 6 CLOSED

👍

from flask-s3.

Unfortunately, this would break backwards compatibility, and as most users probably don't pin their dependency versions, it would break their things.

from flask-s3.

@SunDwarf how about throwing deprecation warnings, and falling back to old naming structure if new structure is not used? I'd be happy to spend some time over the weekend and send a pull request.
(I also have some variables in my config starting with S3_ but thankfully no conflicts!)

from flask-s3.

I was thinking of @bool-dev's approach as well (which is how Django handle's this kind of thing). Support both variable names, and show deprecation warnings until the next major release.

from flask-s3.

I think that's a better idea.

from flask-s3.

@SunDwarf We definitely can't just change the names of the config variables on everyone! ;)

from flask-s3.