CMUEberlyCenter / eberly-docuscope-wa

Other
0 stars 0 forks source link

Bad caching of configuration file. #45

Closed Ringenberg closed 6 months ago

Ringenberg commented 10 months ago

The current configuration caching uses the version string to check for equivalence. This does not work if multiple files have the same version (eg) using date for version number and uploading multiple on the same day.

It is obvious that relying on the user to generate unique non-colliding version numbers is unreasonable.

Also it looks like things are getting stored in local storage instead of session storage as the naming of variable suggests.

Ringenberg commented 10 months ago

de81c1b01ad9f0c650bfb3d68c1f57b4e3cc7f2b