pallets-eco / flask-security-3.0

Quick and simple security for Flask applications
MIT License
1.63k stars 512 forks source link

Flask-Security + Flask-Dynamo #341

Open rdegges opened 9 years ago

rdegges commented 9 years ago

Heyo,

I'm the maintainer / author of Flask-Dynamo, which adds dynamodb support to Flask on top of boto. I'm wondering if you'd be cool to accept a PR with me merging in a datastore / docs for using Dynamo?

I figured it's a fairly popular database nowadays, and I've had requests for it from my users -- so if you'd be up for merging I'd be happy to put in the work to do a clean integration.

Let me know! ^^

Also: when are ya gonna be in the bay area?! Let's hang out >:D

mattupstate commented 9 years ago

@rdegges hey boss! I'm happy to accept a PR for this support.

rdegges commented 9 years ago

Awesome, thanks ^^ I'll get right on it >:)

vladfr commented 6 years ago

@rdegges @mattupstate Hey guys! I'm interested in taking this over. Did anyone submit anything on Dynamo support?

rdegges commented 6 years ago

I did not. I had other distractions and totally forgot about this :(

jirikuncar commented 6 years ago

@vladfr feel free to submit a PR.

vladfr commented 6 years ago

@jirikuncar I looked into this. The problem is that flask_security wants to use an ORM, and flask-dynamo is more of a wrapper - it doesn't put entries from dynamo into object properties. Which means more work would be needed, either in flask-dynamo, or use some other orm/adapter for Dynamo.

Right now, that's a bigger ask than I have time for. Anyone up for pairing on this, ping me.