[9.0] DISET version of the TokenManager service

[atsareg]

The DISET version of the TokenManager service is added. This will help to eventually get rid of all the Tornado services.

BEGINRELEASENOTES

*Framework
NEW: DISET version of the TokenManager service

ENDRELEASENOTES

[andresailer]

Why not put this into 8.0? The TokenManager is the only tornado service I have

[andresailer]

Suggested change

	result = Registry.getIDFromDN(dn)
	if result["OK"]:
	uid = result["Value"]
	# To do this, first find the refresh token stored in the database with the maximum scope
	result = self.__tokenDB.getTokenForUserProvider(uid, idpObj.name)
	if result["OK"] and result["Value"]:
	tokens = result["Value"]
	result = self.__checkProperties(dn, userGroup)
	if result["OK"]:
	# refresh token with requested scope
	result = idpObj.refreshToken(tokens.get("refresh_token"), group=userGroup, scope=scope)
	if result["OK"]:
	return result
	result = Registry.getIDFromDN(dn)
	if not result["OK"]:
	continue
	uid = result["Value"]
	# To do this, first find the refresh token stored in the database with the maximum scope
	result = self.__tokenDB.getTokenForUserProvider(uid, idpObj.name)
	if not result["OK"] or not result["Value"]:
	continue
	tokens = result["Value"]
	result = self.__checkProperties(dn, userGroup)
	if not result["OK"]:
	continue
	# refresh token with requested scope
	result = idpObj.refreshToken(tokens.get("refresh_token"), group=userGroup, scope=scope)
	if result["OK"]:
	return result

[andresailer]

PS: Shouldn't we first result = self.__checkProperties(dn, userGroup) instead of getting getTokenForUserProvider ? Seems we can do that checkProperties before?

[fstagni]

For each and every service for which we have the DIPS and HTTPs version, we have 2 files:

• a "NameOfHandler.py". This contains

class NameOfHandlerMixin:
  # logic end "export_"s here

class NameOfHandler(NotificationHandlerMixin, RequestHandler):
    pass

• a "TornadoNameOfHandler.py". This contains

class TornadoNameOfHandler(NotificationHandlerMixin, TornadoService):
    pass

which should be respected also here. So, at a minimum you need to modify accordingly also the "TornadoTokenManagerHandler.py". Or, at the 2 VERY different one from the other?

BTW I think this should also go to v8.