Code review comment for lp:~mardy/account-plugins/mcloud-plugin-lp1587282

Gary.Wang (gary-wzl77) wrote :

Thanks for this MP, mandy.
It looks good to me.

I know the reason why you apply this change in the signon-plugin-oauth (
And modify the TokenPath here accordingly. :)
<setting name="TokenPath"></setting>

I'd like to test it on my side with my phone, could you please give me the silo number for accounts-sso/signon-plugin-oauth2 with your patch applied after you finish verification.

« Back to merge proposal