summary refs log tree commit diff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #2791 from matrix-org/erikj/media_storage_refactorErik Johnston2018-02-056-12/+156
|\ | | | | Ensure media is in local cache before thumbnailing
| * Add unit testsErik Johnston2018-01-183-0/+109
| |
| * Use better file consumerErik Johnston2018-01-181-5/+10
| |
| * Ensure media is in local cache before thumbnailingErik Johnston2018-01-183-10/+40
| |
* | Merge pull request #2845 from matrix-org/rav/urlcache_error_handlingRichard van der Hoff2018-02-021-21/+34
|\ \ | | | | | | Handle url_previews with no content-type
| * | Handle url_previews with no content-typeRichard van der Hoff2018-02-021-21/+34
| | | | | | | | | | | | | | | | | | | | | avoid failing with an exception if the remote server doesn't give us a Content-Type header. Also, clean up the exception handling a bit.
* | | Merge pull request #2842 from matrix-org/rav/state_resolution_handlerRichard van der Hoff2018-02-025-66/+119
|\ \ \ | | | | | | | | Factor out resolve_state_groups to a separate handler
| * | | Factor out resolve_state_groups to a separate handlerRichard van der Hoff2018-02-014-54/+108
| | | | | | | | | | | | | | | | | | | | | | | | We extract the storage-independent bits of the state group resolution out to a separate functiom, and stick it in a new handler, in preparation for its use from the storage layer.
| * | | Rename resolve_state_groups -> resolve_state_groups_for_eventsRichard van der Hoff2018-02-012-12/+11
| | | | | | | | | | | | | | | | (to make way for a method that actually just does the state group resolution)
* | | | Merge pull request #2841 from matrix-org/rav/refactor_calc_state_deltaRichard van der Hoff2018-02-021-18/+39
|\ \ \ \ | | | | | | | | | | factor _get_new_state_after_events out of _calculate_state_delta
| * | | | factor _get_new_state_after_events out of _calculate_state_deltaRichard van der Hoff2018-01-311-18/+39
| |/ / / | | | | | | | | | | | | This reduces the scope of a bunch of variables
* | | | Merge pull request #2836 from matrix-org/rav/resolve_state_events_docstringRichard van der Hoff2018-02-021-8/+28
|\ \ \ \ | | | | | | | | | | Docstring fixes
| * | | | doc arg types for _seperateRichard van der Hoff2018-02-011-0/+15
| | | | |
| * | | | More docstring fixesRichard van der Hoff2018-02-011-4/+4
| | | | | | | | | | | | | | | | | | | | Fix a couple of errors in docstrings
| * | | | Fix docstring for StateHandler.resolve_state_groupsRichard van der Hoff2018-02-011-4/+9
| | | | | | | | | | | | | | | | | | | | The return type was a complete lie, so fix it
* | | | | Merge pull request #2818 from turt2live/travis/admin-list-mediaRichard van der Hoff2018-02-023-56/+142
|\ \ \ \ \ | |_|_|/ / |/| | | | Add an admin route to get all the media in a room
| * | | | Merge branch 'develop' into travis/admin-list-mediaTravis Ralston2018-02-0147-386/+716
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #2837 from matrix-org/rav/fix_quarantine_mediaRichard van der Hoff2018-02-011-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | Fix sql error in quarantine_media
| * | | | Fix sql error in quarantine_mediaRichard van der Hoff2018-01-301-1/+1
| | |/ / | |/| |
* | | | Merge pull request #2838 from matrix-org/rav/fix_logging_on_dns_failRichard van der Hoff2018-01-311-2/+1
|\ \ \ \ | | | | | | | | | | Remove spurious log argument
| * | | | Remove spurious log argumentRichard van der Hoff2018-01-301-2/+1
| |/ / / | | | | | | | | | | | | ... which would cause scary-looking and unhelpful errors in the log on dns fail
* | | | Merge pull request #2835 from matrix-org/rav/remove_event_type_paramRichard van der Hoff2018-01-311-6/+14
|\ \ \ \ | | | | | | | | | | Remove unused "event_type" param on state.get_current_state_ids
| * | | | docstring for get_current_state_idsRichard van der Hoff2018-01-291-0/+13
| | | | |
| * | | | Remove unused "event_type" param on state.get_current_state_idsRichard van der Hoff2018-01-291-6/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | this param doesn't seem to be used, and is a bit pointless anyway because it can easily be replicated by the caller. It is also horrible, because it changes the return type of the method.
* | | | Merge pull request #2834 from ↵Richard van der Hoff2018-01-311-3/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | matrix-org/rav/better_persist_event_exception_handling Improve exception handling in persist_event
| * | | | Improve exception handling in persist_eventRichard van der Hoff2018-01-291-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. use `deferred.errback()` instead of `deferred.errback(e)`, which means that a Failure object will be constructed using the current exception state, *including* its stack trace - so the stack trace is saved in the Failure, leading to better exception reports. 2. Set `consumeErrors=True` on the ObservableDeferred, because we know that there will always be at least one observer - which avoids a spurious "CRITICAL: unhandled exception in Deferred" error in the logs
* | | | Merge pull request #2833 from matrix-org/rav/pusher_hacksRichard van der Hoff2018-01-311-7/+30
|\ \ \ \ | | | | | | | | | | Logging and metrics for the http pusher
| * | | | Better logging when pushes failRichard van der Hoff2018-01-291-2/+8
| | | | |
| * | | | add appid/device_display_name to to pusher loggingRichard van der Hoff2018-01-291-2/+2
| | | | |
| * | | | Fix logging and add user_idRichard van der Hoff2018-01-221-2/+3
| | | | |
| * | | | logging and debug for http pusherRichard van der Hoff2018-01-221-4/+20
| | | | |
* | | | | Merge pull request #2817 from matrix-org/rav/http_conn_poolRichard van der Hoff2018-01-311-1/+13
|\ \ \ \ \ | | | | | | | | | | | | Use a connection pool for the SimpleHttpClient
| * | | | | Increase http conn pool sizeRichard van der Hoff2018-01-291-1/+6
| |/ / / /
| * | | | Use a connection pool for the SimpleHttpClientRichard van der Hoff2018-01-201-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | In particular I hope this will help the pusher, which makes many requests to sygnal, and is currently negotiating SSL for each one.
* | | | | Script to move remote media to another media storeRichard van der Hoff2018-01-311-0/+133
| |/ / / |/| | |
* | | | Merge pull request #2831 from matrix-org/rav/fix-userdir-search-againRichard van der Hoff2018-01-273-7/+96
|\ \ \ \ | | | | | | | | | | Fix SQL for user search
| * | | | Add tests for user directory searchRichard van der Hoff2018-01-272-0/+89
| | | | |
| * | | | Fix SQL for user searchRichard van der Hoff2018-01-271-7/+7
|/ / / / | | | | | | | | | | | | | | | | | | | | fix some syntax errors for user search when search_all_users is enabled fixes #2801, hopefully
* | | | Merge pull request #2829 from matrix-org/rav/postgres_in_testsRichard van der Hoff2018-01-272-38/+47
|\ \ \ \ | | | | | | | | | | Make it possible to run tests against postgres
| * | | | Make it possible to run tests against postgresRichard van der Hoff2018-01-271-9/+34
| | | | |
| * | | | Create dbpool as normal in testsRichard van der Hoff2018-01-272-45/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... instead of creating our own special SQLiteMemoryDbPool, whose purpose was a bit of a mystery. For some reason this makes one of the tests run slightly slower, so bump the sleep(). Sorry.
| * | | | Run on_new_connection for unit testsRichard van der Hoff2018-01-271-7/+17
|/ / / / | | | | | | | | | | | | | | | | Configure the connectionpool used for unit tests to run the `on_new_connection` function.
* | | | Merge pull request #2828 from matrix-org/rav/kill_memory_datastoreRichard van der Hoff2018-01-271-151/+13
|\ \ \ \ | | | | | | | | | | Remove unused/bitrotted MemoryDataStore
| * | | | Remove unused/bitrotted MemoryDataStoreRichard van der Hoff2018-01-261-151/+13
| | | | | | | | | | | | | | | | | | | | This isn't used, and looks thoroughly bitrotted.
* | | | | Merge pull request #2830 from matrix-org/rav/factor_out_get_connRichard van der Hoff2018-01-2711-130/+17
|\ \ \ \ \ | | | | | | | | | | | | Factor out get_db_conn to HomeServer base class
| * | | | | Factor out get_db_conn to HomeServer base classRichard van der Hoff2018-01-2611-130/+17
| |/ / / / | | | | | | | | | | | | | | | | | | | | This function is identical to all subclasses, so we may as well push it up to the base class to reduce duplication (and make use of it in the tests)
* | | | | Merge pull request #2827 from ↵Neil Johnson2018-01-261-1/+2
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | matrix-org/fix_server_500_on_public_rooms_call_when_no_rooms_exist Fix server 500 on public rooms call when no rooms exist
| * | | | add white space lineNeil Johnson2018-01-261-0/+1
| | | | |
| * | | | rather than try reconstruct the results object, better to guard against the ↵Neil Johnson2018-01-261-6/+2
| | | | | | | | | | | | | | | | | | | | xrange step argument being 0
| * | | | fix return type, should be a dictNeil Johnson2018-01-251-1/+1
| | | | |
| * | | | fix PEP8 violationNeil Johnson2018-01-251-1/+1
| | | | |
| * | | | remove white spaceNeil Johnson2018-01-251-1/+0
| | | | |
| * | | | synapse 500s on a call to publicRooms in the case where the number of public ↵Neil Johnson2018-01-251-0/+5
| | | | | | | | | | | | | | | | | | | | rooms is zero, the specific cause is due to xrange trying to use a step value of zero, but if the total room number really is zero then it makes sense to just bail and save the extra processing
* | | | | Remove spurious unittest.DEBUGRichard van der Hoff2018-01-261-1/+0
| | | | |
* | | | | fix thinko on 3pid whitelistingMatthew Hodgson2018-01-241-2/+2
| | | | |
* | | | | Merge pull request #2816 from matrix-org/rav/metrics_commentsRichard van der Hoff2018-01-231-1/+6
|\ \ \ \ \ | | | | | | | | | | | | Add some comments about the reactor tick time metric
| * | | | | Add some comments about the reactor tick time metricRichard van der Hoff2018-01-191-1/+6
| | | | | |
* | | | | | add ?ts massaging for ASes (#2754)Matthew Hodgson2018-01-231-6/+11
| |/ / / / |/| | | | | | | | | blindly implement ?ts for AS. untested
* | | | | Merge pull request #2821 from matrix-org/rav/matthew_test_fixesRichard van der Hoff2018-01-221-2/+6
|\ \ \ \ \ | | | | | | | | | | | | Matthew's fixes to the unit tests
| * | | | | Matthew's fixes to the unit testsRichard van der Hoff2018-01-221-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | Extracted from https://github.com/matrix-org/synapse/pull/2820
* | | | | | Add federation_domain_whitelist option (#2820)Matthew Hodgson2018-01-2214-7/+146
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Add federation_domain_whitelist gives a way to restrict which domains your HS is allowed to federate with. useful mainly for gracefully preventing a private but internet-connected HS from trying to federate to the wider public Matrix network
* | | | | Merge pull request #2813 from matrix-org/matthew/registrations_require_3pidMatthew Hodgson2018-01-228-23/+178
|\ \ \ \ \ | |/ / / / |/| | | | add registrations_require_3pid and allow_local_3pids
| * | | | fix typo (thanks sytest)Matthew Hodgson2018-01-191-1/+1
| | | | |
| * | | | oops, check all login typesMatthew Hodgson2018-01-191-14/+11
| | | | |
| * | | | fix PR nitpickingMatthew Hodgson2018-01-191-3/+6
| | | | |
| * | | | trailing commasMatthew Hodgson2018-01-192-7/+7
| | | | |
| * | | | rewrite based on PR feedback:Matthew Hodgson2018-01-197-89/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * [ ] split config options into allowed_local_3pids and registrations_require_3pid * [ ] simplify and comment logic for picking registration flows * [ ] fix docstring and move check_3pid_allowed into a new util module * [ ] use check_3pid_allowed everywhere @erikjohnston PTAL
| * | | | fix up v1, and improve errorsMatthew Hodgson2018-01-194-20/+65
| | | | |
| * | | | fix pep8Matthew Hodgson2018-01-191-2/+1
| | | | |
| * | | | mock registrations_require_3pidMatthew Hodgson2018-01-191-0/+1
| | | | |
| * | | | add registrations_require_3pidMatthew Hodgson2018-01-195-13/+110
| | | | | | | | | | | | | | | | | | | | | | | | | lets homeservers specify a whitelist for 3PIDs that users are allowed to associate with. Typically useful for stopping people from registering with non-work emails
| | | * | pep8Travis Ralston2018-01-311-9/+12
| | | | | | | | | | | | | | | Signed-off-by: Travis Ralston <travpc@gmail.com>
| | | * | Add admin api documentation for list media endpointTravis Ralston2018-01-311-0/+23
| | | | | | | | | | | | | | | Signed-off-by: Travis Ralston <travpc@gmail.com>
| | | * | Documentation and namingTravis Ralston2018-01-311-7/+26
| | | | | | | | | | | | | | | Signed-off-by: Travis Ralston <travpc@gmail.com>
| | | * | Appease the linterTravis Ralston2018-01-201-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | These are ids anyways, not mxc uris. Signed-off-by: Travis Ralston <travpc@gmail.com>
| | | * | Add an admin route to get all the media in a roomTravis Ralston2018-01-202-56/+97
| |_|/ / |/| | | | | | | | | | | | | | | This is intended to be used by administrators to monitor the media that is passing through their server, if they wish. Signed-off-by: Travis Ralston <travpc@gmail.com>
* | | | Merge pull request #2814 from matrix-org/rav/fix_urlcache_thumbsRichard van der Hoff2018-01-191-0/+8
|\ \ \ \ | | | | | | | | | | Use the right path for url_preview thumbnails
| * | | | Use the right path for url_preview thumbnailsRichard van der Hoff2018-01-191-0/+8
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | This was introduced by #2627: we were overwriting the original media for url previews with the thumbnails :/ (fixes https://github.com/vector-im/riot-web/issues/6012, hopefully)
* | | | Fix storage provider bug introduced when renamed to store_localErik Johnston2018-01-181-1/+1
| | | |
* | | | Merge pull request #2812 from matrix-org/erikj/media_storage_provider_configErik Johnston2018-01-183-33/+101
|\ \ \ \ | | | | | | | | | | Make storage providers configurable
| * | | | Fix passing wrong config to provider constructorErik Johnston2018-01-181-1/+1
| | | | |
| * | | | Remove duplicate directory testErik Johnston2018-01-181-4/+0
| | | | |
| * | | | Missing staticmethodErik Johnston2018-01-181-0/+1
| | | | |
| * | | | Fixup commentsErik Johnston2018-01-182-3/+7
| | | | |
| * | | | Make storage providers more configurableErik Johnston2018-01-183-31/+98
| | | | |
* | | | | Merge pull request #2804 from matrix-org/erikj/file_consumerErik Johnston2018-01-182-0/+315
|\ \ \ \ \ | | |_|_|/ | |/| | | Add decent impl of a FileConsumer
| * | | | Do logcontexts correctlyErik Johnston2018-01-181-2/+2
| | | | |
| * | | | Move test stuff to testsErik Johnston2018-01-182-33/+47
| | | | |
| * | | | Make all fields privateErik Johnston2018-01-181-31/+31
| | | | |
| * | | | Ensure we registerProducer isn't called twiceErik Johnston2018-01-181-0/+3
| | | | |
| * | | | Fix _notify_empty typoErik Johnston2018-01-181-1/+1
| | | | |
| * | | | Move definition of paused_producer to __init__Erik Johnston2018-01-181-2/+4
| | | | |
| * | | | Fix commentsErik Johnston2018-01-181-3/+3
| | | | |
| * | | | Add decent impl of a FileConsumerErik Johnston2018-01-172-0/+296
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Twisted core doesn't have a general purpose one, so we need to write one ourselves. Features: - All writing happens in background thread - Supports both push and pull producers - Push producers get paused if the consumer falls behind
* | | | Fix typo in thumbnail resource causing access times to be incorrectErik Johnston2018-01-181-2/+2
| | | |
* | | | Merge pull request #2810 from matrix-org/rav/metrics_fixesRichard van der Hoff2018-01-181-2/+4
|\ \ \ \ | | |/ / | |/| | Fix bugs in block metrics
| * | | Fix bugs in block metricsRichard van der Hoff2018-01-181-2/+4
|/ / / | | | | | | | | | ... which I introduced in #2785
* | | Merge pull request #2809 from matrix-org/rav/metrics_errorsRichard van der Hoff2018-01-181-1/+8
|\ \ \ | |_|/ |/| | better exception logging in callbackmetrics
| * | better exception logging in callbackmetricsRichard van der Hoff2018-01-181-1/+8
| | | | | | | | | | | | when we fail to render a metric, give a clue as to which metric it was
* | | Merge pull request #2805 from matrix-org/rav/log_state_resRichard van der Hoff2018-01-171-0/+6
|\ \ \ | |/ / |/| | Log room when doing state resolution
| * | Log room when doing state resolutionRichard van der Hoff2018-01-171-0/+6
| | | | | | | | | | | | Mostly because it helps figure out what is prompting the resolution
* | | Merge pull request #2783 from matrix-org/erikj/media_last_accessedErik Johnston2018-01-175-11/+64
|\ \ \ | | | | | | | | Keep track of last access time for local media
| * | | Remove lost commentErik Johnston2018-01-171-3/+0
| | | |
| * | | Update last access time when thumbnails are viewedErik Johnston2018-01-171-0/+2
| | | |
| * | | Keep track of last access time for local mediaErik Johnston2018-01-174-11/+65
| | |/ | |/|
* | | Merge pull request #2803 from matrix-org/matthew/fix-userdir-sqlMatthew Hodgson2018-01-171-2/+7
|\ \ \ | | | | | | | | fix SQL when searching all users
| * | | fix SQL when searching all usersMatthew Hodgson2018-01-171-2/+7
| | | |
* | | | Merge pull request #2802 from matrix-org/rav/clean_up_resolve_eventsRichard van der Hoff2018-01-172-20/+29
|\ \ \ \ | |/ / / |/| | | Split resolve_events into two functions
| * | | Split resolve_events into two functionsRichard van der Hoff2018-01-172-20/+29
|/ / / | | | | | | | | | ... so that the return type doesn't depend on the arg types
* | | Merge pull request #2795 from matrix-org/rav/track_db_schedulingRichard van der Hoff2018-01-175-4/+43
|\ \ \ | | | | | | | | Track DB scheduling delay per-request
| * \ \ Merge remote-tracking branch 'origin/develop' into rav/track_db_schedulingRichard van der Hoff2018-01-175-34/+78
| |\ \ \
| * | | | Track DB scheduling delay per-requestRichard van der Hoff2018-01-165-4/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For each request, track the amount of time spent waiting for a db connection. This entails adding it to the LoggingContext and we may as well add metrics for it while we are passing.
| * | | | Merge branch 'rav/db_txn_time_in_millis' into rav/track_db_schedulingRichard van der Hoff2018-01-166-31/+49
| |\ \ \ \
* | \ \ \ \ Merge pull request #2797 from matrix-org/rav/user_id_checkingRichard van der Hoff2018-01-172-8/+19
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Sanity checking for user ids
| * | | | | Sanity checking for user idsRichard van der Hoff2018-01-172-8/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check the user_id passed to a couple of APIs for validity, to avoid "IndexError: list index out of range" exception which looks scary and results in a 500 rather than a more useful error. Fixes #1432, among other things
* | | | | | Merge pull request #2793 from matrix-org/rav/db_txn_time_in_millisRichard van der Hoff2018-01-174-10/+17
|\ \ \ \ \ \ | | | | | | | | | | | | | | Track db txn time in millisecs
| * | | | | | Merge branch 'develop' into rav/db_txn_time_in_millisRichard van der Hoff2018-01-174-25/+69
| |\| | | | | | | |_|/ / / | |/| | | |
| * | | | | Track db txn time in millisecsRichard van der Hoff2018-01-164-10/+17
| | | | | | | | | | | | | | | | | | | | | | | | ... to reduce the amount of floating-point foo we do.
* | | | | | Merge pull request #2796 from matrix-org/rav/fix_closed_connection_errorsRichard van der Hoff2018-01-171-9/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix 'NoneType' object has no attribute 'writeHeaders'
| * \ \ \ \ \ Merge branch 'develop' into rav/fix_closed_connection_errorsRichard van der Hoff2018-01-177-47/+102
| |\ \ \ \ \ \ | | | |/ / / / | | |/| | | |
| * | | | | | Fix 'NoneType' object has no attribute 'writeHeaders'Richard van der Hoff2018-01-161-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid throwing a (harmless) exception when we try to write an error response to an http request where the client has disconnected. This comes up as a CRITICAL error in the logs which tends to mislead people into thinking there's an actual problem
* | | | | | | Merge pull request #2794 from matrix-org/rav/rework_run_interactionRichard van der Hoff2018-01-171-20/+31
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | rework runInteraction in terms of runConnection
| * | | | | | Merge branch 'develop' into rav/rework_run_interactionRichard van der Hoff2018-01-177-47/+102
| |\ \ \ \ \ \ | |/ / / / / / |/| | | / / / | | |_|/ / / | |/| | | |
* | | | | | Merge pull request #2792 from matrix-org/rav/optimise_logging_contextRichard van der Hoff2018-01-173-22/+33
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Optimise LoggingContext creation and copying
| * | | | | Merge remote-tracking branch 'origin/develop' into rav/optimise_logging_contextRichard van der Hoff2018-01-174-25/+69
| |\ \ \ \ \ | |/ / / / / |/| | | / / | | |_|/ / | |/| | |
* | | | | Merge pull request #2789 from matrix-org/erikj/fix_thumbnailsErik Johnston2018-01-173-24/+68
|\ \ \ \ \ | | | | | | | | | | | | Fix thumbnailing remote files
| * | | | | Add docstringErik Johnston2018-01-171-1/+1
| | | | | |
| * | | | | Use local varsErik Johnston2018-01-161-6/+2
| | | | | |
| * | | | | Change _generate_thumbnails to take media_typeErik Johnston2018-01-162-7/+10
| | | | | |
| * | | | | Move setting of file_id up to callerErik Johnston2018-01-162-13/+14
| | | | | |
| * | | | | Fix typoErik Johnston2018-01-161-7/+10
| | | | | |
| * | | | | Fix up log linesErik Johnston2018-01-162-7/+10
| | | | | |
| * | | | | Correctly use server_name/file_id when generating/fetching remote thumbnailsErik Johnston2018-01-162-5/+8
| | | | | |
| * | | | | Log when we respond with 404Erik Johnston2018-01-162-1/+8
| | | | | |
| * | | | | Fix thumbnailing remote filesErik Johnston2018-01-162-2/+30
| | | | | |
* | | | | | fix typoMatthew Hodgson2018-01-161-1/+1
| |_|_|/ / |/| | | |
| | * | | Optimise LoggingContext creation and copyingRichard van der Hoff2018-01-163-22/+33
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | It turns out that the only thing we use the __dict__ of LoggingContext for is `request`, and given we create lots of LoggingContexts and then copy them every time we do a db transaction or log line, using the __dict__ seems a bit redundant. Let's try to optimise things by making the request attribute explicit.
| | * | rework runInteraction in terms of runConnectionRichard van der Hoff2018-01-161-20/+31
| |/ / |/| | | | | | | | ... so that we can share the code
* | | Merge pull request #2785 from matrix-org/rav/reorganise_metrics_againRichard van der Hoff2018-01-165-47/+209
|\ \ \ | | | | | | | | Reorganise request and block metrics
| * | | document metrics changesRichard van der Hoff2018-01-162-8/+58
| | | |
| * | | Reorganise request and block metricsRichard van der Hoff2018-01-152-25/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to circumvent the number of duplicate foo:count metrics increasing without bounds, it's time for a rearrangement. The following are all deprecated, and replaced with synapse_util_metrics_block_count: synapse_util_metrics_block_timer:count synapse_util_metrics_block_ru_utime:count synapse_util_metrics_block_ru_stime:count synapse_util_metrics_block_db_txn_count:count synapse_util_metrics_block_db_txn_duration:count The following are all deprecated, and replaced with synapse_http_server_response_count: synapse_http_server_requests synapse_http_server_response_time:count synapse_http_server_response_ru_utime:count synapse_http_server_response_ru_stime:count synapse_http_server_response_db_txn_count:count synapse_http_server_response_db_txn_duration:count The following are renamed (the old metrics are kept for now, but deprecated): synapse_util_metrics_block_timer:total -> synapse_util_metrics_block_time_seconds synapse_util_metrics_block_ru_utime:total -> synapse_util_metrics_block_ru_utime_seconds synapse_util_metrics_block_ru_stime:total -> synapse_util_metrics_block_ru_stime_seconds synapse_util_metrics_block_db_txn_count:total -> synapse_util_metrics_block_db_txn_count synapse_util_metrics_block_db_txn_duration:total -> synapse_util_metrics_block_db_txn_duration_seconds synapse_http_server_response_time:total -> synapse_http_server_response_time_seconds synapse_http_server_response_ru_utime:total -> synapse_http_server_response_ru_utime_seconds synapse_http_server_response_ru_stime:total -> synapse_http_server_response_ru_stime_seconds synapse_http_server_response_db_txn_count:total -> synapse_http_server_response_db_txn_count synapse_http_server_response_db_txn_duration:total synapse_http_server_response_db_txn_duration_seconds
| * | | mechanism to render metrics with alternative namesRichard van der Hoff2018-01-151-13/+40
| | | |
| * | | Add some comments to metrics classesRichard van der Hoff2018-01-151-1/+27
| | | |
* | | | Merge pull request #2790 from matrix-org/rav/preserve_event_logcontext_leakRichard van der Hoff2018-01-161-1/+8
|\ \ \ \ | | | | | | | | | | Fix a logcontext leak in persist_events
| * | | | Fix a logcontext leak in persist_eventsRichard van der Hoff2018-01-161-1/+8
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ObserveableDeferred expects its callbacks to be called without any logcontexts, whereas it turns out we were calling them with the logcontext of the request which initiated the persistence loop. It seems wrong that we are attributing work done in the persistence loop to the request that happened to initiate it, so let's solve this by dropping the logcontext for it. (I'm not sure this actually causes any real problems other than messages in the debug log, but let's clean it up anyway)
* | | | Merge pull request #2787 from matrix-org/rav/worker_event_countsRichard van der Hoff2018-01-162-0/+11
|\ \ \ \ | | | | | | | | | | Metrics for events processed in appservice and fed sender
| * | | | Metrics for events processed in appservice and fed senderRichard van der Hoff2018-01-152-0/+11
| |/ / / | | | | | | | | | | | | More metrics I wished I'd had
* | | | Merge pull request #2786 from matrix-org/rav/rdata_metricsRichard van der Hoff2018-01-161-5/+14
|\ \ \ \ | |_|/ / |/| | | Metrics for number of RDATA commands received
| * | | Metrics for number of RDATA commands receivedRichard van der Hoff2018-01-151-5/+14
| |/ / | | | | | | | | | I found myself wishing we had this.
* | | Merge pull request #2767 from matrix-org/erikj/media_storage_refactorErik Johnston2018-01-167-346/+843
|\ \ \ | |/ / |/| | Refactor MediaRepository to separate out storage
| * | Fix up commentsErik Johnston2018-01-122-7/+16
| | |
| * | Correctly reraise exceptionErik Johnston2018-01-121-2/+4
| | |
| * | Make Responder a context managerErik Johnston2018-01-122-9/+10
| | |
| * | Add missing class varErik Johnston2018-01-121-0/+3
| | |
| * | Merge branch 'develop' of github.com:matrix-org/synapse into ↵Erik Johnston2018-01-129-28/+116
| |\ \ | | | | | | | | | | | | erikj/media_storage_refactor
| * | | Remove unnecessary conditionErik Johnston2018-01-121-4/+1
| | | |
| * | | Remove unused variablesErik Johnston2018-01-121-3/+0
| | | |
| * | | Make class var localErik Johnston2018-01-121-3/+3
| | | |
| * | | CommentsErik Johnston2018-01-123-15/+65
| | | |
| * | | Add StorageProvider conceptErik Johnston2018-01-093-9/+162
| | | |
| * | | Make PreviewUrlResource use MediaStorageErik Johnston2018-01-092-8/+14
| | | |
| * | | Make ThumbnailResource use MediaStorageErik Johnston2018-01-092-48/+68
| | | |
| * | | Use MediaStorage for remote mediaErik Johnston2018-01-092-136/+156
| | | |
| * | | Use MediaStorage for local filesErik Johnston2018-01-092-119/+73
| | | |
| * | | Add MediaStorage classErik Johnston2018-01-091-0/+198
| | | |
| * | | Add some helper classesErik Johnston2018-01-091-0/+73
| | | |
| * | | Split out add_file_headersErik Johnston2018-01-091-28/+42
| | | |
* | | | Merge pull request #2778 from matrix-org/rav/counters_should_be_floatsRichard van der Hoff2018-01-151-3/+10
|\ \ \ \ | |_|_|/ |/| | | Make Counter render floats
| * | | Make Counter render floatsRichard van der Hoff2018-01-121-3/+10
|/ / / | | | | | | | | | | | | | | | | | | | | | Prometheus handles all metrics as floats, and sometimes we store non-integer values in them (notably, durations in seconds), so let's render them as floats too. (Note that the standard client libraries also treat Counters as floats.)
* | | Merge pull request #2777 from matrix-org/rav/fix_remote_thumbnailsRichard van der Hoff2018-01-121-0/+5
|\ \ \ | | | | | | | | Reinstate media download on thumbnail request
| * | | Reinstate media download on thumbnail requestRichard van der Hoff2018-01-121-0/+5
|/ / / | | | | | | | | | | | | We need to actually download the remote media when we get a request for a thumbnail.
* | | Merge pull request #2774 from matrix-org/erikj/synctlErik Johnston2018-01-123-5/+38
|\ \ \ | |_|/ |/| | When using synctl with workers, don't start the main synapse automatically
| * | Actually make it workErik Johnston2018-01-122-2/+15
| | |
| * | RefactorErik Johnston2018-01-121-9/+4
| | |
| * | When using synctl with workers, don't start the main synapse automaticallyErik Johnston2018-01-112-8/+33
| | |
* | | Merge pull request #2766 from matrix-org/rav/room_eventRichard van der Hoff2018-01-111-3/+26
|\ \ \ | |/ / |/| | Add /room/{id}/event/{id} to synapse
| * | Add /room/{id}/event/{id} to synapseRichard van der Hoff2018-01-091-3/+26
| | | | | | | | | | | | | | | | | | Turns out that there is a valid usecase for retrieving event by id (notably having received a push), but event ids should be scoped to room, so /event/{id} is wrong.
* | | Merge branch 'master' of github.com:matrix-org/synapse into developErik Johnston2018-01-113-1/+49
|\ \ \
| * \ \ Merge branch 'release-v0.26.0' of github.com:matrix-org/synapse v0.26.0Erik Johnston2018-01-0577-826/+1829
| |\ \ \
| | * | | Bump version and changelog github/release-v0.26.0 release-v0.26.0Erik Johnston2018-01-052-1/+7
| | | | |
| | * | | Fix rc version number v0.26.0-rc1Erik Johnston2017-12-131-2/+2
| | | | |
| | * | | Bump changelog and versionErik Johnston2017-12-132-1/+38
| | | | |
| * | | | Merge pull request #2737 from Valodim/masterMatthew Hodgson2017-12-171-0/+5
| |\ \ \ \ | | | | | | | | | | | | mention federation tester more prominently in the readme
| | * | | | mention federation tester more prominently in the readmeVincent Breitmoser2017-12-161-0/+5
| |/ / / /
* | | | | Merge pull request #2773 from matrix-org/erikj/hash_bgErik Johnston2018-01-103-10/+18
|\ \ \ \ \ | | | | | | | | | | | | Do bcrypt hashing in a background thread
| * | | | | Do bcrypt hashing in a background threadErik Johnston2018-01-103-10/+18
| | | | | |
* | | | | | Merge pull request #2772 from matrix-org/t3chguy/patch-1Erik Johnston2018-01-101-3/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix publicised groups GET API (singular) over federation
| * | | | | | fix order of operations derp and also use `.get` to default to {}Michael Telatynski2018-01-101-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
| * | | | | | Fix publicised groups API (singular) over federationMichael Telatynski2018-01-101-4/+4
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | which was missing its fed client API, since there is no other API it might as well reuse the bulk one and unwrap it Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
* | | | | | Merge pull request #2770 from matrix-org/rav/fix_request_metricsRichard van der Hoff2018-01-101-11/+19
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Update http request metrics before calling servlet
| * | | | | Update http request metrics before calling servletRichard van der Hoff2018-01-091-11/+19
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | Make sure that we set the servlet name in the metrics object *before* calling the servlet, in case the servlet throws an exception.
* | | | | Merge pull request #2764 from matrix-org/erikj/remove_dead_thumbnail_codeErik Johnston2018-01-092-78/+3
|\ \ \ \ \ | |/ / / / |/| | | | Remove dead code related to default thumbnails
| * | | | Remove dead TODOErik Johnston2018-01-091-2/+0
| | | | |
| * | | | Remove dead code related to default thumbnailsErik Johnston2018-01-092-76/+3
|/ / / /
* | | | Merge pull request #2765 from matrix-org/rav/fix_room_utsRichard van der Hoff2018-01-093-4/+9
|\ \ \ \ | | | | | | | | | | Fix flaky test_rooms UTs
| * | | | enable twisted delayedcall debugging in UTsRichard van der Hoff2018-01-091-1/+5
| | | | |
| * | | | Remove dead codeRichard van der Hoff2018-01-091-3/+0
| | | | | | | | | | | | | | | | | | | | pointless function is pointless
| * | | | Disable user_directory updates for UTsRichard van der Hoff2018-01-091-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Fix flakiness in the UTs caused by the user_directory being updated in the background
* | | | | Merge pull request #2763 from matrix-org/rav/fix_config_utsRichard van der Hoff2018-01-092-3/+18
|\ \ \ \ \ | | | | | | | | | | | | Fix broken config UTs
| * | | | | Fix broken config UTsRichard van der Hoff2018-01-092-3/+18
| |/ / / / | | | | | | | | | | | | | | | | | | | | https://github.com/matrix-org/synapse/pull/2755 broke log-config generation, which in turn broke the unit tests.
* | | | | Merge pull request #2762 from matrix-org/rav/fix_logconfig_indentingRichard van der Hoff2018-01-091-17/+17
|\ \ \ \ \ | |/ / / / |/| | | | Make indentation of generated log config consistent
| * | | | Make indentation of generated log config consistentRichard van der Hoff2018-01-091-17/+17
|/ / / / | | | | | | | | | | | | (we had a mix of 2- and 4-space indents)
* | | | Merge pull request #2761 from turt2live/patch-1Erik Johnston2018-01-081-1/+1
|\ \ \ \ | | | | | | | | | | Fix templating error with unban permission message
| * | | | Fix templating error with unban permission messageTravis Ralston2018-01-071-1/+1
|/ / / / | | | | | | | | | | | | | | | | Fixes https://github.com/matrix-org/synapse/issues/2759 Signed-off-by: Travis Ralston <travpc@gmail.com>
* | | | Merge pull request #2755 from matrix-org/richvdh-patch-1Richard van der Hoff2018-01-051-8/+4
|\ \ \ \ | | | | | | | | | | Remove 'verbosity'/'log_file' from generated cfg
| * | | | Remove 'verbosity'/'log_file' from generated cfgRichard van der Hoff2018-01-051-8/+4
|/ / / / | | | | | | | | | | | | | | | | | | | | ... because these only really exist to confuse people nowadays. Also bring log config more into line with the generated log config, by making `level_for_storage` apply to the `synapse.storage.SQL` logger rather than `synapse.storage`.
* | | | Merge pull request #2744 from matrix-org/rav/login_loggingRichard van der Hoff2018-01-051-4/+10
|\ \ \ \ | | | | | | | | | | Better logging when login can't find a 3pid
| * | | | Better logging when login can't find a 3pidRichard van der Hoff2017-12-201-4/+10
| | | | |
* | | | | Merge pull request #2745 from matrix-org/rav/assert_paramsRichard van der Hoff2018-01-053-60/+68
|\ \ \ \ \ | |/ / / / |/| | | | Check missing fields in event_from_pdu_json
| * | | | Check missing fields in event_from_pdu_jsonRichard van der Hoff2017-12-301-0/+7
| | | | | | | | | | | | | | | | | | | | Return a 400 rather than a 500 when somebody messes up their send_join
| * | | | Factor out `event_from_pdu_json`Richard van der Hoff2017-12-303-37/+44
| | | | | | | | | | | | | | | | | | | | | | | | | turns out we have two copies of this, and neither needs to be an instance method
| * | | | federation_server: clean up importsRichard van der Hoff2017-12-301-14/+11
| | | | |
| * | | | federation_client: clean up importsRichard van der Hoff2017-12-301-13/+10
|/ / / /
* | | | Merge pull request #2683 from seckrv/fix_pwd_auth_prov_typoRichard van der Hoff2017-12-181-2/+2
|\ \ \ \ | | | | | | | | | | synapse/config/password_auth_providers: Fixed bracket typo
| * | | | synapse/config/password_auth_providers: Fixed bracket typoRichard von Seck2017-11-161-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Richard von Seck <richard.von-seck@gmx.net>
* | | | | Merge pull request #2435 from silkeh/listen-ipv6-defaultRichard van der Hoff2017-12-1812-231/+247
|\ \ \ \ \ | |_|_|_|/ |/| | | | Adapt the default config to bind on both IPv4 and IPv6 on all platforms
| * | | | Implement listen_tcp method in remaining workersSilke2017-12-1810-192/+155
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Silke <silke@slxh.eu>
| * | | | Remove logger argument and do not catch replication listenerSilke2017-12-182-24/+18
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Silke <silke@slxh.eu>
| * | | | Add methods for listening on multiple addressesSilke Hofstra2017-12-172-51/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add listen_tcp and listen_ssl which implement Twisted's reactor.listenTCP and reactor.listenSSL for multiple addresses. Signed-off-by: Silke Hofstra <silke@slxh.eu>
| * | | | Allow binds to both :: and 0.0.0.0Silke Hofstra2017-12-172-42/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Binding on 0.0.0.0 when :: is specified in the bind_addresses is now allowed. This causes a warning explaining the behaviour. Configuration changed to match. See #2232 Signed-off-by: Silke Hofstra <silke@slxh.eu>
| * | | | Adapt the default config to bind on IPv6.Willem Mulder2017-12-171-8/+9
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most deployments are on Linux (or Mac OS), so this would actually bind on both IPv4 and IPv6. Resolves #1886. Signed-off-by: Willem Mulder <willemmaster@hotmail.com>
* | | | Merge pull request #2717 from matrix-org/erikj/createroom_contentErik Johnston2017-12-072-5/+9
|\ \ \ \ | | | | | | | | | | Fix wrong avatars when inviting multiple users when creating room
| * \ \ \ Merge branch 'develop' of github.com:matrix-org/synapse into ↵Erik Johnston2017-12-0773-820/+1776
| |\ \ \ \ | |/ / / / |/| | | | | | | | | erikj/createroom_content
* | | | | Merge pull request #2723 from matrix-org/matthew/search-all-local-usersMatthew Hodgson2017-12-0511-37/+234
|\ \ \ \ \ | | | | | | | | | | | | Add all local users to the user_directory and optionally search them
| * | | | | fix StoreError syntaxMatthew Hodgson2017-12-051-1/+1
| | | | | |
| * | | | | speed up the rate of initial spam for usersMatthew Hodgson2017-12-041-11/+12
| | | | | |
| * | | | | better docMatthew Hodgson2017-12-041-1/+4
| | | | | |
| * | | | | try make tests work a bit more...Matthew Hodgson2017-12-041-0/+12
| | | | | |
| * | | | | erik told me toMatthew Hodgson2017-12-041-1/+1
| | | | | |
| * | | | | fix pep8 and testsMatthew Hodgson2017-12-043-8/+6
| | | | | |
| * | | | | switch to a simpler 'search_all_users' button as per review feedbackMatthew Hodgson2017-12-045-37/+30
| | | | | |
| * | | | | Merge branch 'develop' into matthew/search-all-local-usersMatthew Hodgson2017-11-3032-200/+633
| |\ \ \ \ \ | | | |_|/ / | | |/| | |
| * | | | | specify default user_directory_include_patternMatthew Hodgson2017-11-301-0/+1
| | | | | |
| * | | | | remove null constraint on user_dir.room_idMatthew Hodgson2017-11-301-0/+35
| | | | | |
| * | | | | fix alternation operator for FTS4 - how did this ever work!?Matthew Hodgson2017-11-301-1/+1
| | | | | |
| * | | | | fix thinkos galoreMatthew Hodgson2017-11-303-20/+41
| | | | | |
| * | | | | kick the user_directory index when new users registerMatthew Hodgson2017-11-291-0/+8
| | | | | |
| * | | | | untested WIP but might actually workMatthew Hodgson2017-11-296-20/+96
| | | | | |
| * | | | | Add user_directory_include_pattern config param to expand search results to ↵Matthew Hodgson2017-11-295-9/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | additional users Initial commit; this doesn't work yet - the LIKE filtering seems too aggressive. It also needs _do_initial_spam to be aware of prepopulating the whole user_directory_search table with all users... ...and it needs a handle_user_signup() or something to be added so that new signups get incrementally added to the table too. Committing it here as a WIP
* | | | | | Fix error when deleting devicesRichard van der Hoff2017-12-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This was introduced in d7ea8c4 / PR #2728
* | | | | | Merge pull request #2729 from matrix-org/rav/custom_ui_authRichard van der Hoff2017-12-051-24/+57
|\ \ \ \ \ \ | | | | | | | | | | | | | | support custom login types for validating users
| * | | | | | support custom login types for validating usersRichard van der Hoff2017-12-051-24/+57
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Wire the custom login type support from password providers into the UI-auth user-validation flows.
* | | | | | Merge pull request #2728 from matrix-org/rav/validate_user_via_ui_authRichard van der Hoff2017-12-053-74/+102
|\ \ \ \ \ \ | | | | | | | | | | | | | | Factor out a validate_user_via_ui_auth method
| * | | | | | Factor out a validate_user_via_ui_auth methodRichard van der Hoff2017-12-053-74/+102
|/ / / / / / | | | | | | | | | | | | | | | | | | Collect together all the places that validate a logged-in user via UI auth.
* | | | | | Merge pull request #2727 from matrix-org/rav/refactor_ui_auth_returnRichard van der Hoff2017-12-057-48/+103
|\ \ \ \ \ \ | | | | | | | | | | | | | | Refactor UI auth implementation
| * | | | | | Refactor UI auth implementationRichard van der Hoff2017-12-057-48/+103
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | Instead of returning False when auth is incomplete, throw an exception which can be caught with a wrapper.
* | | | | | Merge pull request #2716 from matrix-org/rav/federation_client_postRichard van der Hoff2017-12-041-8/+34
|\ \ \ \ \ \ | |/ / / / / |/| | | | | federation_client script: Support for posting content
| * | | | | federation_client script: Support for posting contentRichard van der Hoff2017-11-281-8/+34
| | | | | |
* | | | | | Merge pull request #2721 from matrix-org/rav/get_user_by_access_token_commentsRichard van der Hoff2017-11-291-1/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Improve comments on get_user_by_access_token
| * | | | | | Improve comments on get_user_by_access_tokenRichard van der Hoff2017-11-291-1/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | because I have to reverse-engineer this every time.