config,handlers/_base: added homeserver config for what state is included in a room invite

[Rugvip]

Related SPEC bug: https://matrix.org/jira/browse/SPEC-357

[matrixbot]

Can one of the admins verify this patch?

[illicitonion]

@matrixbot ok to test

[illicitonion]

@Rugvip LGTM, will merge if jenkins is happy

[Rugvip]

Needs #599

[Rugvip]

Actually wth, added that commit into this instead :p

[illicitonion]

This breaks our unit tests; can you throw in this patch?:

$ git diff
diff --git a/tests/utils.py b/tests/utils.py
index bf7a31f..11d9acb 100644
--- a/tests/utils.py
+++ b/tests/utils.py
@@ -50,6 +50,7 @@ def setup_test_homeserver(name="test", datastore=None, config=None, **kargs):
         config.macaroon_secret_key = "not even a little secret"
         config.server_name = "server.under.test"
         config.trusted_third_party_id_servers = []
+        config.room_invite_state_types = []

     if "clock" not in kargs:
         kargs["clock"] = MockClock()

[ara4n]

Before merging this, we should update the spec to actually articulate what's allowed to go into an invite and not. Agreed that it'd be nice to have this extensibility, but want to do so without creating a spam vector or encouraging fragmentation. Created a https://matrix.org/jira/browse/SPEC-357 spec bug first.

[erikjohnston]

@Rugvip Can you fix the merge conflicts?

[matrixbot]

Can one of the admins verify this patch?

[erikjohnston]

LGTM