summary refs log tree commit diff
Commit message (Collapse)AuthorAgeFilesLines
* Copy everything to backupErik Johnston2017-10-125-77/+151
|
* TypoErik Johnston2017-10-121-1/+1
|
* Remove dead codeErik Johnston2017-10-121-3/+0
|
* Fix typosErik Johnston2017-10-121-20/+26
|
* Fix typoErik Johnston2017-10-121-1/+1
|
* Basic implementation of backup media storeErik Johnston2017-10-124-126/+131
|
* log when we get an exception handling replication updateshera2017-10-121-1/+5
|
* Merge pull request #2531 from matrix-org/dbkr/spamcheck_error_messagesDavid Baker2017-10-121-2/+6
|\ | | | | Allow error strings from spam checker
| * fix isinstanceDavid Baker2017-10-111-1/+1
| |
| * Allow error strings from spam checkerDavid Baker2017-10-111-2/+6
| |
* | Merge pull request #2532 from matrix-org/rav/fix_linearizerRichard van der Hoff2017-10-112-6/+46
|\ \ | | | | | | Fix stackoverflow and logcontexts from linearizer
| * | Fix stackoverflow and logcontexts from linearizerRichard van der Hoff2017-10-112-6/+46
| | | | | | | | | | | | | | | | | | | | | 1. make it not blow out the stack when there are more than 50 things waiting for a lock. Fixes https://github.com/matrix-org/synapse/issues/2505. 2. Make it not mess up the log contexts.
* | | Merge pull request #2529 from matrix-org/rav/fix_transaction_failure_handlingRichard van der Hoff2017-10-111-2/+3
|\ \ \ | | | | | | | | log pdu_failures from incoming transactions
| * | | log pdu_failures from incoming transactionsRichard van der Hoff2017-10-111-2/+3
| |/ / | | | | | | | | | | | | | | | | | | ... even if we have no EDUs. This appears to have been introduced in 476899295f5fd6cff64799bcbc84cd4bf9005e33.
* | | Merge pull request #2515 from matrix-org/rav/fix_receipt_logcontextRichard van der Hoff2017-10-111-0/+51
|\ \ \ | | | | | | | | | | | | | | | | A logformatter which includes the stack when the exception was caught when logging exceptions.
| * | | logformatter: fix AttributeErrorRichard van der Hoff2017-10-111-3/+11
| | | | | | | | | | | | | | | | make sure we have the relevant fields before we try to log them.
| * | | Fancy logformatter to format exceptions betterRichard van der Hoff2017-10-091-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a bit of an experimental change at this point; the idea is to see if it helps us track down where our stack overflows are coming from by logging the stack when the exception was caught and turned into a Failure. (We'll also need https://github.com/richvdh/twisted/commit/edf27044200e74680ea67c525768e36dc9d9af2b). If we deploy this, we'll be able to enable it via the log config yaml.
* | | | Fix fetching remote summariesErik Johnston2017-10-111-2/+2
| | | |
* | | | Merge pull request #2533 from matrix-org/erikj/fix_group_replErik Johnston2017-10-111-7/+7
|\ \ \ \ | | | | | | | | | | Fix group stream replication
| * | | | peeeeeeeeep8888888888888888888888888888Erik Johnston2017-10-111-1/+1
| | | | |
| * | | | Fix group stream replicationErik Johnston2017-10-111-7/+7
|/ / / / | | | | | | | | | | | | | | | | The stream update functions expect the storage function to return a list of tuples.
* | | | Merge pull request #2530 from matrix-org/rav/fix_receipt_logcontextRichard van der Hoff2017-10-111-0/+4
|\ \ \ \ | | | | | | | | | | fix a logcontext leak in read receipt handling
| * | | | fix a logcontext leak in read receipt handlingRichard van der Hoff2017-10-111-0/+4
| | |/ / | |/| |
* | | | Fix typo in invite to groupErik Johnston2017-10-111-1/+1
| | | |
* | | | Fix typoErik Johnston2017-10-111-1/+1
| | | |
* | | | Fix attestations to check correct server nameErik Johnston2017-10-111-10/+20
| |_|/ |/| |
* | | Fix schema delta versionsErik Johnston2017-10-113-1/+1
| | |
* | | Merge pull request #2466 from matrix-org/erikj/groups_mergedErik Johnston2017-10-1135-66/+4793
|\ \ \ | | | | | | | | Initial Group Implementation
| * | | Validate room idsErik Johnston2017-10-111-1/+5
| | | |
| * | | Don't corrupt cacheErik Johnston2017-10-051-0/+1
| | | |
| * | | Merge branch 'develop' into erikj/groups_mergedDavid Baker2017-10-0231-299/+1177
| |\ \ \
| * \ \ \ Merge pull request #2472 from matrix-org/erikj/groups_roomsErik Johnston2017-09-267-4/+73
| |\ \ \ \ | | | | | | | | | | | | Add remove room from group API
| | * | | | Add remove room APIErik Johnston2017-09-266-1/+72
| | | | | |
| | * | | | Add unique index to group_rooms tableErik Johnston2017-09-262-3/+1
| |/ / / /
| * | | | Merge pull request #2471 from matrix-org/erikj/group_summary_publicisedErik Johnston2017-09-261-25/+31
| |\ \ \ \ | | | | | | | | | | | | Add is_publicised to group summary
| | * | | | Add is_publicised to group summaryErik Johnston2017-09-261-25/+31
| |/ / / /
| * | | | Remove user from group summary when the leave the groupErik Johnston2017-09-211-0/+8
| | | | |
| * | | | Revert: Keep room_id's in group summaryErik Johnston2017-09-211-0/+1
| | | | |
| * | | | Keep room_id's in group summaryErik Johnston2017-09-211-1/+0
| | | | |
| * | | | Correctly return next tokenErik Johnston2017-09-201-1/+2
| | | | |
| * | | | Fix initial syncErik Johnston2017-09-201-1/+9
| | | | |
| * | | | Merge pull request #2454 from matrix-org/erikj/groups_sync_creatorErik Johnston2017-09-192-4/+31
| |\ \ \ \ | | | | | | | | | | | | Ensure that creator of group sees group down /sync
| | * | | | Ensure that creator of group sees group down /syncErik Johnston2017-09-192-4/+31
| |/ / / /
| * | | | Merge branch 'develop' of github.com:matrix-org/synapse into erikj/groups_mergedErik Johnston2017-09-1939-592/+1313
| |\ \ \ \
| * \ \ \ \ Merge pull request #2430 from matrix-org/erikj/groups_profile_cacheErik Johnston2017-08-253-2/+30
| |\ \ \ \ \ | | | | | | | | | | | | | | Add user profiles to summary from group server
| | * | | | | Add user profiles to summary from group serverErik Johnston2017-08-253-2/+30
| | | | | | |
| * | | | | | Merge pull request #2429 from matrix-org/erikj/groups_profile_cacheErik Johnston2017-08-2515-47/+292
| |\| | | | | | | | | | | | | | | | | | | Add a remote user profile cache
| | * | | | | Fix typos and reinheritErik Johnston2017-08-251-12/+6
| | | | | | |
| | * | | | | Split out profile handler to fix testsErik Johnston2017-08-2511-29/+35
| | | | | | |
| | * | | | | Add remote profile cacheErik Johnston2017-08-255-5/+237
| | | | | | |
| | * | | | | Add _simple_updateErik Johnston2017-08-251-19/+32
| |/ / / / /
| * | | | | Merge pull request #2410 from matrix-org/erikj/groups_publiciseErik Johnston2017-08-216-0/+197
| |\ \ \ \ \ | | | | | | | | | | | | | | Add ability to publicise group membership
| | * | | | | Use BOOLEAN rather than TEXT typeErik Johnston2017-08-211-1/+1
| | | | | | |
| | * | | | | FixupErik Johnston2017-08-112-9/+14
| | | | | | |
| | * | | | | Add bulk group publicised lookup APIErik Johnston2017-08-095-0/+142
| | | | | | |
| | * | | | | Allow update group publicityErik Johnston2017-08-082-0/+43
| | | | | | |
| | * | | | | Store whether the user wants to publicise their membership of a groupErik Johnston2017-08-083-0/+7
| | | | | | |
| * | | | | | Merge pull request #2426 from matrix-org/erikj/groups_fix_syncErik Johnston2017-08-211-1/+7
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | Groups: Fix mising json.load in initial sync
| | * | | | | Groups: Fix mising json.load in initial syncErik Johnston2017-08-211-1/+7
| |/ / / / /
| * | | | | Remove stale TODO commentsErik Johnston2017-08-041-10/+0
| | | | | |
| * | | | | Merge pull request #2382 from matrix-org/erikj/group_privilegeErik Johnston2017-07-242-1/+85
| |\ \ \ \ \ | | | | | | | | | | | | | | Include users membership in group in summary API
| | * | | | | Update commentErik Johnston2017-07-241-2/+2
| | | | | | |
| | * | | | | Use join rather than joined, etc.Erik Johnston2017-07-241-2/+2
| | | | | | |
| | * | | | | Fix all the typosErik Johnston2017-07-241-8/+7
| | | | | | |
| | * | | | | Include users membership in group in summary APIErik Johnston2017-07-242-0/+60
| | | | | | |
| | * | | | | Check users/rooms are in group before adding to summaryErik Johnston2017-07-241-0/+25
| |/ / / / /
| * | | | | Correctly add joins to correct segmentErik Johnston2017-07-241-1/+1
| | | | | |
| * | | | | Merge pull request #2378 from matrix-org/erikj/group_sync_supportErik Johnston2017-07-2112-12/+283
| |\ \ \ \ \ | | | | | | | | | | | | | | Add groups to sync stream
| | * | | | | Add notifierErik Johnston2017-07-201-0/+1
| | | | | | |
| | * | | | | Fix replication. And notifyErik Johnston2017-07-205-4/+119
| | | | | | |
| | * | | | | Remember to cast to boolErik Johnston2017-07-201-1/+1
| | | | | | |
| | * | | | | Fix bug in generating current tokenErik Johnston2017-07-201-0/+2
| | | | | | |
| | * | | | | Add groups to sync streamErik Johnston2017-07-208-8/+161
| | | | | | |
| * | | | | | Merge pull request #2377 from matrix-org/erikj/group_profile_updateErik Johnston2017-07-205-0/+55
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | Add update group profile API
| | * | | | | Remove spurious content paramErik Johnston2017-07-201-1/+1
| | | | | | |
| | * | | | | Check values are stringsErik Johnston2017-07-201-1/+4
| | | | | | |
| | * | | | | Fix a storage descErik Johnston2017-07-201-1/+1
| | | | | | |
| | * | | | | Add update group profile APIErik Johnston2017-07-205-0/+52
| | | | | | |
| * | | | | | Merge pull request #2374 from matrix-org/erikj/group_server_localErik Johnston2017-07-2011-34/+1457
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Add local group server support
| | * | | | | | Fix typoErik Johnston2017-07-201-1/+1
| | |/ / / / /
| | * | | | | CommentsErik Johnston2017-07-184-13/+24
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-182-103/+88
| | | | | | |
| | * | | | | Fix permissionsErik Johnston2017-07-181-2/+6
| | | | | | |
| | * | | | | Add DELETEErik Johnston2017-07-181-0/+46
| | | | | | |
| | * | | | | Add 'args' param to post_jsonErik Johnston2017-07-181-9/+15
| | | | | | |
| | * | | | | Fix typosErik Johnston2017-07-183-12/+22
| | | | | | |
| | * | | | | Correctly parse query paramsErik Johnston2017-07-181-19/+19
| | | | | | |
| | * | | | | Update federation client pokesErik Johnston2017-07-181-13/+22
| | | | | | |
| | * | | | | Add client apisErik Johnston2017-07-182-10/+188
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-182-15/+22
| | | | | | |
| | * | | | | Use transport client directlyErik Johnston2017-07-181-18/+25
| | | | | | |
| | * | | | | CommentErik Johnston2017-07-181-0/+14
| | | | | | |
| | * | | | | Remove sync stuffErik Johnston2017-07-182-29/+1
| | | | | | |
| | * | | | | Add GroupID typeErik Johnston2017-07-181-0/+5
| | | | | | |
| | * | | | | Remove unused tablesErik Johnston2017-07-181-7/+0
| | | | | | |
| | * | | | | Remove sync stuffErik Johnston2017-07-182-70/+0
| | | | | | |
| | * | | | | Add local group server supportErik Johnston2017-07-1710-2/+1248
| |/ / / / /
| * | | | | Merge pull request #2363 from matrix-org/erikj/group_server_summaryErik Johnston2017-07-174-22/+1131
| |\ \ \ \ \ | | | | | | | | | | | | | | Add group summary APIs
| | * | | | | Ensure category and role ids are non-nullErik Johnston2017-07-141-0/+24
| | | | | | |
| | * | | | | Add DEFAULT_ROLE_IDErik Johnston2017-07-141-4/+5
| | | | | | |
| | * | | | | Correctly increment ordersErik Johnston2017-07-141-2/+2
| | | | | | |
| | * | | | | Remove unused functionsErik Johnston2017-07-131-152/+0
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-132-12/+32
| | | | | | |
| | * | | | | Remove TODOErik Johnston2017-07-131-1/+0
| | | | | | |
| | * | | | | Add summary APIs to federationErik Johnston2017-07-131-11/+223
| | | | | | |
| | * | | | | raise NotImplementedErrorErik Johnston2017-07-131-5/+2
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-123-5/+79
| | | | | | |
| | * | | | | Simplify checking if adminErik Johnston2017-07-121-53/+19
| | | | | | |
| | * | | | | Add group summary APIsErik Johnston2017-07-124-2/+970
| |/ / / / /
| * | | | | Merge pull request #2352 from matrix-org/erikj/group_server_splitErik Johnston2017-07-1211-11/+1206
| |\ \ \ \ \ | | | | | | | | | | | | | | Initial Group Server
| | * | | | | CommentErik Johnston2017-07-121-1/+1
| | | | | | |
| | * | | | | Move commentErik Johnston2017-07-111-2/+2
| | | | | | |
| | * | | | | Add federation API for adding room to groupErik Johnston2017-07-111-0/+18
| | | | | | |
| | * | | | | CommentErik Johnston2017-07-111-5/+8
| | | | | | |
| | * | | | | TypoErik Johnston2017-07-111-1/+1
| | | | | | |
| | * | | | | Rename column to attestation_jsonErik Johnston2017-07-112-5/+5
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-113-5/+9
| | | | | | |
| | * | | | | CommentErik Johnston2017-07-112-0/+26
| | | | | | |
| | * | | | | Remove u/ requirementErik Johnston2017-07-111-2/+3
| | | | | | |
| | * | | | | Lift out visibility parsingErik Johnston2017-07-111-20/+21
| | | | | | |
| | * | | | | CommentsErik Johnston2017-07-114-43/+132
| | | | | | |
| | * | | | | Initial group server implementationErik Johnston2017-07-1011-11/+1064
| | | | | | |
* | | | | | | Merge pull request #2501 from matrix-org/dbkr/channel_notificationsErik Johnston2017-10-113-7/+85
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Support for channel notifications
| * | | | | | | pep8David Baker2017-10-102-3/+5
| | | | | | | |
| * | | | | | | Don't KeyError if no power_levels eventDavid Baker2017-10-101-1/+3
| | | | | | | |
| * | | | | | | Use notification levels in power_levelsDavid Baker2017-10-103-12/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than making the condition directly require a specific power level. This way the level require to notify a room can be configured per room.
| * | | | | | | There was already a constant for thisDavid Baker2017-10-101-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | also update copyright
| * | | | | | | More optimisationDavid Baker2017-10-101-6/+9
| | | | | | | |
| * | | | | | | What year is it!? Who's the president!?David Baker2017-10-101-1/+1
| | | | | | | |
| * | | | | | | Merge branch 'develop' into dbkr/channel_notificationsDavid Baker2017-10-107-107/+183
| |\ \ \ \ \ \ \
| * | | | | | | | Make be fasterDavid Baker2017-10-051-3/+3
| | | | | | | | |
| * | | | | | | | Add fastpath optimisationDavid Baker2017-10-051-3/+9
| | | | | | | | |
| * | | | | | | | pep8David Baker2017-10-051-0/+3
| | | | | | | | |
| * | | | | | | | They're called roomsDavid Baker2017-10-051-4/+4
| | | | | | | | |
| * | | | | | | | Get rule type rightDavid Baker2017-10-051-1/+1
| | | | | | | | |
| * | | | | | | | Support for channel notificationsDavid Baker2017-10-053-8/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add condition type to check the sender's power level and add a base rule using it for @channel notifications.
* | | | | | | | | Merge pull request #2483 from jeremycline/unfreeze-ujson-dumpErik Johnston2017-10-111-1/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Unfreeze event before serializing with ujson
| * | | | | | | | | Unfreeze event before serializing with ujsonJeremy Cline2017-09-301-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In newer versions of https://github.com/esnme/ultrajson, ujson does not serialize frozendicts (introduced in esnme/ultrajson@53f85b1). Although the PyPI version is still 1.35, Fedora ships with a build from commit esnme/ultrajson@2f1d487. This causes the serialization to fail if the distribution-provided package is used. This runs the event through the unfreeze utility before serializing it. Thanks to @ignatenkobrain for tracking down the root cause. fixes #2351 Signed-off-by: Jeremy Cline <jeremy@jcline.org>
* | | | | | | | | | Merge pull request #2490 from matrix-org/erikj/drop_left_room_eventsErik Johnston2017-10-103-0/+65
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | | Ignore incoming events for rooms that we have left
| * | | | | | | | | Fix up commentErik Johnston2017-10-091-1/+2
| | | | | | | | | |
| * | | | | | | | | Invalidate cacheErik Johnston2017-10-031-0/+3
| | | | | | | | | |
| * | | | | | | | | Update commentsErik Johnston2017-10-032-1/+8
| | | | | | | | | |
| * | | | | | | | | Remove spurious line feedErik Johnston2017-10-031-1/+0
| | | | | | | | | |
| * | | | | | | | | Ignore incoming events for rooms that we have leftErik Johnston2017-10-032-0/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When synapse receives an event for a room its not in over federation, it double checks with the remote server to see if it is in fact in the room. This is done so that if the server has forgotten about the room (usually as a result of the database being dropped) it can recover from it. However, in the presence of state resets in large rooms, this can cause a lot of work for servers that have legitimately left. As a hacky solution that supports both cases we drop incoming events for rooms that we have explicitly left. This means that we no longer support the case of servers having forgotten that they've rejoined a room, but that is sufficiently rare that we're not going to support it for now.
* | | | | | | | | | Merge pull request #2520 from matrix-org/rav/process_incoming_rooms_in_parallelRichard van der Hoff2017-10-101-19/+34
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | fed server: process PDUs for different rooms in parallel
| * | | | | | | | | | fed server: process PDUs for different rooms in parallelRichard van der Hoff2017-10-091-19/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With luck, this will give a real-time improvement when there are many rooms and the server ends up calling out to fetch missing events.
* | | | | | | | | | | Merge pull request #2518 from matrix-org/rav/linearize_incoming_transactionsRichard van der Hoff2017-10-101-1/+29
|\| | | | | | | | | | | |_|_|_|_|_|_|_|/ / |/| | | | | | | | | Fed server: use a linearizer for ongoing transactions
| * | | | | | | | | Fed server: use a linearizer for ongoing transactionsRichard van der Hoff2017-10-091-1/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't want to process the same transaction multiple times concurrently, so use a linearizer.
* | | | | | | | | | Merge pull request #2517 from matrix-org/rav/fed_server_refactorRichard van der Hoff2017-10-091-24/+29
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | fed server: refactor on_incoming_transaction
| * | | | | | | | | fed server: refactor on_incoming_transactionRichard van der Hoff2017-10-091-24/+29
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move as much as possible to after the have_responded check, and reduce the number of times we iterate over the pdu list.
* | | | | | | | | Merge pull request #2516 from matrix-org/rav/fix_fed_server_origin_checkRichard van der Hoff2017-10-091-24/+24
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | Fed server: Move origin-check code to _handle_received_pdu
| * | | | | | | | Fed server: Move origin-check code to _handle_received_pduRichard van der Hoff2017-10-091-24/+24
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The response-building code expects there to be an entry in the `results` list for each entry in the pdu_list, so the early `continue` was messing this up. That doesn't really matter, because all that the federation client does is log any errors, but it's pretty poor form.
* | | | | | | | Merge pull request #2508 from matrix-org/rav/federation_queue_logcontextsRichard van der Hoff2017-10-091-16/+32
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix up logcontext handling in (federation) TransactionQueue
| * | | | | | | | Fix up logcontext handling in (federation) TransactionQueueRichard van der Hoff2017-10-061-16/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid using preserve_context_over_function, which has problems with respect to logcontexts.
* | | | | | | | | Merge pull request #2507 from matrix-org/rav/execute_concurrently_log_contextsRichard van der Hoff2017-10-091-2/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix logcontext handling for concurrently_execute
| * | | | | | | | | Fix logcontext handling for concurrently_executeRichard van der Hoff2017-10-061-2/+2
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid preserve_context_over_deferred, which is broken.
* | | | | | | | | Merge pull request #2506 from matrix-org/rav/unhandled_failureRichard van der Hoff2017-10-091-24/+21
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Fix up deferred handling in federation.py
| * | | | | | | | Fix up deferred handling in federation.pyRichard van der Hoff2017-10-061-24/+21
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Avoid preserve_context_over_deferred, which is broken * set consumeErrors=True on defer.gatherResults, to avoid spurious "unhandled failure" erros
* | | | | | | | some comments in the state res codeRichard van der Hoff2017-10-051-0/+13
| | | | | | | |
* | | | | | | | Merge pull request #2502 from matrix-org/dbkr/allow_dms_to_adminsDavid Baker2017-10-053-8/+10
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | Spam checking: add the invitee to user_may_invite
| * | | | | | | Spam checking: add the invitee to user_may_inviteDavid Baker2017-10-053-8/+10
|/ / / / / / /
* | | | | | | Merge pull request #2500 from matrix-org/dbkr/fix_word_boundary_mentionsDavid Baker2017-10-051-2/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix notif kws that start/end with non-word chars
| * | | | | | | Use better method for word boundary searchingDavid Baker2017-10-051-11/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From https://github.com/matrix-org/matrix-js-sdk/commit/ebc95667b8a5777d13e5d3c679972bedae022fd5
| * | | | | | | pep8David Baker2017-10-051-0/+1
| | | | | | | |
| * | | | | | | Fix notif kws that start/end with non-word charsDavid Baker2017-10-051-2/+21
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only prepend / append word bounary characters if the search expression starts or ends with a word character, otherwise they don't work because there's no word bounary between whitespace and a non-word char.
* | | | | | | Merge pull request #2495 from matrix-org/dbkr/spam_check_room_creationDavid Baker2017-10-043-0/+73
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add room creation checks to spam checker
| * | | | | | | spam check room publishingDavid Baker2017-10-042-0/+26
| | | | | | | |
| * | | | | | | un-double indentDavid Baker2017-10-041-1/+1
| | | | | | | |
| * | | | | | | Add room creation checks to spam checkerDavid Baker2017-10-043-0/+47
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lets the spam checker deny attempts to create rooms and add aliases to them.
* | | | | | | Merge pull request #2492 from matrix-org/dbkr/spam_check_invitesDavid Baker2017-10-033-2/+39
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Allow spam checker to reject invites too
| * | | | | | | s/roomid/room_id/David Baker2017-10-031-2/+2
| | | | | | | |
| * | | | | | | pass room id tooDavid Baker2017-10-033-4/+6
| | | | | | | |
| * | | | | | | Federation was passing strings anywayDavid Baker2017-10-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | so pass string everywhere
| * | | | | | | this shouldn't be debugDavid Baker2017-10-031-2/+2
| | | | | | | |
| * | | | | | | better loggingDavid Baker2017-10-031-4/+9
| | | | | | | |
| * | | | | | | Actually write warpper functionDavid Baker2017-10-031-0/+16
| | | | | | | |
| * | | | | | | Skip spam check for admin usersDavid Baker2017-10-031-8/+8
| | | | | | | |
| * | | | | | | Fix param name & lintDavid Baker2017-10-031-2/+4
| | | | | | | |
| * | | | | | | Allow spam checker to reject invites tooDavid Baker2017-10-032-6/+18
|/ / / / / / /
* | | | | | | Merge pull request #2491 from matrix-org/rav/port_db_fixesRichard van der Hoff2017-10-031-4/+7
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Drop search values with nul characters
| * | | | | | Drop search values with nul charactersRichard van der Hoff2017-10-021-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/matrix-org/synapse/issues/2187 contains a report of a port failing due to nul characters somewhere in the search table. Let's try dropping the offending rows.
* | | | | | | Merge branch 'master' of github.com:matrix-org/synapse into developErik Johnston2017-10-025-13/+37
|\ \ \ \ \ \ \
| * | | | | | | Bump version and changelog v0.23.1Erik Johnston2017-10-022-1/+9
| | | | | | | |
| * | | | | | | Make 'affinity' package optionalErik Johnston2017-10-022-2/+17
| | | | | | | |
| * | | | | | | Move quit_with_errorErik Johnston2017-10-022-10/+11
| | | | | | | |
* | | | | | | | Merge branch 'master' of github.com:matrix-org/synapse into developErik Johnston2017-10-023-3/+10
|\| | | | | | | | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Merge branch 'release-v0.23.0' of github.com:matrix-org/synapse v0.23.0Erik Johnston2017-10-0252-859/+1833
| |\ \ \ \ \ \ | | |_|/ / / / | |/| | | | |
| | * | | | | Bump version and change log github/release-v0.23.0 release-v0.23.0Erik Johnston2017-10-022-1/+7
| | | | | | |
| * | | | | | Merge pull request #2464 from rnbdsh/patch-4Richard van der Hoff2017-09-251-2/+3
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove non-existing files, add stop, use synctl
| | * | | | | | Start traditionally, stop synctlrnbdsh2017-09-241-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Starting with synctl lead to "no config file found" Stopping also leads to some (code=exited, status=1/FAILURE), but at least now we can stop the service.
| | * | | | | | Remove non-existing files, add stop, use synctlrnbdsh2017-09-241-2/+2
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Non-existing files, when running the suggested from https://github.com/matrix-org/synapse#configuring-synapse /etc/synapse/log_config.yaml so the --log-config leads to an error /etc/sysconfig/synapse The environment-file or even the /etc/sysconfig does not exist in arch linux Also instead of calling python2 we use synctl, as this seems to be the proper way to start it, and it gives us a more useful error in the systemctl status. And we now allow stop (and therefore restart).
* | | | | | | Merge pull request #2480 from matrix-org/rav/federation_client_loggingRichard van der Hoff2017-09-291-7/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Improve logging of failures in matrixfederationclient
| * | | | | | | Improve logging of failures in matrixfederationclientRichard van der Hoff2017-09-281-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * don't log exception types twice * not all exceptions have a meaningful 'message'. Use the repr rather than attempting to build a string ourselves.
* | | | | | | | Only log if we've removed mediaErik Johnston2017-09-281-2/+4
|/ / / / / / /
* | | | | | | Handle SERVFAILs when doing AAAA lookups for federation (#2477)Richard van der Hoff2017-09-281-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... to cope with people with broken dnssec setups, mostly
* | | | | | | Up the limits on number of url cache entries to delete at one timeErik Johnston2017-09-281-2/+2
| | | | | | |
* | | | | | | Add old indicesErik Johnston2017-09-281-0/+2
| | | | | | |
* | | | | | | Clear up expired url cache every 10sErik Johnston2017-09-281-1/+1
| | | | | | |
* | | | | | | Actually assign result to variableErik Johnston2017-09-281-1/+1
| | | | | | |
* | | | | | | Only pull out local media that were for url cacheErik Johnston2017-09-281-1/+1
| | | | | | |
* | | | | | | Fix typo in url cache expiry timerErik Johnston2017-09-281-1/+1
| | | | | | |
* | | | | | | Merge pull request #2479 from matrix-org/erikj/expire_url_cache_thumbnailsErik Johnston2017-09-281-31/+81
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Support new and old style media id formats
| * | | | | | | Support new and old style media id formatsErik Johnston2017-09-281-31/+81
| | | | | | | |
* | | | | | | | Fix typoErik Johnston2017-09-281-3/+3
| | | | | | | |
* | | | | | | | Merge pull request #2478 from matrix-org/erikj/expire_url_cache_thumbnailsErik Johnston2017-09-285-12/+234
|\| | | | | | | | |_|_|/ / / / |/| | | | | | Delete expired url cache data
| * | | | | | Change expires column to expires_tsErik Johnston2017-09-283-10/+29
| | | | | | |
| * | | | | | More bracketsErik Johnston2017-09-281-1/+1
| | | | | | |
| * | | | | | Delete expired url cache dataErik Johnston2017-09-285-5/+208
|/ / / / / /
* | | | | | Merge pull request #2476 from matrix-org/erikj/joined_members_authErik Johnston2017-09-282-13/+26
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix /joined_members to work with AS users
| * | | | | | Fix /joined_members to work with AS usersErik Johnston2017-09-282-13/+26
| | | | | | |
* | | | | | | Merge pull request #2475 from matrix-org/erikj/joined_members_authErik Johnston2017-09-272-10/+38
|\| | | | | | | | | | | | | | | | | | | | Fix bug where /joined_members didn't check user was in room
| * | | | | | Fix bug where /joined_members didn't check user was in roomErik Johnston2017-09-272-10/+38
|/ / / / / /
* | | | | | Merge pull request #2474 from matrix-org/dbkr/spam_check_moduleDavid Baker2017-09-276-23/+74
|\ \ \ \ \ \ | | | | | | | | | | | | | | Make the spam checker a module
| * | | | | | Don't test is spam_checker not NoneDavid Baker2017-09-271-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes it's a Mock object which is not none but is still not what we're after
| * | | | | | Remove unintentional debuggingDavid Baker2017-09-271-1/+0
| | | | | | |
| * | | | | | pep8David Baker2017-09-272-0/+2
| | | | | | |
| * | | | | | Forgot the new file again :(David Baker2017-09-271-0/+35
| | | | | | |
| * | | | | | Make the spam checker a moduleDavid Baker2017-09-265-23/+33
| | | | | | |
* | | | | | | pep8David Baker2017-09-261-0/+1
|/ / / / / /
* | | | | | Merge pull request #2473 from matrix-org/dbkr/factor_out_module_loadingDavid Baker2017-09-262-12/+51
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Factor out module loading to a separate place
| * | | | | unnecessary parensDavid Baker2017-09-261-1/+1
| | | | | |
| * | | | | Add module_loader.pyDavid Baker2017-09-261-0/+41
| | | | | |
| * | | | | Factor out module loading to a separate placeDavid Baker2017-09-261-12/+10
|/ / / / / | | | | | | | | | | | | | | | So it can be reused
* | | | | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.23.0 v0.23.0-rc2Erik Johnston2017-09-263-19/+39
|\ \ \ \ \
| * \ \ \ \ Merge pull request #2470 from matrix-org/erikj/sync_speed_fixErik Johnston2017-09-251-5/+18
| |\ \ \ \ \ | | | | | | | | | | | | | | Refactor to speed up incremental syncs
| | * | | | | Refactor to speed up incremental syncsErik Johnston2017-09-251-5/+18
| |/ / / / /
| * | | | | Merge pull request #2468 from maxidor/developRichard van der Hoff2017-09-251-13/+20
| |\ \ \ \ \ | | | | | | | | | | | | | | Clarify recommended network setup
| | * | | | | Include review feedbackMax Dor2017-09-251-5/+3
| | | | | | |
| | * | | | | Clarify recommended network setupMax Dor2017-09-251-15/+24
| |/ / / / /
| * | | | | Merge branch 'release-v0.23.0' of github.com:matrix-org/synapse into developErik Johnston2017-09-252-1/+33
| |\ \ \ \ \
| * \ \ \ \ \ Merge pull request #2390 from r3dey3/developRichard van der Hoff2017-09-251-1/+1
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix iteration of requests_missing_keys; list doesn't have .values()
| | * \ \ \ \ \ Merge branch 'develop' into developRichard van der Hoff2017-09-2548-805/+1855
| | |\ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| | * | | | | | Fix iteration of requests_missing_keys; list doesn't have .values()Kenny Keslar2017-07-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Kenny Keslar <r3dey3@r3dey3.com>
* | | | | | | | Bump changelog and versionErik Johnston2017-09-262-1/+9
| |_|/ / / / / |/| | | | | |
* | | | | | | Update CHANGES v0.23.0-rc1Erik Johnston2017-09-251-3/+11
| | | | | | |
* | | | | | | Bump version and changesErik Johnston2017-09-252-1/+25
|/ / / / / /
* | | | | | Fix logcontxt leak in keyclient (#2465)Richard van der Hoff2017-09-251-12/+5
| | | | | | | | | | | | | | | | | | preserve_context_over_function doesn't do what you want it to do.
* | | | | | Do an AAAA lookup on SRV record targets (#2462)Richard van der Hoff2017-09-222-24/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support SRV records which point at AAAA records, as well as A records. Fixes https://github.com/matrix-org/synapse/issues/2405
* | | | | | fix thinko'd docstringMatthew Hodgson2017-09-221-1/+1
| | | | | |
* | | | | | Fix device list when rejoining room (#2461)Erik Johnston2017-09-221-2/+14
| | | | | |
* | | | | | Merge pull request #2459 from matrix-org/rav/keyring_cleanupsRichard van der Hoff2017-09-205-214/+353
|\ \ \ \ \ \ | | | | | | | | | | | | | | Clean up Keyring code
| * | | | | | Fix logcontexts in _check_sigs_and_hashesRichard van der Hoff2017-09-202-60/+62
| | | | | | |
| * | | | | | Add some more tests for KeyringRichard van der Hoff2017-09-201-37/+140
| | | | | | |
| * | | | | | Fix logcontext handling in verify_json_objects_for_serverRichard van der Hoff2017-09-201-8/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | preserve_context_over_fn is essentially broken, because (a) it pointlessly drops the current logcontext before calling its wrapped function, which means we don't get any useful logcontexts for _handle_key_deferred; (b) it wraps the resulting deferred in a _PreservingContextDeferred, which is very dangerous because you then can't yield on it without leaking context back into the reactor. Instead, let's specify that the resultant deferreds call their callbacks with no logcontext.
| * | | | | | Turn _start_key_lookups into an inlineCallbacks functionRichard van der Hoff2017-09-201-40/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... which means that logcontexts can be correctly preserved for the stuff it does. get_server_verify_keys is now called with the logcontext, so needs to preserve_fn when it fires off its nested inlineCallbacks function. Also renames get_server_verify_keys to reflect the fact it's meant to be private.
| * | | | | | Fix potential race in _start_key_lookupsRichard van der Hoff2017-09-201-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the verify_request.deferred has already completed, then `remove_deferreds` will be called immediately. It therefore might resolve the server_to_deferred deferred while there are still other requests for that server in flight. To avoid that, we should build the complete list of requests, and *then* add the callbacks.
| * | | | | | Add some comments to _start_key_lookupsRichard van der Hoff2017-09-201-0/+8
| | | | | | |
| * | | | | | Consistency for how verify_request.deferred is calledRichard van der Hoff2017-09-201-13/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Define that it is run with no log context, and make sure that happens. If we aren't careful to reset the logcontext, we can't bung the deferreds into defer.gatherResults etc. We don't actually do that directly, but we *do* resolve other deferreds from affected callbacks (notably the server_to_deferred map in _start_key_lookups), and those *do* get passed into defer.gatherResults. It turns out that this way ends up being least confusing.
| * | | | | | Factor out _start_key_lookupsRichard van der Hoff2017-09-201-7/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | ... to make it easier to see what's going on.
| * | | | | | Replace `server_and_json` with `verify_requests`Richard van der Hoff2017-09-201-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a precursor to factoring some of this code out.