summary refs log tree commit diff
Commit message (Collapse)AuthorAgeFilesLines
* Measure federation send transaction resourcesErik Johnston2016-08-102-5/+21
|
* 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-084-31/+87
|\ \ | |/ |/| Fix token login
| * PEP8Richard van der Hoff2016-08-081-0/+1
| |
| * Fix login with m.login.tokenRichard van der Hoff2016-08-084-32/+87
| | | | | | | | | | login with token (as used by CAS auth) was broken by 067596d, such that it always returned a 401.
* | Merge branch 'release-v0.17.0' of github.com:matrix-org/synapse v0.17.0Erik Johnston2016-08-08136-1642/+5931
|\ \
| * | Capatailize HTMLErik Johnston2016-08-081-1/+1
| | |
| * | Be bolderErik Johnston2016-08-081-1/+1
| | |
| * | Update changelogErik Johnston2016-08-081-3/+7
| | |
| * | Update changelogErik Johnston2016-08-081-1/+1
| | |
| * | Bump changelog and versionErik Johnston2016-08-082-1/+16
| | |
| * | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0813-68/+124
| |\|
| | * Merge pull request #992 from matrix-org/erikj/psutil_conditionalErik Johnston2016-08-083-6/+16
| | |\ | | | | | | | | Make psutil optional
| | | * 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
| | | |
| | * | Print newline after result in federation_client scriptErik Johnston2016-08-051-0/+1
| | | |
| | * | 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
| | | |/
| | * | Merge pull request #988 from matrix-org/erikj/ignore_comments_previewErik Johnston2016-08-051-1/+2
| | |\ \ | | | |/ | | |/| Don't include html comments in description
| | | * Don't include html comments in descriptionErik Johnston2016-08-051-1/+2
| | |/
| * | Update changelog v0.17.0-rc4Erik Johnston2016-08-051-2/+2
| | |
| * | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-051-21/+50
| |\|
| | * Merge pull request #987 from matrix-org/erikj/fix_backfill_authErik Johnston2016-08-051-21/+50
| | |\ | | | | | | | | Fix backfill auth events
| | | * Fix backfill auth eventsErik Johnston2016-08-051-21/+50
| | |/
| * | Bump version and changelogErik Johnston2016-08-052-1/+17
| | |
| * | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0525-369/+1070
| |\|
| | * Merge pull request #986 from matrix-org/erikj/stateErik Johnston2016-08-051-2/+16
| | |\ | | | | | | | | Check if we already have the events returned by /state/
| | | * Fix typoErik Johnston2016-08-051-1/+1
| | | |
| | | * Check if we already have the events returned by /state/Erik Johnston2016-08-051-2/+16
| | | |
| | * | Merge pull request #985 from matrix-org/erikj/fix_integrity_retryErik Johnston2016-08-052-1/+9
| | |\| | | | | | | | | Tweak integrity error recovery to work as intended
| | | * 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-042-14/+211
| | |\ | | | | | | | | Change the way we summarize URLs
| | | * TypoErik Johnston2016-08-041-1/+1
| | | |
| | | * Test summarizationErik Johnston2016-08-042-52/+193
| | | |
| | | * Merge branch 'develop' of github.com:matrix-org/synapse into erikj/xpath_fixErik Johnston2016-08-0423-349/+769
| | | |\
| | | * | 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.
| | * | | Merge pull request #979 from matrix-org/erikj/state_ids_apiErik Johnston2016-08-044-1/+151
| | |\ \ \ | | | |/ / | | |/| | Add /state_ids federation API
| | | * | Fix typos.Erik Johnston2016-08-041-3/+3
| | | | |
| | | * | Merge branch 'develop' of github.com:matrix-org/synapse into erikj/state_ids_apiErik Johnston2016-08-0418-305/+572
| | | |\ \ | | | |/ / | | |/| |
| | * | | Merge pull request #982 from matrix-org/erikj/fix_port_scriptErik Johnston2016-08-041-43/+132
| | |\ \ \ | | | | | | | | | | | | Port script: Handle the fact that some tables have negative rowid rows
| | | * | | Allow upgrading from old port_from_sqlite3 formatErik Johnston2016-08-041-0/+21
| | | | | |
| | | * | | Handle the fact that some tables have negative rowid rowsErik Johnston2016-08-041-43/+111
| | | | | |
| | * | | | Merge pull request #981 from matrix-org/rav/omit_device_displayname_if_nullRichard van der Hoff2016-08-041-3/+4
| | |\ \ \ \ | | | | | | | | | | | | | | keys/query: Omit device displayname if null
| | | * | | | keys/query: Omit device displayname if nullRichard van der Hoff2016-08-041-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | ... which makes it more consistent with user displaynames.
| | * | | | | Factor out more common code from the jenkins scripts (#980)Mark Haines2016-08-044-138/+44
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Factor out more common code from the jenkins scripts * Fix install_and_run path * Poke jenkins * Poke jenkins
| | * | | | Merge pull request #978 from matrix-org/rav/device_name_in_e2e_devicesRichard van der Hoff2016-08-033-20/+141
| | |\ \ \ \ | | | | | | | | | | | | | | Include device name in /keys/query response
| | | * | | | PEP8Richard van der Hoff2016-08-032-12/+10
| | | | | | |
| | | * | | | Include device name in /keys/query responseRichard van der Hoff2016-08-033-20/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an 'unsigned' section which includes the device display name.
| | * | | | | Merge pull request #977 from matrix-org/rav/return_all_devicesRichard van der Hoff2016-08-032-3/+52
| | |\| | | | | | | |/ / / | | |/| | | keys/query: return all users which were asked for
| | | * | | keys/query: return all users which were asked forRichard van der Hoff2016-08-032-3/+52
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | 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-0322-139/+507
| | | |\| | |
| | | * | | | 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
| | | | | | |
| | * | | | | Merge pull request #975 from matrix-org/erikj/multi_event_persistErik Johnston2016-08-031-20/+18
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | Ensure we only persist an event once at a time
| | | * | | | | 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.
| | * | | | Print status code in federation_client.pyErik Johnston2016-08-031-0/+1
| | | |_|/ | | |/| |
| | * | | Factor out some of the code shared between the sytest scripts (#974)Mark Haines2016-08-026-69/+71
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Factor out some of the code shared between the different sytest jenkins scripts * Exclude jenkins from the MANIFEST * Fix dendron build * Missing new line * Poke jenkins * Export the PORT_BASE and PORT_COUNT * Poke jenkins
| * | | Bump version and changelog v0.17.0-rc3Erik Johnston2016-08-022-2/+8
| | | |
| * | | Merge branch 'develop' of github.com:matrix-org/synapse into release-v0.17.0Erik Johnston2016-08-0227-122/+745
| |\| |
| | * | Merge pull request #971 from matrix-org/erikj/fed_stateErik Johnston2016-08-021-3/+3
| | |\ \ | | | | | | | | | | Fix response cache
| | | * | 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
| | | | |
| | * | | Print authorization header for federation_client.pyErik Johnston2016-08-021-5/+6
| | | | |
| | * | | Merge pull request #967 from matrix-org/erikj/fed_readerErik Johnston2016-08-0210-24/+350
| | |\ \ \ | | | | | | | | | | | | Split out the federation reading portions into a separate.
| | | * | | Change default jenkins port base and countErik Johnston2016-08-023-6/+7
| | | | | |
| | | * | | Merge branch 'develop' of github.com:matrix-org/synapse into erikj/fed_readerErik Johnston2016-08-024-4/+24
| | | |\ \ \
| | | * | | | Make jenkins dendron test federation read apisErik Johnston2016-07-291-0/+1
| | | | | | |
| | | * | | | 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-2954-345/+1250
| | | |\ \ \ \ \
| | | * | | | | | 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 pull request #968 from matrix-org/dbkr/fix_add_email_on_registerDavid Baker2016-08-021-9/+7
| | |\ \ \ \ \ \ \ | | | |_|_|_|/ / / | | |/| | | | | | Fix adding emails on registration
| | | * | | | | | Merge branch 'develop' into dbkr/fix_add_email_on_registerMark Haines2016-08-024-4/+24
| | | |\ \ \ \ \ \ | | | |/ / / / / / | | |/| | | | | |
| | * | | | | | | Merge pull request #970 from matrix-org/erikj/clockErik Johnston2016-08-022-3/+16
| | |\ \ \ \ \ \ \ | | | |_|_|_|_|_|/ | | |/| | | | | | Ignore AlreadyCalled errors on timer cancel
| | | * | | | | | Ignore AlreadyCalled errors on timer cancelErik Johnston2016-07-312-3/+16
| | | | | | | | |
| | * | | | | | | missing --upgradeMatthew Hodgson2016-08-011-1/+1
| | |/ / / / / /
| | * | | | | | 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
| | | | | | |
| | * | | | | Merge pull request #966 from matrix-org/markjh/fix_pushMark Haines2016-07-295-52/+205
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | Create separate methods for getting messages to push
| | | * | | | | Fix SQL to supply arguments in the same orderMark Haines2016-07-291-1/+1
| | | | | | | |
| | | * | | | | Create separate methods for getting messages to pushMark Haines2016-07-285-51/+204
| | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| | * | | | | Merge pull request #965 from matrix-org/kegan/comment-push-actions-fnKegsay2016-07-281-2/+26
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | Comment get_unread_push_actions_for_user_in_range function
| | | * | | | | Comment get_unread_push_actions_for_user_in_range functionKegan Dougal2016-07-281-2/+26
| | |/ / / / /
| | * | | | | Merge pull request #963 from matrix-org/erikj/admin_docsErik Johnston2016-07-283-0/+46
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | Add some basic admin API docs
| | | * | | | | Use correct pathErik Johnston2016-07-281-1/+1
| | | | | | | |
| | | * | | | | Add some basic admin API docsErik Johnston2016-07-283-0/+46
| | | |/ / / /
| | * | | | | Merge branch 'release-v0.17.0' of github.com:matrix-org/synapse into developErik Johnston2016-07-282-1/+65
| | |\| | | |
| | * | | | | 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
| * | | | | | | Typo v0.17.0-rc2Erik Johnston2016-08-021-1/+1
| | | | | | | |
| * | | | | | | Change wordingErik Johnston2016-08-021-2/+2
| | | | | | | |
| * | | | | | | Bump changeog and versionErik Johnston2016-08-022-1/+18
| | |_|/ / / / | |/| | | | |
| * | | | | | Add authors to changelog v0.17.0-rc1Erik Johnston2016-07-281-4/+5
| | | | | | |
| * | | | | | Update changelogErik Johnston2016-07-281-0/+4
| | | | | | |
| * | | | | | 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-282-3/+6
| | | | | |
| * | | | | 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-272-1/+57
| |/ / / /
| * | | | Merge pull request #957 from matrix-org/markjh/verifyMark Haines2016-07-271-74/+80
| |\ \ \ \ | | | | | | | | | | | | Clean up verify_json_objects_for_server
| | * \ \ \ 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
| | | | |
| * | | | Merge pull request #953 from matrix-org/rav/requesterRichard van der Hoff2016-07-2611-75/+101
| |\ \ \ \ | | |/ / / | |/| | | Add `create_requester` function
| | * | | Add `create_requester` functionRichard van der Hoff2016-07-2611-75/+101
| |/ / / | | | | | | | | | | | | | | | | 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-263-59/+115
| |\ \ \
| | * \ \ Merge pull request #951 from matrix-org/rav/flake8Richard van der Hoff2016-07-262-9/+5
| | |\ \ \ | | | | | | | | | | | | 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 :)
| | | * | | Fix flake8 configurationRichard van der Hoff2016-07-261-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apparently flake8 v3 doesn't like trailing comments on config settings. Also remove the pep8 config, which didn't work (because it was missing W503) and duplicated the flake8 config. We don't use pep8 on its own, so the config was duplicative.
| | * | | | 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-263-15/+83
| |/ / / /
| * | | | Merge pull request #949 from matrix-org/rav/update_devicesDavid Baker2016-07-2613-28/+295
| |\ \ \ \ | | | | | | | | | | | | Implement updates and deletes for devices
| | * | | | Implement updating devicesRichard van der Hoff2016-07-265-8/+119
| | | | | | | | | | | | | | | | | | | | | | | | You can update the displayname of devices now.
| | * | | | Implement deleting devicesRichard van der Hoff2016-07-2611-21/+176
| | | | | |
| | * | | | Log when adding listenersRichard van der Hoff2016-07-251-0/+1
| | | | | |
| * | | | | federation doesn't work over ipv6 yet thanks to twistedMatthew Hodgson2016-07-261-1/+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
| | | | |
| * | | | Merge pull request #945 from matrix-org/rav/background_reindexRichard van der Hoff2016-07-254-23/+113
| |\ \ \ \ | | | | | | | | | | | | Create index on user_ips in the background
| | * | | | PEP8Richard van der Hoff2016-07-251-1/+1
| | | | | |
| | * | | | Fix background_update testsRichard van der Hoff2016-07-252-13/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | Merge pull request #947 from matrix-org/rav/unittest_loggingRichard van der Hoff2016-07-251-4/+7
| |\ \ \ \ | | |/ / / | |/| | | Slightly saner logging for unittests
| | * | | Slightly saner logging for unittestsRichard van der Hoff2016-07-251-4/+7
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | 1. Give the handler used for logging in unit tests a formatter, so that the output is slightly more meaningful 2. Log some synapse.storage stuff, because it's useful.
| * | | Merge pull request #944 from matrix-org/rav/devices_returns_listDavid Baker2016-07-212-9/+12
| |\ \ \ | | |_|/ | |/| | make /devices return a list
| | * | make /devices return a listRichard van der Hoff2016-07-212-9/+12
| |/ / | | | | | | | | | 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-213-19/+90
| |\ \ | | | | | | | | Implement GET /device/{deviceId}
| | * | Fix PEP8 errorsRichard van der Hoff2016-07-212-1/+2
| | | |
| | * | Implement GET /device/{deviceId}Richard van der Hoff2016-07-213-19/+89
| | | |
| * | | 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.
| * | Merge branch 'rav/get_devices_api' into developRichard van der Hoff2016-07-211-3/+0
| |\ \
| | * | storage/client_ips: remove some dead codeRichard van der Hoff2016-07-211-3/+0
| | | |
| * | | Merge pull request #939 from matrix-org/rav/get_devices_apiDavid Baker2016-07-2110-17/+395
| |\| | | | |/ | |/| GET /devices endpoint
| | * Fix PEP8 errorsRichard van der Hoff2016-07-202-3/+1
| | |
| | * Merge branch 'develop' into rav/get_devices_apiRichard van der Hoff2016-07-206-26/+86
| | |\ | | |/ | |/| | | | (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-203-8/+34
| |\ \ | | | | | | | | Record device_id in client_ips
| | * | Record device_id in client_ipsRichard van der Hoff2016-07-203-8/+34
| | | | | | | | | | | | | | | | | | | | 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-202-18/+49
| |\ \ \ | | |/ / | |/| | Register a device_id in the /v2/register flow.
| | * | Register a device_id in the /v2/register flow.Richard van der Hoff2016-07-202-18/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-2010-17/+397
| | |/ | |/| | | | | | | | | | | | | | | | 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-205-1/+54
| |\ | | | | | | Add metrics for psutil derived memory usage
| | * Make jenkins install deps on unit testsErik Johnston2016-07-201-0/+4
| | |
| | * 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-207-0/+47
| |\ \ | | | | | | | | Type annotations
| | * | MANIFEST.in: Add *.pyiRichard van der Hoff2016-07-191-0/+1
| | | |
| | * | 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-205-22/+57
| |\ \ \ | | |_|/ | |/| | Further registration refactoring
| | * | Further registration refactoringRichard van der Hoff2016-07-195-22/+57
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * `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-208-15/+246
| |\ \ | | |/ | |/| Feature: Add filter to /messages. Add 'contains_url' to filter.
| | * CommentErik Johnston2016-07-201-0/+4
| | |
| | * Add filter param to /messages APIErik Johnston2016-07-144-11/+29
| | |
| | * 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-192-79/+104
| |\ \ | | | | | | | | 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-192-76/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-1912-31/+354
| |\ \ \ | | |/ / | |/| | 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-1812-31/+354
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Merge pull request #930 from matrix-org/markjh/handlersMark Haines2016-07-191-3/+14
| |\ \ \ | | |/ / | |/| | Update docstring on Handlers.
| | * | 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
| | | | | |