summary refs log tree commit diff
path: root/synapse (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge remote-tracking branch 'origin/develop' into ↵David Baker2016-08-169-211/+244
|\ | | | | | | dbkr/contains_display_name_override
| * Flake8Erik Johnston2016-08-161-4/+1
| |
| * Fix up preview URL API. Add tests.Erik Johnston2016-08-161-157/+196
| | | | | | | | | | | | | | | | | | This includes: - Splitting out methods of a class into stand alone functions, to make them easier to test. - Adding unit tests to split out functions, testing HTML -> preview. - Handle the fact that elements in lxml may have tail text.
| * Merge pull request #1012 from matrix-org/erikj/limit_backfill_uriErik Johnston2016-08-161-4/+5
| |\ | | | | | | Limit number of extremeties in backfill request
| | * Limit number of extremeties in backfill requestErik Johnston2016-08-161-4/+5
| | | | | | | | | | | | | | | This works around a bug where if we make a backfill request with too many extremeties it causes the request URI to be too long.
| * | Merge pull request #1010 from matrix-org/erikj/refactor_deletionsErik Johnston2016-08-166-47/+42
| |\ \ | | |/ | |/| Refactor user_delete_access_tokens. Invalidate get_user_by_access_token to slaves.
| | * Use cached get_user_by_access_token in slavesErik Johnston2016-08-163-2/+3
| | |
| | * Refactor user_delete_access_tokens. Invalidate get_user_by_access_token to ↵Erik Johnston2016-08-153-45/+39
| | | | | | | | | | | | slaves.
| * | Merge branch 'fix_integrity_retry' of https://github.com/Ralith/synapse into ↵Erik Johnston2016-08-161-1/+2
| |\ \ | | |/ | |/| | | | Ralith-fix_integrity_retry
| | * Add missing database corruption recovery caseBenjamin Saunders2016-08-141-1/+2
| | | | | | | | | | | | Signed-off-by: Benjamin Saunders <ben.e.saunders@gmail.com>
* | | s/underride/override/ in the rule_id tooDavid Baker2016-08-161-1/+1
| | |
* | | Move display name ruleDavid Baker2016-08-151-16/+20
|/ / | | | | | | As per https://github.com/matrix-org/matrix-doc/pull/373 and comment
* | Merge pull request #1009 from matrix-org/erikj/event_splitErik Johnston2016-08-1524-92/+367
|\ \ | | | | | | Split out /events to synchrotron
| * | Doc get_next() context manager usageErik Johnston2016-08-151-0/+4
| | |
| * | Rename table. Add docs.Erik Johnston2016-08-154-6/+16
| | |
| * | Remove broken cache stuffErik Johnston2016-08-152-21/+0
| | |
| * | Use cached version of get_aliases_for_roomErik Johnston2016-08-151-1/+1
| | |
| * | Implement cache replication streamErik Johnston2016-08-155-30/+92
| | |
| * | Add some invalidations to a cache_streamErik Johnston2016-08-157-31/+117
| | |
| * | Always run txn.after_callbacksErik Johnston2016-08-151-7/+8
| | |
| * | Make synchrotron accept /eventsErik Johnston2016-08-125-18/+66
| | |
| * | Dont invoke get_handlers fromClientV1RestServletErik Johnston2016-08-129-1/+86
| | | | | | | | | | | | | | | | | | hs.get_handlers() can not be invoked from split out processes. Moving the invocations down a level means that we can slowly split out individual servlets.
* | | Log the value which is observed in the first place.Daniel Ehlers2016-08-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | The name 'result' is of bool type and has no len property, resulting in a TypeError. Futhermore in the flow control conn.response is observed and hence should be reported. Signed-off-by: Daniel Ehlers <sargon@toppoint.de>
* | | Fix AttributeError when bind_dn is not defined.Daniel Ehlers2016-08-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | In case one does not define bind_dn in ldap configuration, filter attribute is not declared. Since auth code only uses ldap_filter attribute when according LDAP mode is selected, it is safe to only declare the attribute in that case. Signed-off-by: Daniel Ehlers <sargon@toppoint.de>
* | | Only process one local membership event per room at a timeErik Johnston2016-08-121-1/+1
|/ /
* | Merge pull request #997 from Half-Shot/developErik Johnston2016-08-113-5/+8
|\ \ | | | | | | Don't change status_msg on /sync
| * | Synced up synchrotron set_state with PresenceHandler set_stateWill Hunt2016-08-111-1/+1
| | |
| * | Don't change status_msg on /syncWill Hunt2016-08-102-4/+7
| | |
* | | Include prev_content in redacted state eventsErik Johnston2016-08-111-0/+2
| | |
* | | PEP8Erik Johnston2016-08-101-1/+3
| | |
* | | Clean up TransactionQueueErik Johnston2016-08-104-221/+162
| | |
* | | Merge pull request #999 from matrix-org/erikj/measure_moreErik Johnston2016-08-102-5/+21
|\ \ \ | | | | | | | | Measure federation send transaction resources
| * | | Measure federation send transaction resourcesErik Johnston2016-08-102-5/+21
| |/ /
* | | Don't stop on 4xx series errorsErik Johnston2016-08-101-11/+1
| | |
* | | Also check if server is in the roomErik Johnston2016-08-101-2/+7
| | |
* | | Also pull out rejected eventsErik Johnston2016-08-101-1/+1
| | |
* | | Correctly auth /event/ requestsErik Johnston2016-08-101-5/+6
| | |
* | | Only resign our own eventsErik Johnston2016-08-101-9/+10
| | |
* | | Store if we fail to fetch an event from a destinationErik Johnston2016-08-101-1/+36
|/ /
* | Merge pull request #996 from matrix-org/erikj/tls_errorErik Johnston2016-08-102-13/+19
|\ \ | | | | | | Don't print stack traces when failing to get remote keys
| * | Don't print stack traces when failing to get remote keysErik Johnston2016-08-102-13/+19
| | |
* | | /login: Respond with a 403 when we get an invalid m.login.tokenRichard van der Hoff2016-08-091-3/+3
|/ /
* | Merge pull request #995 from matrix-org/rav/clean_up_cas_loginDavid Baker2016-08-091-125/+33
|\ \ | | | | | | Clean up CAS login code
| * | Clean up CAS login codeRichard van der Hoff2016-08-081-125/+33
| |/ | | | | | | | | | | | | Remove some apparently unused code. Clean up parse_cas_response, mostly to catch the exception if the CAS response isn't valid XML.
* | Merge pull request #994 from matrix-org/rav/fix_cas_loginDavid Baker2016-08-081-0/+1
|\ \ | | | | | | Fix CAS login
| * | Fix CAS loginRichard van der Hoff2016-08-081-0/+1
| |/ | | | | | | Attempting to log in with CAS was giving a 500 error.
* | Merge pull request #993 from matrix-org/rav/fix_token_loginRichard van der Hoff2016-08-083-28/+38
|\ \ | |/ |/| Fix token login
| * Fix login with m.login.tokenRichard van der Hoff2016-08-083-28/+38
| | | | | | | | | | login with token (as used by CAS auth) was broken by 067596d, such that it always returned a 401.
* | Bump changelog and versionErik Johnston2016-08-081-1/+1
| |
* | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0812-68/+123
|\|
| * Make psutil optionalErik Johnston2016-08-083-6/+16
| |
| * Merge pull request #991 from matrix-org/erikj/retry_makeErik Johnston2016-08-051-11/+24
| |\ | | | | | | Retry joining via other servers if first one failed. Fix some other bugs.
| | * Fix a couple of python bugsErik Johnston2016-08-051-6/+8
| | |
| | * Retry joining via other servers if first one failedErik Johnston2016-08-051-5/+16
| | |
| * | Merge pull request #990 from matrix-org/erikj/fed_versErik Johnston2016-08-056-49/+80
| |\ \ | | |/ | |/| Add federation /version API
| | * Add federation /version APIErik Johnston2016-08-056-9/+25
| | |
| | * Change the way we specify if we require auth or notErik Johnston2016-08-051-40/+55
| | |
| * | Merge pull request #989 from matrix-org/erikj/raise_404Erik Johnston2016-08-051-1/+1
| |\ \ | | | | | | | | Raise 404 when couldn't find event
| | * | Raise 404 when couldn't find eventErik Johnston2016-08-051-1/+1
| | |/
| * / Don't include html comments in descriptionErik Johnston2016-08-051-1/+2
| |/
* | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-051-21/+50
|\|
| * Fix backfill auth eventsErik Johnston2016-08-051-21/+50
| |
* | Bump version and changelogErik Johnston2016-08-051-1/+1
| |
* | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0514-145/+573
|\|
| * Fix typoErik Johnston2016-08-051-1/+1
| |
| * Check if we already have the events returned by /state/Erik Johnston2016-08-051-2/+16
| |
| * Delete more tablesErik Johnston2016-08-051-0/+5
| |
| * Fallback to /state/ on both 400 and 404Erik Johnston2016-08-051-1/+1
| |
| * Tweak integrity error recovery to work as intendedBenjamin Saunders2016-08-042-1/+4
| |
| * Merge pull request #973 from matrix-org/erikj/xpath_fixErik Johnston2016-08-041-14/+72
| |\ | | | | | | Change the way we summarize URLs
| | * TypoErik Johnston2016-08-041-1/+1
| | |
| | * Test summarizationErik Johnston2016-08-041-52/+54
| | |
| | * Merge branch 'develop' of github.com:matrix-org/synapse into erikj/xpath_fixErik Johnston2016-08-0413-125/+411
| | |\
| | * | Don't infer paragrahs from newlinesErik Johnston2016-08-021-2/+1
| | | |
| | * | Comment on why we cloneErik Johnston2016-08-021-0/+1
| | | |
| | * | Spelling.Erik Johnston2016-08-021-3/+3
| | | |
| | * | Make it actually compileErik Johnston2016-08-021-1/+1
| | | |
| | * | Change the way we summarize URLsErik Johnston2016-08-021-11/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using XPath is slow on some machines (for unknown reasons), so use a different approach to get a list of text nodes. Try to generate a summary that respect paragraph and then word boundaries, adding ellipses when appropriate.
| * | | Merge pull request #983 from matrix-org/erikj/retry_on_integrity_errorErik Johnston2016-08-041-4/+66
| |\ \ \ | | |_|/ | |/| | Retry event persistence on IntegrityError
| | * | Fix typoErik Johnston2016-08-041-3/+4
| | | |
| | * | Retry event persistence on IntegrityErrorErik Johnston2016-08-041-4/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to a bug in the porting script some backfilled events were not correctly persisted, causing irrecoverable IntegrityErrors on future attempts to persist those events. This commit adds a retry mechanism invoked upon IntegrityError, where when retried the tables are purged for all references to the events being persisted.
| * | | Fix typos.Erik Johnston2016-08-041-3/+3
| | | |
| * | | Merge branch 'develop' of github.com:matrix-org/synapse into erikj/state_ids_apiErik Johnston2016-08-0411-124/+260
| |\| |
| | * | keys/query: Omit device displayname if nullRichard van der Hoff2016-08-041-3/+4
| | | | | | | | | | | | | | | | ... which makes it more consistent with user displaynames.
| | * | PEP8Richard van der Hoff2016-08-031-10/+10
| | | |
| | * | Include device name in /keys/query responseRichard van der Hoff2016-08-032-20/+51
| | | | | | | | | | | | | | | | Add an 'unsigned' section which includes the device display name.
| | * | keys/query: return all users which were asked forRichard van der Hoff2016-08-031-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | In the situation where all of a user's devices get deleted, we want to indicate this to a client, so we want to return an empty dictionary, rather than nothing at all.
| | * | Merge branch 'rav/null_default_device_displayname' into developRichard van der Hoff2016-08-034-5/+23
| | |\ \
| | | * | PEP8Richard van der Hoff2016-08-031-1/+1
| | | | |
| | | * | Default device_display_name to nullRichard van der Hoff2016-08-034-5/+23
| | | |/ | | | | | | | | | | | | | | | | | | | | It turns out that it's more useful to return a null device display name (and let clients decide how to handle it: eg, falling back to device_id) than using a constant string like "unknown device".
| | * | Merge branch 'rav/refactor_device_query' into developRichard van der Hoff2016-08-036-82/+167
| | |\ \
| | | * | E2eKeysHandler: minor tweaksRichard van der Hoff2016-08-031-5/+3
| | | | | | | | | | | | | | | | | | | | PR feedback
| | | * | Merge branch 'develop' into rav/refactor_device_queryMark Haines2016-08-0315-64/+427
| | | |\|
| | | * | E2E keys: Make federation query share code with client queryRichard van der Hoff2016-08-023-47/+92
| | | | | | | | | | | | | | | | | | | | | | | | | Refactor the e2e query handler to separate out the local query, and then make the federation handler use it.
| | | * | Move e2e query logic into a handlerRichard van der Hoff2016-08-014-60/+102
| | | | |
| | * | | Remove other bit of deduplicationErik Johnston2016-08-031-19/+0
| | | | |
| | * | | Ensure we only persist an event once at a timeErik Johnston2016-08-031-1/+18
| | | |/ | | |/|
| * | | Tidy up get_eventsErik Johnston2016-08-041-12/+10
| | | |
| * | | Rename fields to _idsErik Johnston2016-08-032-4/+4
| | | |
| * | | Fix copy + paste failsErik Johnston2016-08-032-6/+11
| | | |
| * | | Actually call get_room_stateErik Johnston2016-08-031-0/+4
| | | |
| * | | Fix syntax errorErik Johnston2016-08-031-1/+8
| | | |
| * | | CommentErik Johnston2016-08-031-0/+14
| | | |
| * | | Add /state_ids federation APIErik Johnston2016-08-034-3/+125
| |/ / | | | | | | | | | | | | The new API only returns the event_ids for the state, as most requesters will already have the vast majority of the events already.
* | | Bump version and changelog v0.17.0-rc3Erik Johnston2016-08-021-1/+1
| | |
* | | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0218-110/+643
|\| |
| * | Don't double wrap 200Erik Johnston2016-08-021-2/+2
| | |
| * | Fix response cacheErik Johnston2016-08-021-1/+1
| | |
| * | Merge pull request #940 from matrix-org/erikj/fed_state_cacheErik Johnston2016-08-025-32/+60
| |\ \ | | | | | | | | Cache federation state responses
| | * | Cache federation state responsesErik Johnston2016-07-215-32/+60
| | | |
| * | | Merge pull request #967 from matrix-org/erikj/fed_readerErik Johnston2016-08-027-18/+342
| |\ \ \ | | | | | | | | | | Split out the federation reading portions into a separate.
| | * \ \ Merge branch 'develop' of github.com:matrix-org/synapse into erikj/fed_readerErik Johnston2016-08-023-3/+23
| | |\ \ \
| | * \ \ \ Merge branch 'develop' of github.com:matrix-org/synapse into erikj/fed_readerErik Johnston2016-07-291-4/+31
| | |\ \ \ \
| | * \ \ \ \ Merge branch 'develop' of github.com:matrix-org/synapse into erikj/fed_readerErik Johnston2016-07-2937-295/+929
| | |\ \ \ \ \
| | * | | | | | Add destination retry to slave storeErik Johnston2016-07-282-0/+32
| | | | | | | |
| | * | | | | | Add slace storage functions for public room listErik Johnston2016-07-283-0/+48
| | | | | | | |
| | * | | | | | Add get_auth_chain to slave storeErik Johnston2016-07-281-0/+4
| | | | | | | |
| | * | | | | | Add more key storage funcs into slave storeErik Johnston2016-07-282-24/+26
| | | | | | | |
| | * | | | | | Split out a FederationReader processErik Johnston2016-07-214-0/+238
| | | | | | | |
| * | | | | | | Merge branch 'develop' into dbkr/fix_add_email_on_registerMark Haines2016-08-023-3/+23
| |\ \ \ \ \ \ \ | | | |_|_|/ / / | | |/| | | | |
| | * | | | | | Ignore AlreadyCalled errors on timer cancelErik Johnston2016-07-312-3/+16
| | | |_|_|_|/ | | |/| | | |
| | * | | | | Merge pull request #958 from matrix-org/paul/SYN-738Paul Evans2016-07-291-0/+7
| | |\ \ \ \ \ | | | |_|_|/ / | | |/| | | | Forbid non-ASes from registering users whose names begin with '_'
| | | * | | | Forbid non-ASes from registering users whose names begin with '_' (SYN-738)Paul "LeoNerd" Evans2016-07-271-0/+7
| | | | | | |
| * | | | | | Fix adding emails on registrationDavid Baker2016-07-291-9/+7
| |/ / / / / | | | | | | | | | | | | | | | | | | Synapse was not adding email addresses to accounts registered with an email address, due to too many different variables called 'result'. Rename both of them. Also remove the defer.returnValue() with no params because that's not a thing.
| * | | | | Merge pull request #964 from matrix-org/erikj/fed_join_fixErik Johnston2016-07-291-4/+31
| |\ \ \ \ \ | | |_|_|/ / | |/| | | | Handle the case of missing auth events when joining a room
| | * | | | Mention that func will fetch auth eventsErik Johnston2016-07-291-0/+2
| | | | | |
| | * | | | Update docstringErik Johnston2016-07-291-0/+6
| | | | | |
| | * | | | Handle the case of missing auth events when joining a roomErik Johnston2016-07-281-4/+23
| | | | | |
| * | | | | Fix SQL to supply arguments in the same orderMark Haines2016-07-291-1/+1
| | | | | |
| * | | | | Create separate methods for getting messages to pushMark Haines2016-07-284-51/+163
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | for the email and http pushers rather than trying to make a single method that will work with their conflicting requirements. The http pusher needs to get the messages in ascending stream order, and doesn't want to miss a message. The email pusher needs to get the messages in descending timestamp order, and doesn't mind if it misses messages.
| * | | | | Comment get_unread_push_actions_for_user_in_range functionKegan Dougal2016-07-281-2/+26
| | | | | |
| * | | | | Merge branch 'release-v0.17.0' of github.com:matrix-org/synapse into developErik Johnston2016-07-281-1/+1
| |\| | | |
| * | | | | Merge pull request #962 from matrix-org/markjh/retryMark Haines2016-07-281-1/+1
| |\ \ \ \ \ | | | | | | | | | | | | | | Fix retry utils to check if the exception is a subclass of CME
| | * | | | | Fix retry utils to check if the exception is a subclass of CMEMark Haines2016-07-281-1/+1
| | | | | | |
| * | | | | | Merge pull request #941 from matrix-org/erikj/key_client_fixErik Johnston2016-07-281-3/+9
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | Send the correct host header when fetching keys
* | | | | | | Bump changeog and versionErik Johnston2016-08-021-1/+1
| |_|/ / / / |/| | | | |
* | | | | | Merge branch 'erikj/key_client_fix' of github.com:matrix-org/synapse into ↵Erik Johnston2016-07-281-3/+9
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | release-v0.17.0
| * | | | | Cache getPeerErik Johnston2016-07-211-2/+5
| | | | | |
| * | | | | Set host not pathErik Johnston2016-07-211-1/+1
| | | | | |
| * | | | | Send the correct host header when fetching keysErik Johnston2016-07-211-3/+6
| | |_|_|/ | |/| | |
* | | | | Bump version and changelogErik Johnston2016-07-281-1/+1
| | | | |
* | | | | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-07-287-14/+85
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Merge pull request #961 from matrix-org/dbkr/fix_push_invite_nameErik Johnston2016-07-281-1/+4
| |\ \ \ \ | | | | | | | | | | | | Don't include name of room for invites in push
| | * | | | Don't include name of room for invites in pushDavid Baker2016-07-281-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | Avoids insane pushes like, "Bob invited you to invite from Bob"
| * | | | | Merge pull request #960 from matrix-org/rav/support_r0.2Richard van der Hoff2016-07-281-1/+5
| |\ \ \ \ \ | | |/ / / / | |/| | | | Add r0.2.0 to the "supported versions" list
| | * | | | Add r0.1.0 to the "supported versions" listRichard van der Hoff2016-07-281-0/+1
| | | | | |
| | * | | | Add r0.2.0 to the "supported versions" listRichard van der Hoff2016-07-281-1/+4
| | | | | |
| * | | | | TypoDavid Baker2016-07-281-1/+1
| |/ / / /
| * | | | Merge pull request #956 from matrix-org/rav/check_device_id_on_key_uploadRichard van der Hoff2016-07-274-11/+75
| |\ \ \ \ | | |_|/ / | |/| | | Make the device id on e2e key upload optional
| | * | | key upload tweaksRichard van der Hoff2016-07-271-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Add v2_alpha URL back in, since things seem to be using it. 2. Don't reject the request if the device_id in the upload request fails to match that in the access_token.
| | * | | SQL syntax fixRichard van der Hoff2016-07-271-1/+1
| | | | |
| | * | | Delete e2e keys on device deleteRichard van der Hoff2016-07-273-4/+28
| | | | |
| | * | | Make the device id on e2e key upload optionalRichard van der Hoff2016-07-262-12/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should now be able to get our device_id from the access_token, so the device_id on the upload request is optional. Where it is supplied, we should check that it matches. For active access_tokens without an associated device_id, we ought to register the device in the devices table. Also update the table on upgrade so that all of the existing e2e keys are associated with real devices.
* | | | | Bump CHANGES and versionErik Johnston2016-07-271-1/+1
|/ / / /
* | | | Merge branch 'develop' into markjh/verifyMark Haines2016-07-272-3/+11
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: synapse/crypto/keyring.py
| * \ \ \ Merge pull request #955 from matrix-org/markjh/only_from2Mark Haines2016-07-271-2/+9
| |\ \ \ \ | | | | | | | | | | | | Add a couple more checks to the keyring
| | * | | | Add a couple more checks to the keyringMark Haines2016-07-261-2/+9
| | | | | |
| * | | | | Merge pull request #954 from matrix-org/markjh/even_more_fixesMark Haines2016-07-272-9/+11
| |\ \ \ \ \ | | |/ / / / | |/| | | | Fix a couple of bugs in the transaction and keyring code
| | * | | | Fix a couple of bugs in the transaction and keyring codeMark Haines2016-07-262-9/+11
| | |/ / /
* | / / / Clean up verify_json_objects_for_serverMark Haines2016-07-271-68/+75
|/ / / /
* | | | Merge pull request #952 from matrix-org/markjh/more_fixesMark Haines2016-07-261-0/+4
|\ \ \ \ | |/ / / |/| | | Check if the user is banned when handling 3pid invites
| * | | Check if the user is banned when handling 3pid invitesMark Haines2016-07-261-0/+4
| | | |
* | | | Add `create_requester` functionRichard van der Hoff2016-07-267-49/+79
|/ / / | | | | | | | | | | | | Wrap the `Requester` constructor with a function which provides sensible defaults, and use it throughout
* | | Fix typoRichard van der Hoff2016-07-261-1/+1
| | |
* | | Merge branch 'develop' into rav/delete_refreshtoken_on_delete_deviceRichard van der Hoff2016-07-262-55/+113
|\ \ \
| * \ \ Merge pull request #951 from matrix-org/rav/flake8Richard van der Hoff2016-07-261-5/+3
| |\ \ \ | | | | | | | | | | Fix flake8 noise
| | * | | Fix flake8 violationRichard van der Hoff2016-07-261-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | Apparently flake8 v3 puts the error on a different line to v2. Easiest way to make sure that happens is by putting the whole statement on one line :)
| * | | | Merge pull request #948 from matrix-org/markjh/auth_fixesMark Haines2016-07-261-50/+110
| |\ \ \ \ | | |/ / / | |/| | | Don't add rejections to the state_group, persist all rejections
| | * | | Fix how push_actions are redacted.Mark Haines2016-07-261-8/+10
| | | | |
| | * | | Don't add rejected events if we've seen them befrore. Add some comments to ↵Mark Haines2016-07-261-5/+48
| | | | | | | | | | | | | | | | | | | | explain what the code is doing mechanically
| | * | | Only update the events and event_json tables for rejected eventsMark Haines2016-07-252-52/+63
| | | | |
| | * | | Don't add the events to forward extremities if the event is rejectedMark Haines2016-07-251-3/+6
| | | | |
| | * | | Don't add rejections to the state_group, persist all rejectionsMark Haines2016-07-252-5/+6
| | | | |
* | | | | Delete refresh tokens when deleting devicesRichard van der Hoff2016-07-262-15/+49
|/ / / /
* | | | Merge pull request #949 from matrix-org/rav/update_devicesDavid Baker2016-07-2610-22/+213
|\ \ \ \ | | | | | | | | | | Implement updates and deletes for devices
| * | | | Implement updating devicesRichard van der Hoff2016-07-263-8/+67
| | | | | | | | | | | | | | | | | | | | You can update the displayname of devices now.
| * | | | Implement deleting devicesRichard van der Hoff2016-07-269-15/+146
| | | | |
| * | | | Log when adding listenersRichard van der Hoff2016-07-251-0/+1
| | | | |
* | | | | typoMatthew Hodgson2016-07-261-1/+1
|/ / / /
* | | | Merge pull request #946 from matrix-org/dbkr/log_recaptcha_hostnameDavid Baker2016-07-251-2/+11
|\ \ \ \ | |/ / / |/| | | Log the hostname the reCAPTCHA was completed on
| * | | Use get to avoid KeyErrorsDavid Baker2016-07-221-1/+1
| | | |
| * | | Log the hostname the reCAPTCHA was completed onDavid Baker2016-07-221-2/+11
| | | | | | | | | | | | | | | | This could be useful information to have in the logs. Also comment about how & why we don't verify the hostname.
* | | | fix: defer.returnValue takes one argumentMark Haines2016-07-251-1/+1
| | | |
* | | | background updates: fix assert againRichard van der Hoff2016-07-251-1/+1
| | | |
* | | | background updates: Fix assertion to do somethingRichard van der Hoff2016-07-251-2/+2
| | | |
* | | | Fix background_update testsRichard van der Hoff2016-07-251-7/+20
| | | | | | | | | | | | | | | | | | | | | | | | A bit of a cleanup for background_updates, and make sure that the real background updates have run before we start the unit tests, so that they don't interfere with the tests.
* | | | PEP8Richard van der Hoff2016-07-221-1/+0
| | | |
* | | | Create index on user_ips in the backgroundRichard van der Hoff2016-07-223-12/+80
|/ / / | | | | | | | | | | | | | | | | | | | | | user_ips is kinda big, so really we want to add the index in the background once we're running. Replace the schema delta with one which will do that. I've done this in a way that's reasonably easy to reuse as there a few other indexes I need, and I don't suppose they will be the last.
* | / make /devices return a listRichard van der Hoff2016-07-211-5/+5
| |/ |/| | | | | Turns out I specced this to return a list of devices rather than a dict of them
* | Merge pull request #943 from matrix-org/rav/get_device_apiDavid Baker2016-07-212-9/+62
|\ \ | | | | | | Implement GET /device/{deviceId}
| * | Fix PEP8 errorsRichard van der Hoff2016-07-211-1/+1
| | |
| * | Implement GET /device/{deviceId}Richard van der Hoff2016-07-212-9/+62
| | |
* | | Merge pull request #942 from matrix-org/rav/fix_register_deviceidDavid Baker2016-07-211-11/+10
|\ \ \ | |/ / |/| | Preserve device_id from first call to /register
| * | Preserve device_id from first call to /registerRichard van der Hoff2016-07-211-11/+10
| |/ | | | | | | | | device_id may only be passed in the first call to /register, so make sure we fish it out of the register `params` rather than the body of the final call.
* | storage/client_ips: remove some dead codeRichard van der Hoff2016-07-211-3/+0
| |
* | Merge branch 'develop' into rav/get_devices_apiRichard van der Hoff2016-07-204-22/+67
|\| | | | | | | (pick up PR #938 in the hope of fixing the UTs)
| * Merge pull request #938 from matrix-org/rav/add_device_id_to_client_ipsRichard van der Hoff2016-07-202-7/+25
| |\ | | | | | | Record device_id in client_ips
| | * Record device_id in client_ipsRichard van der Hoff2016-07-202-7/+25
| | | | | | | | | | | | | | | Record the device_id when we add a client ip; it's somewhat redundant as we could get it via the access_token, but it will make querying rather easier.
| * | Don't explode if we have no snapshots yetErik Johnston2016-07-201-0/+3
| | |
| * | Merge pull request #937 from matrix-org/rav/register_device_on_registerDavid Baker2016-07-201-15/+39
| |\ \ | | |/ | |/| Register a device_id in the /v2/register flow.
| | * Register a device_id in the /v2/register flow.Richard van der Hoff2016-07-201-15/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This doesn't cover *all* of the registration flows, but it does cover the most common ones: in particular: shared_secret registration, appservice registration, and normal user/pass registration. Pull device_id from the registration parameters. Register the device in the devices table. Associate the device with the returned access and refresh tokens. Profit.
* | | GET /devices endpointRichard van der Hoff2016-07-207-5/+198
|/ / | | | | | | | | | | | | implement a GET /devices endpoint which lists all of the user's devices. It also returns the last IP where we saw that device, so there is some dancing to fish that out of the user_ips table.
* / More doc-commentsRichard van der Hoff2016-07-202-10/+14
|/ | | | Fix some more comments on some things
* Merge pull request #936 from matrix-org/erikj/log_rssErik Johnston2016-07-204-1/+50
|\ | | | | Add metrics for psutil derived memory usage
| * Add metrics for psutil derived memory usageErik Johnston2016-07-204-1/+50
| |
* | Merge pull request #935 from matrix-org/erikj/backfill_notifsErik Johnston2016-07-201-5/+6
|\ \ | | | | | | Don't notify pusher pool for backfilled events
| * | Don't notify pusher pool for backfilled eventsErik Johnston2016-07-201-5/+6
| |/
* | Merge pull request #933 from matrix-org/rav/type_annotationsRichard van der Hoff2016-07-206-0/+46
|\ \ | | | | | | Type annotations
| * | Type annotationsRichard van der Hoff2016-07-196-0/+46
| | | | | | | | | | | | | | | Add some type annotations to help PyCharm (in particular) to figure out the types of a bunch of things.
* | | Merge pull request #932 from matrix-org/rav/register_refactorDavid Baker2016-07-204-21/+52
|\ \ \ | |_|/ |/| | Further registration refactoring
| * | Further registration refactoringRichard van der Hoff2016-07-194-21/+52
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | * `RegistrationHandler.appservice_register` no longer issues an access token: instead it is left for the caller to do it. (There are two of these, one in `synapse/rest/client/v1/register.py`, which now simply calls `AuthHandler.issue_access_token`, and the other in `synapse/rest/client/v2_alpha/register.py`, which is covered below). * In `synapse/rest/client/v2_alpha/register.py`, move the generation of access_tokens into `_create_registration_details`. This means that the normal flow no longer needs to call `AuthHandler.issue_access_token`; the shared-secret flow can tell `RegistrationHandler.register` not to generate a token; and the appservice flow continues to work despite the above change.
* | Merge pull request #922 from matrix-org/erikj/file_api2Erik Johnston2016-07-206-9/+239
|\ \ | |/ |/| Feature: Add filter to /messages. Add 'contains_url' to filter.
| * CommentErik Johnston2016-07-201-0/+4
| |
| * Add filter param to /messages APIErik Johnston2016-07-142-5/+22
| |
| * Add support for filters in paginate_room_eventsErik Johnston2016-07-141-1/+51
| |
| * Add 'contains_url' to filterErik Johnston2016-07-141-3/+20
| |
| * Add sender and contains_url field to events tableErik Johnston2016-07-142-0/+142
| |
* | Merge pull request #931 from matrix-org/rav/refactor_registerDavid Baker2016-07-191-78/+102
|\ \ | | | | | | rest/client/v2_alpha/register.py: Refactor flow somewhat.
| * | Don't bind email unless threepid contains expected fieldsRichard van der Hoff2016-07-191-28/+25
| | |
| * | rest/client/v2_alpha/register.py: Refactor flow somewhat.Richard van der Hoff2016-07-191-75/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is meant to be an *almost* non-functional change, with the exception that it fixes what looks a lot like a bug in that it only calls `auth_handler.add_threepid` and `add_pusher` once instead of three times. The idea is to move the generation of the `access_token` out of `registration_handler.register`, because `access_token`s now require a device_id, and we only want to generate a device_id once registration has been successful.
* | | Merge pull request #929 from matrix-org/rav/support_deviceid_in_loginRichard van der Hoff2016-07-1910-24/+265
|\ \ \ | |/ / |/| | Add device_id support to /login
| * | schema fixRichard van der Hoff2016-07-191-1/+1
| | | | | | | | | | | | device_id should be text, not bigint.
| * | Add device_id support to /loginRichard van der Hoff2016-07-1810-24/+265
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a 'devices' table to the storage, as well as a 'device_id' column to refresh_tokens. Allow the client to pass a device_id, and initial_device_display_name, to /login. If login is successful, then register the device in the devices table if it wasn't known already. If no device_id was supplied, make one up. Associate the device_id with the access token and refresh token, so that we can get at it again later. Ensure that the device_id is copied from the refresh token to the access_token when the token is refreshed.
* | | Summary lineMark Haines2016-07-191-2/+1
| | |
* | | Update docstring on Handlers.Mark Haines2016-07-191-3/+15
|/ / | | | | | | To indicate it is deprecated.
* | Merge pull request #928 from matrix-org/rav/refactor_loginRichard van der Hoff2016-07-182-65/+82
|\ \ | | | | | | Refactor login flow
| * | Refactor login flowRichard van der Hoff2016-07-182-65/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure that we have the canonical user_id *before* calling get_login_tuple_for_user_id. Replace login_with_password with a method which just validates the password, and have the caller call get_login_tuple_for_user_id. This brings the password flow into line with the other flows, and will give us a place to register the device_id if necessary.
* | | Merge pull request #924 from matrix-org/erikj/purge_historyErik Johnston2016-07-181-19/+15
|\ \ \ | | | | | | | | Fix /purge_history bug
| * | | Fix /purge_history bugErik Johnston2016-07-151-19/+15
| | | | | | | | | | | | | | | | This was caused by trying to insert duplicate backward extremeties
* | | | Merge pull request #925 from matrix-org/markjh/auth_fixMark Haines2016-07-181-0/+5
|\ \ \ \ | | | | | | | | | | Fix 500 ISE when sending alias event without a state_key
| * | | | Fix 500 ISE when sending alias event without a state_keyMark Haines2016-07-151-0/+5
| | | | |
* | | | | Merge pull request #927 from Half-Shot/developErik Johnston2016-07-181-5/+6
|\ \ \ \ \ | |_|_|/ / |/| | | | Fall back to 'username' if 'user' is not given for appservice registration.
| * | | | Use body.get to check for 'user'Will Hunt2016-07-161-2/+1
| | | | |
| * | | | Fall back to 'username' if 'user' is not given for appservice reg.Will Hunt2016-07-161-3/+5
| |/ / /
* / / / Fix a doc-commentRichard van der Hoff2016-07-181-1/+1
|/ / / | | | | | | | | | The `store` in a handler is a generic DataStore, not just an events.StateStore.
* | | Merge pull request #923 from matrix-org/erikj/purge_historyErik Johnston2016-07-152-2/+13
|\| | | | | | | | Various purge_history fixes
| * | Pull out min stream_ordering from ex_outlier_streamErik Johnston2016-07-151-1/+2
| | |
| * | event_backwards_extremeties may not be emptyErik Johnston2016-07-151-1/+11
| | |
* | | Merge pull request #919 from matrix-org/erikj/auth_fixErik Johnston2016-07-153-5/+35
|\ \ \ | |/ / |/| | Various auth.py fixes.
| * | CommentErik Johnston2016-07-152-0/+6
| | |
| * | Check sender signed eventErik Johnston2016-07-143-6/+12
| | |
| * | Check if alias event's state_key matches sender's domainErik Johnston2016-07-131-0/+11
| | |
| * | Check creation event's room_id domain matches sender'sErik Johnston2016-07-131-0/+7
| | |
* | | Merge pull request #921 from matrix-org/erikj/account_deactivateErik Johnston2016-07-141-0/+44
|\ \ \ | | | | | | | | Feature: Add an /account/deactivate endpoint
| * | | Add hs objectErik Johnston2016-07-141-0/+1
| | | |
| * | | Only accept password authErik Johnston2016-07-141-12/+0
| | | |
| * | | Add an /account/deactivate endpointErik Johnston2016-07-141-0/+55
| | |/ | |/|
* | | Merge pull request #918 from negzi/bugfix_for_token_expiryErik Johnston2016-07-144-6/+10
|\ \ \ | |/ / |/| | Bug fix: expire invalid access tokens
| * | Bug fix: expire invalid access tokensNegar Fazeli2016-07-134-6/+10
| |/
* | Merge pull request #915 from matrix-org/dbkr/more_requesttokensDavid Baker2016-07-133-30/+103
|\ \ | |/ |/| Add requestToken endpoints