bot accounts can expire

[TurBoss]

could this time be extended to maybe 1 year?

thanks

[abma]

weird, from what i read here, bots should be kept forever:

uberserver/SQLUsers.py

Line 692 in e63fee4

response = self.sess().query(User).filter(User.ingame_time == 0).filter(User.last_login < now - timedelta(days=28)).filter(User.bot == 0).filter(User.access == "user")

[abma]

are you sure the user account had the botflag set?

[TurBoss]

Hello,

this account was created with the CREATEBOTACCOUNT command

[TurBoss]

found this

uberserver/SQLUsers.py

Line 710 in e63fee4

# remove botflags from clients that didn't log in for 3 months:

[abma]

hm, not sure how cleanly solve this.

i guess there needs to be a "task" which checks which users are online and write this to db.

the cleanup / audit thing then should also take this into account to ignore users which where recently online.

[abma]

as springrts.com server is automaticly rebooted after a (automatic) kernel update, IMHO 365 days should be enough to never be hit.

i leave this open, as there is only a workarround active

[silentwings]

My dev envir is long gone, but imo the best way to fix this bug (which was my fault) would be to exclude logged in users from the effect of audit_access. Iirc can check if the user(name) is online within the for loops and continue if so.

[abma]

My dev envir is long gone, but imo the best way to fix this bug (which was my fault) would be to exclude logged in users from the effect of audit_access. Iirc can check if the user(name) is online within the for loops and continue if so.

i guess thats not enough: IMHO a correct fix would be a task which is run by timer (daily) which updates last_login (or a new attribute like last_online).

i.e. think about the server going offline, server is going up again, the audit task is run. (the risk is pretty low as long as the host of uberserver is restarted because of kernel updates)

[silentwings]

Hm, you are right

uberserver/server.py

Line 59 in cc4155c

clean_loop = task.LoopingCall(_root.scheduled_clean)

will execute the looping call the first time during boot.