search

euphorie / Euphorie

Euphorie is a tool for risk assessment
https://pythonhosted.org/Euphorie/
GNU General Public License v2.0
12 stars 6 forks source link

Notifications: Don't fail on data inconsistency when retrieving the s… #676

Closed thet closed 10 months ago

thet commented 10 months ago

…ession url.

ale-rt commented 10 months ago

I still do get why all of a sudden we have this unconsistency... Is it a possibility that we have session that do not resolve a proper path?

thet commented 10 months ago

@ale-rt Manuel (/cc @reinhardt ) mentioned that we imported the production data at one point over to existing staging data and did no cleanup. This could be related.

thet commented 10 months ago

@reinhardt I now changed handling of missing titles from:

Screenshot from 2023-12-01 16-06-44

to:

Screenshot from 2023-12-01 16-12-03

reinhardt commented 10 months ago

I still do get why all of a sudden we have this unconsistency... Is it a possibility that we have session that do not resolve a proper path?

Apparently we usually filter out sessions that don't correspond to a proper ZODB path: https://github.com/euphorie/Euphorie/blob/main/src/euphorie/client/model.py#L1237

That may be a reason we usually don't notice inconsistencies like this.