aleitner commented on code in PR #633: URL: https://github.com/apache/guacamole-server/pull/633#discussion_r3198295312
########## src/protocols/rdp/rdp.c: ########## @@ -327,6 +328,149 @@ static BOOL rdp_freerdp_authenticate(freerdp* instance, char** username, } +#ifdef HAVE_FREERDP_AAD_SUPPORT +#include "aad.h" + +/** + * Callback invoked by FreeRDP when an Azure AD access token is required. + * + * @param instance + * The FreeRDP instance associated with the RDP session. + * + * @param tokenType + * The type of access token being requested (ACCESS_TOKEN_TYPE_AAD or + * ACCESS_TOKEN_TYPE_AVD). + * + * @param token + * Pointer to a string which will receive the access token. This function + * must allocate and populate this string. + * + * @param count + * Number of additional variadic arguments. + * + * @param ... + * Additional arguments (for AAD: scope and req_cnf) + * Review Comment: Kept the variadic signature (FreeRDP imposes it) but renamed to `rdp_freerdp_get_aad_access_token` and updated the docstring to make clear the implementation is AAD-only -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
