Forum OpenACS Development: sec_lookup_property.update_last_hit_dml

Request notifications

Hi

we've been looking at some issues with sessions and I noticed that the update statement in sec_lookup_property.update_last_hit_dml seems to be missing the module. See http://cvs.openacs.org/browse/OpenACS/openacs-4/packages/acs-tcl/tcl/security-procs.xql?r=1.10

It's currently:
update sec_session_properties
set last_hit = :new_last_hit
where session_id = :id and
property_name = :name

but I'm wondering, shouldn't it be:
update sec_session_properties
set last_hit = :new_last_hit
where session_id = :id and
module = :module and
property_name = :name

Thoughts?