search

dlr-eoc / prosEO

prosEO – A Processing System for Earth Observation Data
GNU General Public License v3.0
14 stars 1 forks source link

GUI: Caching of drop-down lists not working correctly #128

Closed tangobravo62 closed 3 years ago

tangobravo62 commented 3 years ago

If the GUI is used concurrently, one user may overwrite another user's cached list of product classes, processors etc. due to missing thread separation. This may lead to the GUI for a user logged in to mission A to show product classes from mission B.

Workaround: Disable caching (implemented).

Permanent solution: Add thread separation for instance variables in all GUI classes, and check for correct user/mission before use of cached data.

Note: Caching data may improve speed, but contradicts the principle of stateless behaviour.

emelchinger commented 3 years ago

Now the cached data is attached to the logged in user.

tangobravo62 commented 3 years ago

Tested succesfully.