As shown in the attached patch.

CommentFileSizeAuthor
shindig_integrator_appservice_use_appid.patch851 bytesproppy

Comments

ramsharma2005’s picture

ramsharma2005 commented

Hi,

I think it is far better to use it from token instead of 4appId because of security reasons. Shindig does not check for any sessions and other private data. It checks for valid token only, so the token is the final reliable entity for shindig.

So, we would like to stick with the current implementation. Please let me know if you have any other reasons to use it from $appId.

proppy’s picture

proppy commented

Thanks for the reply.

For some reason token->getAppId() does not match $appId in our setup, further debugging will be needed in order to figure it out.