Support for MSC2140 (terms of service for IS/IM) #988
Conversation
Co-Authored-By: J. Ryan Stinnett <jryans@gmail.com>
and IMs shouldn't have a slash
There was a problem hiding this comment.
Thanks, this looks good to me! 😁
However, do take a look at this React SDK comment before merging, as I am unsure which SDK should handle such things.
src/base-apis.js
Outdated
| case SERVICE_TYPES.IS: | ||
| return baseUrl + httpApi.PREFIX_IDENTITY_V2 + '/terms'; | ||
| case SERVICE_TYPES.IM: | ||
| return baseUrl + '/terms'; |
There was a problem hiding this comment.
:( at not using the namespace for the IM
There was a problem hiding this comment.
I looked at changing but stopped on thinking about changing all the other ones. I guess we could just use it for this one though.
There was a problem hiding this comment.
I'd recommend it, just for the sake of not making the problem worse. The baseUrl might have a path component (scalar is /api, dimension is /api/v1/scalar) which might have to be trimmed off before applying the spec prefix + /terms
There was a problem hiding this comment.
Ah yes, the fact that we have configs with the api prefix is the main problem - from a riot/scalar PoV we can just ship the config change with the new app version but I don't know how this would work for anyone with a custom IM: I was going to run it past you with your Dimension hat on.
There was a problem hiding this comment.
Right, so we can't just update this path because then we'd have to move the /api/ from the config to the hardcoded part in the source, and this would break dimension since for dimension, the /api part goes in the middle.
I think our only option is going to be to update all the paths at once, so once scalar supports _matrix paths, we can then patch riot develop to use _matrix paths across the board and it will be compatible with deployed scalar and any dimension new enough to support the _matrix paths.
There was a problem hiding this comment.
Dimension supports the route at the root, like the spec implies. You're welcome to copy the parsing from Dimension (which does use the root-level route on upstreams):
const parsed = new URL(baseUrl);
parsed.pathname = `/_matrix/integrations/v1/terms`;
return parsed.toString();
getTerms is un-authed so doesn't need the access token
Support for MSC2140