search

camac-suisse / ansible-role-camac

3 stars 11 forks source link

fix(dms): reduce oidc bearer revalidation time #41

Closed czosel closed 5 years ago

czosel commented 5 years ago

Longer caching causes issues with the document merge service when roles are switched (incorrect templates being returned)

sliverc commented 5 years ago

Not sure this is a good idea... this might be happening a lot during testing but not in production... and a logout login should solve this as well - and I think to resolve Camac caching its also a good idea to logout anyway.

czosel commented 5 years ago

The issue would occur a lot in production because users regularly switch the currently active group.

sliverc commented 5 years ago

I see that's the problem.... hmm maybe better to split up the caching in document merge service to make token and group caching time in two vars.

czosel commented 5 years ago

Good idea :+1:

sliverc commented 5 years ago

see https://github.com/adfinis-sygroup/document-merge-service/pull/155

once this is merged this PR I think would be obsolete