Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

db sessions #290

Open
haxmeadroom opened this issue Sep 1, 2018 · 1 comment
Open

db sessions #290

haxmeadroom opened this issue Sep 1, 2018 · 1 comment
Labels
db SL2 Database stuff help wanted low priority

Comments

@haxmeadroom
Copy link

haxmeadroom commented Sep 1, 2018 •
edited
Loading

I haven't noticed a threading issue lately, I'm still worried about db sessions. @ehennenfent suggested a zope sessions manager that might work with sqlalchemy so wouldn't hurt to give that a try.
@haxmeadroom haxmeadroom added the db SL2 Database stuff label Sep 1, 2018
@woodruffw
Copy link
Member

Ran into a threading issue with the winchecksec stuff, but was able to mitigate it by doing some session cleanup and explicitly telling the DB that we don't care about expired Checksec instances. Doing this more consistently with a real session manager would still probably be best.

@woodruffw woodruffw added this to the Polish milestone Oct 1, 2018
@woodruffw woodruffw removed this from the Polish milestone Nov 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
db SL2 Database stuff help wanted low priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@woodruffw @haxmeadroom