certtools / intelmq-api

FastAPI-based API for the IntelMQ project
https://docs.intelmq.org/latest/user/api/
1 stars 7 forks source link

feature request: sessions in the intelmq-api #42

Open aaronkaplan opened 1 year ago

aaronkaplan commented 1 year ago

Hi,

I would like to suggest that we move the sessions from sqlite to redis. What do you think about this idea?

sebix commented 1 year ago

Then system administrators should never flush the complete redis DB to reset IntelMQ's caches and bots, as that would also delete the user accounts. May have unintended consequences. We should document that well.

aaronkaplan commented 1 year ago

Okay. That’s a good point then. ---MobileOn 02.02.2023, at 11:37, Sebastian @.***> wrote: Then system administrators should never flush the complete redis DB to reset IntelMQ's caches and bots, as that would also delete the user accounts. May have unintended consequences. We should document that well.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

kamil-certat commented 1 year ago

Sessio management can be in redis, but user management need to be held separated. Anyway, I would suggest redis or sqlite as two possibilities, it's such a small API, that I don't think it makes sense to push people to do the change.