summary refs log tree commit diff
path: root/synapse (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Port #4422 debug logging from hotfixesErik Johnston2019-03-061-7/+46
|
* Merge pull request #4792 from matrix-org/anoa/replication_tokensAndrew Morgan2019-03-061-3/+28
|\ | | | | Support batch updates in the worker sender
| * Simplify token replication logicAndrew Morgan2019-03-051-23/+14
| |
| * Clean up logic and add commentsAndrew Morgan2019-03-041-11/+18
| |
| * Clearer branching, fix missing list clearAndrew Morgan2019-03-041-4/+11
| |
| * Prevent replication wedgingAndrew Morgan2019-03-041-4/+24
| |
* | Make registration ratelimiter separate from the main events oneBrendan Abolivier2019-03-063-2/+6
| |
* | Revert "Split ratelimiters in two (one for events, one for registration)"Brendan Abolivier2019-03-065-11/+7
| | | | | | | | This reverts commit d7dbad3526136cfc9fdbd568635be5016fb637db.
* | Split ratelimiters in two (one for events, one for registration)Brendan Abolivier2019-03-055-7/+11
| |
* | Move settings from registration to ratelimiting in config fileBrendan Abolivier2019-03-052-18/+20
| |
* | Merge pull request #4772 from jbweston/jbweston/server-version-apiErik Johnston2019-03-051-0/+23
|\ \ | | | | | | Add 'server_version' endpoint to admin API
| * | Add 'server_version' endpoint to admin APIJoseph Weston2019-03-011-0/+23
| | | | | | | | | | | | | | | This is required because the 'Server' HTTP header is not always passed through proxies.
* | | Add rate-limiting on registration (#4735)Brendan Abolivier2019-03-056-31/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rate-limiting for registration * Add unit test for registration rate limiting * Add config parameters for rate limiting on auth endpoints * Doc * Fix doc of rate limiting function Co-Authored-By: babolivier <contact@brendanabolivier.com> * Incorporate review * Fix config parsing * Fix linting errors * Set default config for auth rate limiting * Fix tests * Add changelog * Advance reactor instead of mocked clock * Move parameters to registration specific config and give them more sensible default values * Remove unused config options * Don't mock the rate limiter un MAU tests * Rename _register_with_store into register_with_store * Make CI happy * Remove unused import * Update sample config * Fix ratelimiting test for py2 * Add non-guest test
* | | Merge pull request #4794 from matrix-org/anoa/erroneous_dollahsAndrew Morgan2019-03-051-22/+22
|\ \ \ | | | | | | | | Remove unnecessary dollar signs
| * | | Remove unnecessary dollar signsAndrew Morgan2019-03-041-22/+22
| | | | | | | | | | | | | | | | | | | | A dollar sign is already appended to the end of each PATH, so there's no need to add one in the PATH declaration as well.
* | | | Merge pull request #4699 from matrix-org/erikj/stop_fed_not_in_roomErik Johnston2019-03-053-29/+126
|\ \ \ \ | | | | | | | | | | Stop backpaginating when events not visible
| * | | | clarify commentsErik Johnston2019-03-052-9/+12
| | | | |
| * | | | Fix missing null guardErik Johnston2019-03-041-1/+1
| | | | |
| * | | | Only check history visibility when filteringErik Johnston2019-03-042-35/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When filtering events to send to server we check more than just history visibility. However when deciding whether to backfill or not we only care about the history visibility.
| * | | | s/get_forward_events/get_successor_events/Erik Johnston2019-03-042-3/+3
| | | | |
| * | | | Merge branch 'develop' of github.com:matrix-org/synapse into ↵Erik Johnston2019-03-0436-191/+571
| |\ \ \ \ | | | | | | | | | | | | | | | | | | erikj/stop_fed_not_in_room
| * | | | | Update commentsErik Johnston2019-02-271-3/+13
| | | | | |
| * | | | | Stop backpaginating when events not visibleErik Johnston2019-02-203-5/+78
| | | | | |
* | | | | | Merge pull request #4799 from matrix-org/rav/clean_up_replication_codeErik Johnston2019-03-051-60/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Clean ups in replication notifier
| * | | | | | Remove redundant PreserveLoggingContextRichard van der Hoff2019-03-041-10/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both (!) things that register as replication listeners do the right thing wrt logcontexts, so this is redundant.
| * | | | | | Remove unused `wait_for_replication` methodRichard van der Hoff2019-03-041-50/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I guess this was used once? It's not now, anyway.
* | | | | | | Merge pull request #4796 from matrix-org/erikj/factor_out_e2e_keysErik Johnston2019-03-059-747/+762
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Allow /keys/{changes,query} API to run on worker
| * | | | | | | Allow /keys/{changes,query} API to run on workerErik Johnston2019-03-042-2/+13
| | | | | | | |
| * | | | | | | Split DeviceHandler into master and workerErik Johnston2019-03-042-170/+179
| | | | | | | |
| * | | | | | | Fixup slave storesErik Johnston2019-03-046-577/+572
| | |_|_|/ / / | |/| | | | |
* | | | | | | Add some debug about processing read receipts.Richard van der Hoff2019-03-041-6/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm hoping to establish which rooms are having lots of RRs sent for them, and how old the events are when they are sent.
* | | | | | | Clean up read-receipt handling.Richard van der Hoff2019-03-041-59/+44
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove a call to run_as_background_process: there is no need to run this as a background process, because build_and_send_edu does not block. We may as well inline the whole of _push_remotes.
* | | | | | Include a default configuration file in the 'docs' directory. (#4791)Richard van der Hoff2019-03-044-13/+25
|/ / / / /
* | / / / Avoid rebuilding Edu objects in worker mode (#4770)Richard van der Hoff2019-03-045-19/+36
| |/ / / |/| | | | | | | | | | | | | | | In worker mode, on the federation sender, when we receive an edu for sending over the replication socket, it is parsed into an Edu object. There is no point extracting the contents of it so that we can then immediately build another Edu.
* | | | Fix incorrect log about not persisting duplicate state event. (#4776)Richard van der Hoff2019-03-011-3/+4
| |_|/ |/| | | | | We were logging this when it was not true.
* | | Merge branch 'master' into developRichard van der Hoff2019-03-011-1/+1
|\ \ \ | |_|/ |/| |
| * | 0.99.2Richard van der Hoff2019-03-011-1/+1
| | |
* | | Minor docstring fixes for MatrixFederationAgent (#4765)Andrew Morgan2019-02-281-1/+5
| | |
* | | Make 'event_id' a required parameter in federated state requests (#4741)Amber Brown2019-02-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * make 'event_id' a required parameter in federated state requests As per the spec: https://matrix.org/docs/spec/server_server/r0.1.1.html#id40 Signed-off-by: Joseph Weston <joseph@weston.cloud> * add changelog entry for bugfix Signed-off-by: Joseph Weston <joseph@weston.cloud> * Update server.py
* | | Fix parsing of Content-Disposition headers (#4763)Richard van der Hoff2019-02-271-20/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix parsing of Content-Disposition headers TIL: filenames in content-dispostion headers can contain semicolons, and aren't %-encoded. * fix python2 incompatibility * Fix docstrings
* | | Move from TravisCI to BuildKite (#4752)Amber Brown2019-02-271-1/+1
| | |
* | | Merge pull request #4759 from matrix-org/erikj/3pid_client_readerErik Johnston2019-02-272-33/+35
|\ \ \ | | | | | | | | Move /account/3pid to client_reader
| * | | Move /account/3pid to client_readerErik Johnston2019-02-272-33/+35
| | | |
* | | | Merge pull request #4758 from matrix-org/erikj/use_presence_replicationErik Johnston2019-02-272-2/+8
|\ \ \ \ | | | | | | | | | | When presence is disabled don't send over replication
| * | | | When presence is enabled don't send over replicationErik Johnston2019-02-272-2/+8
| |/ / /
* / / / Move server key queries to federation readerErik Johnston2019-02-271-1/+5
|/ / /
* | | Merge pull request #4749 from matrix-org/erikj/replication_connection_backoffErik Johnston2019-02-273-5/+39
|\ \ \ | |/ / |/| | Fix tightloop over connecting to replication server
| * | Move connecting logic into ClientReplicationStreamProtocolErik Johnston2019-02-272-18/+17
| | |
| * | Increase the max delay between retry attemptsErik Johnston2019-02-261-1/+1
| | | | | | | | | | | | | | | Otherwise if you have many workers they can easily take out master with their connection attempts
| * | Fix tightloop over connecting to replication serverErik Johnston2019-02-262-4/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the client failed to process incoming commands during the initial set up of the replication connection it would immediately disconnect and reconnect, resulting in a tightloop. This can happen, for example, when subscribing to a stream that has a row that is too long in the backlog. The fix here is to not consider the connection successfully set up until the client has succesfully subscribed and caught up with the streams. This ensures that the retry logic timers aren't reset until then, meaning that if an error does happen during start up the client will continue backing off before retrying again.
* | | 0.99.2rc1 v0.99.2rc1Richard van der Hoff2019-02-271-1/+1
| | |
* | | Limit cache invalidation replication line length (#4748)Erik Johnston2019-02-272-5/+27
| | |
* | | Add database version to phonehome stats. (#4753)Richard van der Hoff2019-02-274-0/+45
|/ /
* | Merge branch 'develop' of github.com:matrix-org/synapse into ↵Andrew Morgan2019-02-26193-4548/+9279
|\ \ | | | | | | | | | anoa/public_rooms_federate_develop
| * | Revert "Prevent showing non-fed rooms in fed /publicRooms"Andrew Morgan2019-02-264-70/+17
| | |
| * | Merge pull request #4736 from matrix-org/anoa/public_rooms_federateAndrew Morgan2019-02-264-17/+70
| |\ \ | | | | | | | | Config option to prevent showing non-fed rooms in fed /publicRooms
| | * | Cleaner chunk logicAndrew Morgan2019-02-261-5/+4
| | | |
| | * | Correct indentAndrew Morgan2019-02-261-3/+3
| | | |
| | * | Correct docstring types and chunk logicAndrew Morgan2019-02-261-10/+11
| | | |
| | * | Clean up room chunk logicAndrew Morgan2019-02-261-6/+8
| | | |
| | * | Make not showing non-federated rooms the defaultAndrew Morgan2019-02-262-13/+1
| | | |
| | * | Result may be NoneAndrew Morgan2019-02-261-1/+1
| | | |
| | * | Address rich commentsAndrew Morgan2019-02-262-7/+31
| | | |
| | * | Fix group's call to generate_room_entryAndrew Morgan2019-02-251-4/+2
| | | |
| | * | elif not else ifAndrew Morgan2019-02-251-1/+1
| | | |
| | * | syntax derpAndrew Morgan2019-02-251-1/+1
| | | |
| | * | Simplify call to generate_room_entryAndrew Morgan2019-02-251-17/+13
| | | |
| | * | Merge branch 'develop' of github.com:matrix-org/synapse into ↵Erik Johnston2019-02-25192-4516/+9135
| | |\ \ | | | | | | | | | | | | | | | anoa/public_rooms_federate
| | * | | Docs and arg name clarificationAndrew Morgan2019-02-251-2/+15
| | | | |
| | * | | Don't restrict non-fed rooms over client APIsAndrew Morgan2019-02-253-14/+22
| | | | |
| | * | | Config option to prevent showing non-fed rooms in fed /publicRoomsAndrew Morgan2019-02-252-2/+26
| | | | |
| * | | | Don't log stack trace when client has gone away during media download (#4738)Erik Johnston2019-02-251-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Don't log stack trace when client has gone away during media download * Newsfile * Fixup newsfile
| * | | | Fix ACME config for python 2. (#4717)Richard van der Hoff2019-02-251-3/+7
| | | | | | | | | | | | | | | Fixes #4675.
| * | | | switch from google.com to recaptcha.net for reCAPTCHA (#4731)Matthew Hodgson2019-02-254-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * add trivial clarification about jemalloc * switch from google.com to recaptcha.net because https://developers.google.com/recaptcha/docs/faq#can-i-use-recaptcha-globally
| * | | | Merge pull request #4737 from matrix-org/erikj/failure_log_tbErik Johnston2019-02-253-7/+12
| |\ \ \ \ | | | | | | | | | | | | Log tracebacks correctly
| | * | | | Log tracebacks correctlyErik Johnston2019-02-253-7/+12
| | | |/ / | | |/| |
| * | | | Merge pull request #4718 from matrix-org/erikj/fix_backfill_state_shredErik Johnston2019-02-251-3/+31
| |\ \ \ \ | | | | | | | | | | | | Fix backfill storing incorrect state for events
| | * | | | Fix typoErik Johnston2019-02-251-1/+1
| | | | | |
| | * | | | More commentsErik Johnston2019-02-251-0/+4
| | | | | |
| | * | | | Assert rather than clobber the valuesErik Johnston2019-02-251-2/+2
| | | | | |
| | * | | | Add comments and paranoiaErik Johnston2019-02-251-2/+23
| | | | | |
| | * | | | Fix backfill storing incorrect state for eventsErik Johnston2019-02-221-1/+4
| | | | | |
| * | | | | Merge pull request #4721 from matrix-org/erikj/msc_1866Erik Johnston2019-02-252-2/+26
| |\ \ \ \ \ | | | | | | | | | | | | | | MSC 1866 - Use M_UNSUPPORTED_ROOM_VERSION for invite API
| | * | | | | MSC 1866 - Use M_UNSUPPORTED_ROOM_VERSION for invite APIErik Johnston2019-02-232-2/+26
| | | | | | |
| * | | | | | Merge pull request #4722 from ↵Erik Johnston2019-02-251-21/+51
| |\ \ \ \ \ \ | | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | matrix-org/erikj/correctly_handle_keyring_exceptions Handle errors when fetching remote server keys
| | * | | | | raise_from already raisesErik Johnston2019-02-251-4/+4
| | | | | | |
| | * | | | | Handle errors when fetching remote server keysErik Johnston2019-02-231-21/+51
| | |/ / / /
| * / / / / Correctly proxy exception in frontend_proxy workerErik Johnston2019-02-231-5/+10
| |/ / / /
| * | | | Merge pull request #4716 from matrix-org/erikj/pusher_loggingErik Johnston2019-02-223-1/+16
| |\ \ \ \ | | | | | | | | | | | | Fix up pusher logging a bit
| | * | | | Add missing returnErik Johnston2019-02-221-0/+1
| | | | | |
| | * | | | Correctly handle PusherConfigExceptionErik Johnston2019-02-221-0/+9
| | | | | |
| | * | | | Drop logging level of creating a pusherErik Johnston2019-02-221-1/+1
| | | | | |
| | * | | | Correctly handle null data in HttpPusherErik Johnston2019-02-221-0/+5
| | |/ / /
| * / / / Fix state cache invalidation on workersErik Johnston2019-02-222-12/+35
| |/ / /
| * | | Add prometheus metrics for number of badge update pushes. (#4709)Richard van der Hoff2019-02-221-8/+25
| | | | | | | | | | | | | | | | | | | | | | | | We're counting the number of push notifications, but not the number of badges; I'd like to see if they are significant.
| * | | bail out early in on_new_receipts if no pushers (#4706)Richard van der Hoff2019-02-211-0/+8
| | | |
| * | | Run push_receipts_to_remotes as background job (#4707)Richard van der Hoff2019-02-211-34/+34
| | | | | | | | | | | | I suspect the CPU usage metrics for this are going to /dev/null at the moment.
| * | | Merge pull request #4263 from rkfg/developErik Johnston2019-02-211-1/+1
| |\ \ \ | | |_|/ | |/| | Prevent crash on pagination.
| | * | Prevent crash on pagination.rkfg2018-12-061-1/+1
| | | |
| * | | Merge pull request #4694 from matrix-org/erikj/fix_sentry_config_formatErik Johnston2019-02-201-0/+2
| |\ \ \ | | | | | | | | | | Fixup generated metrics config
| | * | | Fixup generated metrics configErik Johnston2019-02-201-0/+2
| | | | |
| * | | | Add metrics for number of outgoing EDUs, by type (#4695)Richard van der Hoff2019-02-202-6/+18
| | | | |
| * | | | UPSERT many functionality (#4644)Amber Brown2019-02-201-12/+134
| | | | |
| * | | | Logging improvements for the pusher (#4691)Richard van der Hoff2019-02-202-13/+14
| | | | |
| * | | | Fix TaskStopped exceptions when outbound requests time out (#4690)Richard van der Hoff2019-02-203-18/+29
| | | | |
| * | | | Merge pull request #4678 from matrix-org/rav/tls_install_instructionsRichard van der Hoff2019-02-2024-196/+248
| |\ \ \ \ | | |/ / / | |/| | | Attempt to clarify installation/config instructions
| | * | | Attempt to make default config more consistentRichard van der Hoff2019-02-1924-196/+248
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The general idea here is that config examples should just have a hash and no extraneous whitespace, both to make it easier for people who don't understand yaml, and to make the examples stand out from the comments.
| * | | | Fix registration on workers (#4682)Erik Johnston2019-02-2012-150/+275
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move RegistrationHandler init to HomeServer * Move post registration actions to RegistrationHandler * Add post regisration replication endpoint * Newsfile
| * | | | Merge pull request #4671 from matrix-org/erikj/state_cache_invalidationErik Johnston2019-02-193-32/+70
| |\ \ \ \ | | |/ / / | |/| | | Batch cache invalidation over replication
| | * | | DocsErik Johnston2019-02-191-2/+3
| | | | |
| | * | | Use itertoolsErik Johnston2019-02-191-2/+2
| | | | |
| | * | | DocumentationErik Johnston2019-02-191-4/+4
| | | | |
| | * | | Batch cache invalidation over replicationErik Johnston2019-02-183-32/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently whenever the current state changes in a room invalidate a lot of caches, which cause *a lot* of traffic over replication. Instead, lets batch up all those invalidations and send a single poke down the replication streams. Hopefully this will reduce load on the master process by substantially reducing traffic.
| * | | | Merge pull request #4652 from matrix-org/babolivier/acme-delegatedBrendan Abolivier2019-02-192-4/+20
| |\ \ \ \ | | | | | | | | | | | | Support .well-known delegation when issuing certificates through ACME
| | * | | | Improve config documentationBrendan Abolivier2019-02-191-3/+11
| | | | | |
| | * | | | Fetch ACME domain into an instance memberBrendan Abolivier2019-02-191-4/+5
| | | | | |
| | * | | | Use a configuration parameter to give the domain to generate a certificate forBrendan Abolivier2019-02-182-25/+11
| | | | | |
| | * | | | Merge branch 'develop' into babolivier/acme-delegatedBrendan Abolivier2019-02-188-40/+213
| | |\ \ \ \
| | * | | | | Typo in info logAndrew Morgan2019-02-181-1/+1
| | | | | | | | | | | | | | | | | | | | | Co-Authored-By: babolivier <contact@brendanabolivier.com>
| | * | | | | Various cosmetics to make TravisCI happyBrendan Abolivier2019-02-151-2/+5
| | | | | | |
| | * | | | | Remove unused importBrendan Abolivier2019-02-151-1/+0
| | | | | | |
| | * | | | | Support .well-known delegation when issuing certificates through ACMEBrendan Abolivier2019-02-151-4/+23
| | | | | | |
| * | | | | | Merge pull request #4670 from matrix-org/erikj/register_login_splitErik Johnston2019-02-192-41/+43
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Split /login into client_reader
| | * | | | | | Split /login into client_readerErik Johnston2019-02-182-41/+43
| | | | | | | |
| * | | | | | | Try and make TLS federation client code faster (#4674)Richard van der Hoff2019-02-191-8/+6
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix to use makeContext so that we don't need to rebuild the certificateoptions each time
| | * | | | | | | fix to use makeContext so that we don't need to rebuild the ↵Amber Brown2019-02-191-8/+6
| | | |_|_|/ / / | | |/| | | | | | | | | | | | | | | | | | | | | certificateoptions each time
| * | | | | | | Merge pull request #4642 from matrix-org/anoa/bans_room_upgradeAndrew Morgan2019-02-191-0/+22
| |\ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | | Transfer bans on room upgrade
| | * | | | | | lintAndrew Morgan2019-02-181-1/+3
| | | | | | | |
| | * | | | | | Move member event processing and changelog fixAndrew Morgan2019-02-181-9/+7
| | | | | | | |
| | * | | | | | Membership events are done laterAndrew Morgan2019-02-181-12/+20
| | | | | | | |
| | * | | | | | Transfer bans on room upgradeAndrew Morgan2019-02-181-0/+14
| | | | | | | |
| * | | | | | | Merge pull request #4632 from matrix-org/erikj/basic_sentryErik Johnston2019-02-183-1/+60
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Add basic optional sentry.io integration
| | * | | | | | | Fixup error handling and messageErik Johnston2019-02-181-5/+6
| | | | | | | | |
| | * | | | | | | FixupErik Johnston2019-02-181-1/+1
| | | | | | | | |
| | * | | | | | | Fixup comments and add warningErik Johnston2019-02-132-5/+10
| | | | | | | | |
| | * | | | | | | CommentsErik Johnston2019-02-121-0/+8
| | | | | | | | |
| | * | | | | | | Raise an appropriate error message if sentry_sdk missingErik Johnston2019-02-121-1/+15
| | | | | | | | |
| | * | | | | | | Basic sentry integrationErik Johnston2019-02-123-0/+31
| | | | | | | | |
| * | | | | | | | Merge pull request #4666 from matrix-org/erikj/register_login_splitErik Johnston2019-02-188-193/+419
| |\ \ \ \ \ \ \ \ | | | |_|/ / / / / | | |/| | | | | | Split out registration to worker
| | * | | | | | | Move register_device into handlerErik Johnston2019-02-184-102/+74
| | | | | | | | |
| | * | | | | | | Update docsErik Johnston2019-02-181-1/+1
| | | | | | | | |
| | * | | | | | | Split out registration to workerErik Johnston2019-02-187-147/+401
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows registration to be handled by a worker, though the actual write to the database still happens on master. Note: due to the in-memory session map all registration requests must be handled by the same worker.
| * | | | | | | | Merge pull request #4668 from matrix-org/erikj/catch_exceptionsErik Johnston2019-02-181-5/+9
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Correctly handle HttpResponseException when handling device updates
| | * | | | | | | | pep8Erik Johnston2019-02-181-1/+5
| | | | | | | | | |
| | * | | | | | | | Correctly handle HttpResponseExceptionErik Johnston2019-02-181-4/+4
| | | |_|_|_|/ / / | | |/| | | | | |
| * | | | | | | | Merge pull request #4669 from matrix-org/erikj/log_exceptionErik Johnston2019-02-181-3/+3
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Cleanup top level request exception logging
| | * | | | | | | | Cleanup top level request exception loggingErik Johnston2019-02-181-3/+3
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Firstly, we always logged that the request was being handled via `JsonResource._async_render`, so we change that to use the servlet name we add to the request. Secondly, we pass the exception information to the logger rather than formatting it manually. This makes it consistent with other exception logging, allwoing logging hooks and formatters to access the exception information.
| * | | | | | | | Merge pull request #4651 from matrix-org/matthew/well-known-corsErik Johnston2019-02-181-0/+3
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | set CORS on .well-known URI to unbreak modular
| | * | | | | | | | FixupErik Johnston2019-02-181-0/+1
| | | | | | | | | |
| | * | | | | | | | fix missig importMatthew Hodgson2019-02-181-0/+1
| | | | | | | | | |
| | * | | | | | | | tabsMatthew Hodgson2019-02-141-1/+1
| | | | | | | | | |
| | * | | | | | | | set CORS on .well-known URI to unbreak modularMatthew Hodgson2019-02-141-0/+1
| | | |_|_|_|/ / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | otherwise a riot/web running on foo.riot.im can't query the .well-known on foo.modular.im...
| * | | | | | | | Merge pull request #4667 from matrix-org/erikj/fix_revoke_guest_access_workersErik Johnston2019-02-182-1/+3
| |\ \ \ \ \ \ \ \ | | |_|/ / / / / / | |/| | | | | | | Fix kicking guest users in worker mode
| | * | | | | | | Fix kicking guest users in worker modeErik Johnston2019-02-182-1/+3
| | | |/ / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When guest_access changes from allowed to forbidden all local guest users should be kicked from the room. This did not happen when revocation was received from federation on a worker. Presumably broken in #4141
| * | | | | | | Merge pull request #4643 from matrix-org/erikj/catch_exceptionsErik Johnston2019-02-184-8/+19
| |\ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | | Correctly handle RequestSendFailed exceptions
| | * | | | | | Correctly handle RequestSendFailed exceptionsErik Johnston2019-02-144-8/+19
| | | |_|/ / / | | |/| | | | | | | | | | | | | | | | | | This mainly reduces the number of exceptions we log.
| * | | | | | config: Remove a repeated word from a logger warningJuuso "Linda" Lapinlampi2019-02-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The warning for missing macaroon_secret_key was "missing missing".
| * | | | | | Merge pull request #4647 from matrix-org/erikj/add_room_publishing_rulesErik Johnston2019-02-153-31/+193
| |\ \ \ \ \ \ | | |_|/ / / / | |/| | | | | Add configurable room list publishing rules
| | * | | | | Move defaults up into codeErik Johnston2019-02-151-22/+44
| | | | | | |
| | * | | | | Hoist up checks to reduce overall workErik Johnston2019-02-151-17/+15
| | | | | | |
| | * | | | | Fixup commentsErik Johnston2019-02-151-8/+27
| | | | | | |
| | * | | | | Clarify commentsErik Johnston2019-02-141-3/+3
| | | | | | |
| | * | | | | Clarify and fix behaviour when there are multiple aliasesErik Johnston2019-02-141-6/+18
| | | | | | |
| | * | | | | Only fetch aliases when publishing roomsErik Johnston2019-02-141-15/+15
| | | | | | |
| | * | | | | Fixup commentsErik Johnston2019-02-141-8/+15
| | | | | | |
| | * | | | | Add configurable room list publishing rulesErik Johnston2019-02-143-22/+126
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | This allows specifying who and what is allowed to be published onto the public room list
| * | | | | Merge branch 'master' into developRichard van der Hoff2019-02-143-7/+16
| |\ \ \ \ \
| | * | | | | 0.99.1.1 v0.99.1.1Richard van der Hoff2019-02-141-1/+1
| | | | | | |
| | * | | | | Fix errors in acme provisioning (#4648)Richard van der Hoff2019-02-142-6/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Better logging for errors on startup * Fix "TypeError: '>' not supported" when starting without an existing certificate * Fix a bug where an existing certificate would be reprovisoned every day
| * | | | | | Merge branch 'master' into developRichard van der Hoff2019-02-141-2/+2
| |\| | | | | | | |/ / / / | |/| | | |
| | * | | | 0.99.1 v0.99.1Richard van der Hoff2019-02-141-2/+2
| | | | | |
| | * | | | 0.99.1rc2 v0.99.1rc2Richard van der Hoff2019-02-131-1/+1
| | | | | |
| * | | | | Merge pull request #4450 from 14mRh4X0r/fix-dependency-messageRichard van der Hoff2019-02-131-2/+5
| |\ \ \ \ \ | | | | | | | | | | | | | | Fix error message for optional dependencies
| | * | | | | Fix error message for optional dependenciesWillem Mulder2019-01-231-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Willem Mulder <willemmaster@hotmail.com>
| * | | | | | Merge remote-tracking branch 'origin/release-v0.99.1' into developRichard van der Hoff2019-02-133-22/+34
| |\ \ \ \ \ \ | | | |/ / / / | | |/| | | |
| | * | | | | Update synapse/app/_base.pyErik Johnston2019-02-131-1/+1
| | | | | | | | | | | | | | | | | | | | | Co-Authored-By: richvdh <1389908+richvdh@users.noreply.github.com>
| | * | | | | Use `listen_tcp` for the replication listenerRichard van der Hoff2019-02-131-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the "can't listen on 0.0.0.0" error. Also makes it more consistent with what we do elsewhere.
| | * | | | | Improve logging around listening servicesRichard van der Hoff2019-02-132-11/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I wanted to bring listen_tcp into line with listen_ssl in terms of returning a list of ports, and wanted to check that was a safe thing to do - hence the logging in `refresh_certificate`. Also, pull the 'Synapse now listening' message up to homeserver.py, because it was being duplicated everywhere else.
| | * | | | | Special-case the default bind_addresses for metrics listenerRichard van der Hoff2019-02-132-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | turns out it doesn't really support ipv6, so let's hack around that by only listening on ipv4 by default.
| * | | | | | Run `black` on user directory code (#4635)Amber Brown2019-02-132-131/+116
| |/ / / / /
| * | | | | Update changelog and versionErik Johnston2019-02-121-1/+1
| | | | | |
| * | | | | Merge pull request #4608 from matrix-org/anoa/acls_room_upgradeErik Johnston2019-02-121-0/+1
| |\ \ \ \ \ | | |_|/ / / | |/| | | | Transfer Server ACLs on room upgrade
| | * | | | Transfer Server ACLs on room upgradeAndrew Morgan2019-02-111-0/+1
| | | | | |
| * | | | | Merge pull request #4627 from matrix-org/erikj/user_ips_analyzeErik Johnston2019-02-122-3/+31
| |\ \ \ \ \ | | | | | | | | | | | | | | Analyze user_ips before running deduplication
| | * | | | | Fix pep8Erik Johnston2019-02-121-1/+1
| | | | | | |
| | * | | | | Analyze user_ips before running deduplicationErik Johnston2019-02-122-3/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to the table locks taken out by the naive upsert, the table statistics may be out of date. During deduplication it is important that the correct index is used as otherwise a full table scan may be incorrectly used, which can end up thrashing the database badly.
| * | | | | | Merge pull request #4626 from matrix-org/erikj/fixup_user_ips_dedupeErik Johnston2019-02-121-3/+60
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Reduce user_ips bloat during dedupe background update
| | * | | | | | Reduce user_ips bloat during dedupe background updateErik Johnston2019-02-121-3/+60
| | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The background update to remove duplicate rows naively deleted and reinserted the duplicates. For large tables with a large number of duplicates this causes a lot of bloat (with postgres), as the inserted rows are appended to the table, since deleted rows will not be overwritten until a VACUUM has happened. This should hopefully also help ensure that the query in the last batch uses the correct index, as inserting a large number of new rows without analyzing will upset the query planner.
| * | | | | | Merge pull request #4625 from matrix-org/rav/fix_generate_config_warningsErik Johnston2019-02-121-7/+2
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | fix self-signed cert notice from generate-config
| | * | | | | | fix self-signed cert notice from generate-configRichard van der Hoff2019-02-121-7/+2
| | |/ / / / / | | | | | | | | | | | | | | | | | | | | | fixes #4620
| * | | | | | Disable TLS by default (#4614)Richard van der Hoff2019-02-122-27/+27
| | | | | | |
| * | | | | | Fix error when loading cert if tls is disabled (#4618)Richard van der Hoff2019-02-122-17/+45
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If TLS is disabled, it should not be an error if no cert is given. Fixes #4554.
| * | | | | fix testsRichard van der Hoff2019-02-111-1/+1
| | | | | |
| * | | | | Infer no_tls from presence of TLS listenersRichard van der Hoff2019-02-115-18/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than have to specify `no_tls` explicitly, infer whether we need to load the TLS keys etc from whether we have any TLS-enabled listeners.
| * | | | | Merge branch 'rav/no_create_server_contexts_if_no_tls' into rav/tls_cert/workRichard van der Hoff2019-02-112-3/+6
| |\ \ \ \ \
| | * | | | | Don't create server contexts when TLS is disabledRichard van der Hoff2019-02-112-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | we aren't going to use them anyway.
| * | | | | | Merge branch 'rav/tls_config_logging_fixes' into rav/tls_cert/workRichard van der Hoff2019-02-112-22/+38
| |\| | | | |
| | * | | | | Logging improvements around TLS certsRichard van der Hoff2019-02-112-22/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Log which file we're reading keys and certs from, and refactor the code a bit in preparation for other work
| * | | | | | Fail cleanly if listener config lacks a 'port'Richard van der Hoff2019-02-111-0/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | ... otherwise we would fail with a mysterious KeyError or something later.
| * | | | | Move ClientTLSOptionsFactory init out of refresh_certificates (#4611)Richard van der Hoff2019-02-113-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's nothing to do with refreshing the certificates. No idea why it was here.
| * | | | | Merge pull request #4580 from matrix-org/uhoreg/e2e_backup_add_updatingErik Johnston2019-02-113-5/+124
| |\ \ \ \ \ | | | | | | | | | | | | | | add updating of backup versions
| | * | | | | make sure version is in body and wrap in linearizer queueHubert Chathi2019-02-082-12/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | also add tests
| | * | | | | re-try to make isort happyHubert Chathi2019-02-061-1/+1
| | | | | | |
| | * | | | | fix import to make isort happyHubert Chathi2019-02-061-2/+7
| | | | | | |
| | * | | | | add new endpoint to update backup versionsHubert Chathi2019-02-063-1/+87
| | | | | | |
| | * | | | | return proper error codes for some 404sHubert Chathi2019-02-061-4/+16
| | | | | | |
| * | | | | | Clean up default listener configuration (#4586)Richard van der Hoff2019-02-111-48/+81
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rearrange the comments to try to clarify them, and expand on what some of it means. Use a sensible default 'bind_addresses' setting. For the insecure port, only bind to localhost, and enable x_forwarded, since apparently it's for use behind a load-balancer.
| * | | | | ACME Reprovisioning (#4522)Amber Brown2019-02-114-25/+88
| | | | | |
| * | | | | Be tolerant of blank TLS fingerprints config (#4589)Amber Brown2019-02-111-1/+5
| | | | | |
| * | | | | Merge pull request #4420 from matrix-org/jaywink/openid-listenerErik Johnston2019-02-115-45/+130
| |\ \ \ \ \ | | | | | | | | | | | | | | New listener resource for the federation API "openid/userinfo" endpoint
| | * | | | | Fix flake8 issuesJason Robinson2019-01-232-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Remove openid resource from default configJason Robinson2019-01-231-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead document it commented out. Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Document `servlet_groups` parametersJason Robinson2019-01-231-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Split federation OpenID userinfo endpoint out of the federation resourceJason Robinson2019-01-234-46/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows the OpenID userinfo endpoint to be active even if the federation resource is not active. The OpenID userinfo endpoint is called by integration managers to verify user actions using the client API OpenID access token. Without this verification, the integration manager cannot know that the access token is valid. The OpenID userinfo endpoint will be loaded in the case that either "federation" or "openid" resource is defined. The new "openid" resource is defaulted to active in default configuration. Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Make SynapseHomeServer _http_listener use self.get_reactor()Jason Robinson2019-01-231-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For all the homeserver classes, only the FrontendProxyServer passes its reactor when doing the http listen. Looking at previous PR's looks like this was introduced to make it possible to write a test, otherwise when you try to run a test with the test homeserver it tries to do a real bind to a port. Passing the reactor that the homeserver is instantiated with should probably be the right thing to do anyway? Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Make FederationReaderServer _http_listen use self.get_reactor()Jason Robinson2019-01-231-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For all the homeserver classes, only the FrontendProxyServer passes its reactor when doing the http listen. Looking at previous PR's looks like this was introduced to make it possible to write a test, otherwise when you try to run a test with the test homeserver it tries to do a real bind to a port. Passing the reactor that the homeserver is instantiated with should probably be the right thing to do anyway? Signed-off-by: Jason Robinson <jasonr@matrix.org>
| | * | | | | Add parameterized Python module to test dependenciesJason Robinson2019-01-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows running parameterized tests. BSD license. Signed-off-by: Jason Robinson <jasonr@matrix.org>
| * | | | | | Allow "unavailable" presence status for /sync (#4592)Valentin Anger2019-02-082-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Allow "unavailable" presence status for /sync Closes #3772, closes #3779 Signed-off-by: Valentin Anger <valentin.an.1999@gmail.com> * Add changelog for PR 4592
| * | | | | | Fix 'no unique or exclusion constraint' error (#4591)Richard van der Hoff2019-02-081-6/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add more tables to the list of tables which need a background update to complete before we can upsert into them, which fixes a race against the background updates.
| * | | | | | Deduplicate some code in synapse.app (#4567)Amber Brown2019-02-0813-158/+82
| | | | | | |
| * | | | | | Merge branch 'master' into developRichard van der Hoff2019-02-052-4/+4
| |\ \ \ \ \ \
| | * | | | | | v0.99.0 v0.99.0Richard van der Hoff2019-02-051-1/+1
| | | | | | | |
| | * | | | | | Fix default ACME config for py2 (#4564)Richard van der Hoff2019-02-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #4559
| | * | | | | | fix typo in config comments (#4557)Richard van der Hoff2019-02-051-2/+2
| | | | | | | |
| | * | | | | | 0.99.0rc4 v0.99.0rc4Richard van der Hoff2019-02-011-1/+1
| | | | | | | |
| * | | | | | | Filter user directory state query to a subset of state events (#4462)Andrew Morgan2019-02-051-4/+12
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Filter user directory state query to a subset of state events * Add changelog
| * | | | | | Merge pull request #4546 from ↵Richard van der Hoff2019-02-011-2/+15
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | matrix-org/rav/silence_critical_error_from_federation Fix noisy "twisted.internet.task.TaskStopped" errors in logs
| | * | | | | | Fix noisy "twisted.internet.task.TaskStopped" errors in logsRichard van der Hoff2019-02-011-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #4003
| * | | | | | | Merge remote-tracking branch 'origin/release-v0.99.0' into developRichard van der Hoff2019-02-011-20/+7
| |\ \ \ \ \ \ \ | | | |/ / / / / | | |/| | | | |
| | * | | | | | Merge pull request #4544 from matrix-org/rav/skip_invalid_well_knownRichard van der Hoff2019-02-011-19/+6
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Treat an invalid .well-known the same as an absent one
| | | * | | | | | Treat an invalid .well-known the same as an absent oneRichard van der Hoff2019-02-011-19/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... basically, carry on and fall back to SRV etc.
| | * | | | | | | Fix b'ab' noise in logsRichard van der Hoff2019-02-011-1/+1
| | |/ / / / / /
| * | | | | | | Merge branch 'release-v0.99.0' into developRichard van der Hoff2019-02-011-22/+63
| |\| | | | | |
| | * | | | | | Cache failures to parse .well-knownRichard van der Hoff2019-02-011-13/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add a Measure block around the .well-known fetch
| | * | | | | | better logging for federation connectionsRichard van der Hoff2019-01-311-5/+16
| | | | | | | |
| | * | | | | | Update federation routing logic to check .well-known before SRVRichard van der Hoff2019-01-311-5/+5
| | |/ / / / /
| * | | | | | Merge pull request #4530 from matrix-org/anoa/room_upgrade_federatableAndrew Morgan2019-01-312-11/+37
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Copy over non-federatable trait on room upgrade
| | * | | | | | Raise an exception instead of returning NoneAndrew Morgan2019-01-311-2/+9
| | | | | | | |
| | * | | | | | New function for getting room's create eventAndrew Morgan2019-01-312-18/+21
| | | | | | | |
| | * | | | | | Merge branch 'develop' of github.com:matrix-org/synapse into ↵Andrew Morgan2019-01-315-3/+34
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | anoa/room_upgrade_federatable
| | * | | | | | | lintAndrew Morgan2019-01-311-2/+4
| | | | | | | | |
| | * | | | | | | Copy over non-federatable trait on room upgradeAndrew Morgan2019-01-301-0/+14
| | | | | | | | |
| * | | | | | | | Merge branch 'release-v0.99.0' into developRichard van der Hoff2019-01-313-8/+44
| |\ \ \ \ \ \ \ \ | | |_|/ / / / / / | |/| | / / / / / | | | |/ / / / / | | |/| | | | |
| | * | | | | | v0.99.0rc3 v0.99.0rc3Richard van der Hoff2019-01-311-1/+1
| | | | | | | |
| | * | | | | | Add some debug for membership syncing issues (#4538)Richard van der Hoff2019-01-311-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | I can't figure out what's going on with #4422 and #4436; perhaps this will help.
| | * | | | | | Fix infinite loop when an event is redacted in a v3 room (#4535)Richard van der Hoff2019-01-311-5/+32
| | | | | | | |
| * | | | | | | Reject large transactions on federation (#4513)Andrew Morgan2019-01-311-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Reject large transactions on federation * Add changelog * lint * Simplify large transaction handling
| * | | | | | | Merge remote-tracking branch 'origin/release-v0.99.0' into developRichard van der Hoff2019-01-302-2/+2
| |\| | | | | |
| | * | | | | | v0.99.0rc2 v0.99.0rc2Richard van der Hoff2019-01-301-1/+1
| | | | | | | |
| | * | | | | | Merge remote-tracking branch 'origin/develop' into release-v0.99.0Richard van der Hoff2019-01-301-1/+13
| | |\ \ \ \ \ \