DIRACGrid / diracx

The neXt DIRAC incarnation
GNU General Public License v3.0
9 stars 20 forks source link

feat: support refresh token #45

Closed aldbr closed 1 year ago

aldbr commented 1 year ago

This PR aims at solving https://github.com/DIRACGrid/diracx/issues/24

We also added a few tests:

Questions:

Refresh token rotation: what if we commit while the revocation fails?

This does not happen, we first revoke the token:

What should happen if a user does the device_code flow multiple times in a row to get many pairs of tokens. Should we revoke the previous RT each time an interactive flow is triggered?

I would say that we should keep them all valid as they can be used for different purposes.