We treat an empty string identity server as not having one

[turt2live]

It should default to the client as far as matrix-org/matrix-spec-proposals#2230 is concerned

[turt2live]

actually, @lampholder, what do you think about treating invalid data as null? The spec doesn't really say what we're supposed to do, but I would generally consider invalid as "Disconnected".

[jryans]

I think it seems reasonable to consider anything falsy or otherwise invalid as the same as null and disconnected.

@turt2live, I believe that's what Riot already does, so is the only task to clarify the spec on this point?

[turt2live]

yea, I'm tempted to throw this into spec territory and not riot territory. It's completely reasonable that invalid values be treated as none imo.

[jryans]

Great, I filed a spec issue we can use to track over there.