Update repo + Docker #23
Conversation
tallen42
left a comment
tallen42
left a comment
There was a problem hiding this comment.
Left changes on specific files, but overall this is a great QOL upgrade to Jumpstart! Thank you so much :)
tallen42
left a comment
tallen42
left a comment
There was a problem hiding this comment.
Fire changes thank you so much! One last note w/ launching the app and we're good.
|
@tallen42 Should we do anything about newer versions of flask_limiter requiring a storage backend to store limits? https://flask-limiter.readthedocs.io/en/stable/#configuring-a-storage-backend Docs note that in-memory storage is for development only, but doesn't elaborate about why this is the case |
|
What are we storing in memory? |
|
Before updating flask_limiter, all rate-limiting data was stored in memory, but now flask_limiter moved to persistent storage methods like memcached, mongodb, redis, etc. It still supports in-memory storage, but says it shouldn't be used in production and gives no reason why. Not sure if it's just over complicating a simple issue or not. |
2 participants
All dependencies were updated to latest, project now supports latest python (3.13).
Added dockerfile and documentation.
Also fixed up readme and added additional info.