/rooms/.../members is missing `at` and `not_membership` parameters

[Half-Shot]

Riot is calling these, but they are not given in the spec under https://matrix.org/docs/spec/client_server/unstable.html#get-matrix-client-r0-rooms-roomid-members.

This could be a synapse or a Riot bug, but given it sounds like it's useful, I'm tracking it in here.

[turt2live]

Duplicate of #1227

[Half-Shot]

No it's not. That proposal happened and the resulting spec PR got merged, with this missing #1758

[turt2live]

The spec pr is not fully merged, and I don't think it's a good idea to keep tracking lazy loading omissions like this. I'd rather see someone go through and properly make the spec reflect reality for lazy loading so we can consider the proposal merged.

[Half-Shot]

With respect, this is one of those things that is going to get lost. I would be fine with this, if lazy-loading wasn't now in use by Riot by default and therefore any third party implementations now HAVE to implement these keys to have a working client.

It's a bit lame to have open issues, but really I don't know why we have a situation where a spec isn't fully merged. With respect to fixing the spec, I am happy to close this once we have fixed the issue.

[Half-Shot]

Discussed oob with @turt2live that we can keep the issue open, but there are people on the case.

[richvdh]

see also #1868