Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Don't specify default as dict | Erik Johnston | 2017-04-25 | 1 | -1/+1 |
| | |||||
* | Reduce size of joined_user cache | Erik Johnston | 2017-04-25 | 1 | -2/+5 |
| | | | | | | | | The _get_joined_users_from_context cache stores a mapping from user_id to avatar_url and display_name. Instead of storing those in a dict, store them in a namedtuple as that uses much less memory. We also try converting the string to ascii to further reduce the size. | ||||
* | Support authenticated SMTP | Daniel Dent | 2017-04-05 | 1 | -1/+5 |
| | | | | | | Closes (SYN-714) #1385 Signed-off-by: Daniel Dent <matrixcontrib@contactdaniel.net> | ||||
* | Merge pull request #2075 from matrix-org/erikj/cache_speed | Erik Johnston | 2017-03-31 | 1 | -5/+2 |
|\ | | | | | Speed up cached function access | ||||
| * | Don't convert to deferreds when not necessary | Erik Johnston | 2017-03-30 | 1 | -5/+2 |
| | | |||||
* | | Cache glob to regex at a higher level for push | Erik Johnston | 2017-03-29 | 1 | -47/+57 |
|/ | |||||
* | Don't recreate so many sets | Erik Johnston | 2017-03-16 | 1 | -4/+4 |
| | |||||
* | Get current state by using current_state_events table | Erik Johnston | 2017-03-10 | 1 | -1/+1 |
| | |||||
* | Ensure we pass positive ints to delay function | Erik Johnston | 2017-02-22 | 1 | -1/+2 |
| | |||||
* | Fix email push in pusher worker | Erik Johnston | 2017-02-02 | 1 | -2/+2 |
| | | | | | | This was broken when device list updates were implemented, as Mailer could no longer instantiate an AuthHandler due to a dependency on federation sending. | ||||
* | Merge pull request #1811 from aperezdc/unhardcode-riot-urls | David Baker | 2017-01-18 | 1 | -6/+14 |
|\ | | | | | Allow configuring the Riot URL used in notification emails | ||||
| * | Allow configuring the Riot URL used in notification emails | Adrian Perez de Castro | 2017-01-13 | 1 | -6/+14 |
| | | | | | | | | | | | | | | | | | | | | The URLs used for notification emails were hardcoded to use either matrix.to or vector.im; but for self-hosted setups where Riot is also self-hosted it may be desirable to allow configuring an alternative Riot URL. Fixes #1809. Signed-off-by: Adrian Perez de Castro <aperez@igalia.com> | ||||
* | | Get state at event rather than for room in push | Erik Johnston | 2017-01-17 | 1 | -1/+1 |
|/ | |||||
* | Add /room/<room_id>/joined_members API | Erik Johnston | 2016-12-08 | 1 | -6/+6 |
| | | | | | | This returns the currently joined members in the room with their display names and avatar urls. This is more efficient than /members for large rooms where you don't need the full events. | ||||
* | Fix push notifications for a single unread message | Erik Johnston | 2016-10-18 | 1 | -1/+1 |
| | |||||
* | Reduce redundant database work in email pusher | Erik Johnston | 2016-10-18 | 1 | -0/+4 |
| | | | | | | | Update the last stream ordering if the `get_unread_push_actions_for_user_in_range_for_email` returns no new push actions. This reduces the range that it needs to check next iteration. | ||||
* | Fix email push notifs being dropped | Erik Johnston | 2016-10-13 | 1 | -14/+25 |
| | | | | | | | | A lot of email push notifications were failing to be sent due to an exception being thrown along one of the (many) paths. This was due to a change where we moved from pulling out the full state for each room, but rather pulled out the event ids for the state and separately loaded the full events when needed. | ||||
* | _id field must uniquely identify different conditions | Erik Johnston | 2016-09-19 | 1 | -2/+2 |
| | |||||
* | proposal for notifying on e2e events | Matthew Hodgson | 2016-09-17 | 1 | -0/+49 |
| | |||||
* | Ensure we don't mutate the cache of push rules | Erik Johnston | 2016-09-15 | 1 | -9/+1 |
| | |||||
* | Make bulk_get_push_rules_for_room use get_joined_users_from_context cache | Erik Johnston | 2016-09-05 | 1 | -1/+1 |
| | |||||
* | Merge pull request #1060 from matrix-org/erikj/state_ids | Erik Johnston | 2016-09-01 | 1 | -1/+1 |
|\ | | | | | Assign state groups in state handler. | ||||
| * | Correctly handle the difference between prev and current state | Erik Johnston | 2016-08-31 | 1 | -1/+1 |
| | | |||||
* | | Fix email notifs by adding missing param | Erik Johnston | 2016-08-30 | 1 | -1/+1 |
|/ | |||||
* | Fix push room names for rooms with only an alias | Erik Johnston | 2016-08-30 | 1 | -1/+1 |
| | |||||
* | Use state handler instead of get_users_in_room/get_joined_hosts | Erik Johnston | 2016-08-26 | 1 | -1/+1 |
| | |||||
* | Fix room name in email notifs | Erik Johnston | 2016-08-26 | 1 | -19/+20 |
| | |||||
* | Typo | Erik Johnston | 2016-08-26 | 1 | -1/+0 |
| | |||||
* | Fix up push to use get_current_state_ids | Erik Johnston | 2016-08-25 | 4 | -20/+226 |
| | |||||
* | Replace context.current_state with context.current_state_ids | Erik Johnston | 2016-08-25 | 2 | -19/+17 |
| | |||||
* | Preserve some logcontexts | Erik Johnston | 2016-08-24 | 2 | -9/+12 |
| | |||||
* | Move _bulk_get_push_rules_for_room to storage layer | Erik Johnston | 2016-08-19 | 2 | -34/+9 |
| | |||||
* | Make push Measure finer grained | Erik Johnston | 2016-08-17 | 1 | -4/+5 |
| | |||||
* | Merge remote-tracking branch 'origin/develop' into ↵ | David Baker | 2016-08-16 | 1 | -4/+4 |
|\ | | | | | | | dbkr/contains_display_name_override | ||||
| * | Refactor user_delete_access_tokens. Invalidate get_user_by_access_token to ↵ | Erik Johnston | 2016-08-15 | 1 | -4/+4 |
| | | | | | | | | slaves. | ||||
* | | s/underride/override/ in the rule_id too | David Baker | 2016-08-16 | 1 | -1/+1 |
| | | |||||
* | | Move display name rule | David Baker | 2016-08-15 | 1 | -16/+20 |
|/ | | | | As per https://github.com/matrix-org/matrix-doc/pull/373 and comment | ||||
* | Ignore AlreadyCalled errors on timer cancel | Erik Johnston | 2016-07-31 | 2 | -3/+16 |
| | |||||
* | Create separate methods for getting messages to push | Mark Haines | 2016-07-28 | 2 | -4/+4 |
| | | | | | | | | | | | 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. | ||||
* | Typo | David Baker | 2016-07-28 | 1 | -1/+1 |
| | |||||
* | even more pep8 | David Baker | 2016-06-24 | 1 | -0/+1 |
| | |||||
* | Use similar naming we use in email notifs for push | David Baker | 2016-06-24 | 2 | -20/+22 |
| | | | | Fixes https://github.com/vector-im/vector-web/issues/1654 | ||||
* | Fix ``KeyError: 'msgtype'``. Use ``.get`` | Mark Haines | 2016-06-17 | 1 | -14/+15 |
| | | | | | | | | | | | | | Fixes a key error where the mailer tried to get the ``msgtype`` of an event that was missing a ``msgtype``. ``` File "synapse/push/mailer.py", line 264, in get_notif_vars File "synapse/push/mailer.py", line 285, in get_message_vars File ".../frozendict/__init__.py", line 10, in __getitem__ return self.__dict[key] KeyError: 'msgtype' ``` | ||||
* | Remove dead code. | Mark Haines | 2016-06-07 | 2 | -34/+0 |
| | | | | | | | | | | | Loading push rules now happens in the datastore, so we can remove the methods that loaded them outside the datastore. The ``waiting_for_join_list`` in federation handler is populated by anything, so can be removed. The ``_get_members_events_txn`` method isn't called from anywhere so can be removed. | ||||
* | 3rd time lucky: we'd already calculated it above | David Baker | 2016-06-03 | 1 | -3/+1 |
| | |||||
* | Oops, we're using the dict form | David Baker | 2016-06-03 | 1 | -1/+3 |
| | |||||
* | Add substitutions to email notif From | David Baker | 2016-06-03 | 1 | -1/+1 |
| | |||||
* | Merge branch 'develop' into matthew/brand-from-header | Matthew Hodgson | 2016-06-03 | 5 | -61/+67 |
|\ | |||||
| * | Load push rules in storage layer, so that they get cached | Erik Johnston | 2016-06-03 | 2 | -36/+22 |
| | | |||||
| * | Remove unnecessary sets | Erik Johnston | 2016-06-03 | 1 | -6/+3 |
| | | |||||
| * | Only get local users | Erik Johnston | 2016-06-03 | 1 | -2/+5 |
| | | |||||
| * | Merge pull request #811 from matrix-org/erikj/state_users_in_room | Erik Johnston | 2016-06-03 | 2 | -11/+18 |
| |\ | | | | | | | Use state to calculate get_users_in_room | ||||
| | * | Use state to calculate get_users_in_room | Erik Johnston | 2016-06-01 | 2 | -11/+18 |
| | | | |||||
| * | | Merge pull request #809 from matrix-org/erikj/cache_receipts_in_room | Erik Johnston | 2016-06-03 | 1 | -4/+4 |
| |\| | | | | | | | Add get_users_with_read_receipts_in_room cache | ||||
| | * | Add get_users_with_read_receipts_in_room cache | Erik Johnston | 2016-06-01 | 1 | -4/+4 |
| | | | |||||
| * | | Merge pull request #821 from matrix-org/dbkr/email_unsubscribe | David Baker | 2016-06-02 | 2 | -6/+19 |
| |\ \ | | | | | | | | | Email unsubscribe links that don't require logging in | ||||
| | * | | peppate | David Baker | 2016-06-02 | 1 | -1/+3 |
| | | | | |||||
| | * | | Working unsubscribe links going straight to the HS | David Baker | 2016-06-02 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | and authed by macaroons that let you delete pushers and nothing else | ||||
| | * | | Merge remote-tracking branch 'origin/dbkr/email_notif_string_fmt_error' into ↵ | David Baker | 2016-06-02 | 1 | -1/+1 |
| | |\ \ | | | | | | | | | | | | | | | | dbkr/email_unsubscribe | ||||
| | * \ \ | Merge remote-tracking branch 'origin/develop' into dbkr/email_unsubscribe | David Baker | 2016-06-02 | 2 | -4/+10 |
| | |\ \ \ | |||||
| | * \ \ \ | Merge branch 'dbkr/split_out_auth_handler' into dbkr/email_unsubscribe | David Baker | 2016-06-02 | 1 | -0/+2 |
| | |\ \ \ \ | |||||
| | * | | | | | Email unsubscribing that may in theory, work | David Baker | 2016-06-02 | 2 | -6/+17 |
| | | |_|_|/ | | |/| | | | | | | | | | | | | | | | Were it not for that fact that you can't use the base handler in the pusher because it pulls in the world. Comitting while I fix that on a different branch. | ||||
| * | | | | | Merge pull request #820 from matrix-org/dbkr/email_notif_string_fmt_error | David Baker | 2016-06-02 | 1 | -1/+1 |
| |\ \ \ \ \ | | | |_|_|/ | | |/| | | | Fix error in email notification string formatting | ||||
| | * | | | | Fix error in email notification string formatting | David Baker | 2016-06-02 | 1 | -1/+1 |
| | | |_|/ | | |/| | | |||||
* | / | | | brand the email from header | Matthew Hodgson | 2016-06-02 | 1 | -1/+8 |
|/ / / / | |||||
* / / / | add some branding debugging | Matthew Hodgson | 2016-06-02 | 1 | -0/+1 |
|/ / / | |||||
* | | | Disable INCLUDE_ALL_UNREAD_NOTIFS | Erik Johnston | 2016-06-02 | 1 | -1/+2 |
| | | | |||||
* | | | if an email pusher specifies a brand param, use it | Matthew Hodgson | 2016-06-02 | 2 | -3/+8 |
| |/ |/| | |||||
* | | Use user_id in email greeting if display name is null | David Baker | 2016-06-02 | 1 | -0/+2 |
|/ | |||||
* | Add caches to bulk_get_push_rules* | Erik Johnston | 2016-05-31 | 1 | -3/+5 |
| | |||||
* | Check for presence of 'avatar_url' key | David Baker | 2016-05-24 | 1 | -1/+3 |
| | |||||
* | fix NPE in room ordering | Matthew Hodgson | 2016-05-23 | 1 | -2/+2 |
| | |||||
* | consistency is the better part of valour | Matthew Hodgson | 2016-05-23 | 1 | -1/+1 |
| | |||||
* | Tune email notifs to make them quieter: | Matthew Hodgson | 2016-05-23 | 2 | -18/+56 |
| | | | | | | | | * After initial 10 minute window, only alert every 24h for room notifs * Reset room state after 6h of idleness * Synchronise throttles for messages sent in the same notif, so the 24 hourly notifs 'line up' * Fix the email subjects to say what triggered the notification * Order the rooms in reverse activity order in the email, so the 'reason' room should always come first | ||||
* | Merge pull request #793 from matrix-org/matthew/one-push-badge-per-convo | Matthew Hodgson | 2016-05-18 | 1 | -1/+3 |
|\ | | | | | increment badge count per missed convo, not per msg | ||||
| * | increment badge count per missed convo, not per msg | Matthew Hodgson | 2016-05-18 | 1 | -1/+3 |
| | | |||||
* | | Merge pull request #786 from matrix-org/matthew/email_notifs_tuning | Matthew Hodgson | 2016-05-17 | 2 | -9/+30 |
|\| | | | | | tune email notifs, fix CSS a bit, and add debugging details | ||||
| * | tune email notifs, fix CSS a bit, and add debugging details | Matthew Hodgson | 2016-05-16 | 2 | -9/+30 |
| | | |||||
* | | Fix emailpusher import | David Baker | 2016-05-16 | 1 | -1/+11 |
|/ | | | | Try importing at the root level rather than conditionally importing, as per comment | ||||
* | fix matrix.to URLs | Matthew Hodgson | 2016-05-13 | 1 | -2/+2 |
| | |||||
* | Make email notifs work on the pusher synapse | David Baker | 2016-05-13 | 1 | -0/+1 |
| | | | | Plus general bugfix to email notif code | ||||
* | move filter_events_for_client out of base handler | Mark Haines | 2016-05-11 | 3 | -8/+9 |
| | |||||
* | Make pep8 happy | David Baker | 2016-05-10 | 1 | -1/+1 |
| | |||||
* | Switch from CSS to Table layout for HTML mails so they work in Outlook ↵ | Matthew Hodgson | 2016-05-10 | 1 | -7/+18 |
| | | | | | | | aka Word Remove templates-vector and theme templates with variables instead Switch to matrix.to URLs by default for links | ||||
* | better mail subject lines | Matthew Hodgson | 2016-05-05 | 1 | -1/+1 |
| | |||||
* | fix layout; handle app naming in synapse, not jinja | Matthew Hodgson | 2016-05-05 | 1 | -16/+31 |
| | |||||
* | pep8 | Matthew Hodgson | 2016-05-05 | 1 | -2/+4 |
| | |||||
* | handle fragments correctly on mxc URLs. | Matthew Hodgson | 2016-05-05 | 1 | -14/+45 |
| | | | | | | | switch to vector.im permalinks as matrix.to isn't ready yet. merge overlapping notifications together. give one message of context after a notification (in the unlikely event it exists, but it's possible thanks to throttling). include name of app in mail templates | ||||
* | spell out emailpusher full path | Matthew Hodgson | 2016-05-05 | 1 | -1/+1 |
| | |||||
* | Include no context | David Baker | 2016-05-04 | 1 | -1/+1 |
| | | | | until we can de-dup between the context and other notifs | ||||
* | Add date header & message id | David Baker | 2016-05-04 | 1 | -0/+2 |
| | |||||
* | pep8 | David Baker | 2016-05-04 | 1 | -0/+1 |
| | |||||
* | copyright | Matthew Hodgson | 2016-05-04 | 1 | -0/+22 |
| | |||||
* | Unused import | David Baker | 2016-05-04 | 1 | -1/+0 |
| | |||||
* | Catch all exceptions when creating a pusher | David Baker | 2016-05-04 | 1 | -2/+2 |
| | |||||
* | Add constant for throttle multiplier | David Baker | 2016-04-29 | 1 | -1/+2 |
| | |||||
* | Remove unused function | David Baker | 2016-04-29 | 1 | -4/+0 |
| | |||||
* | Remove unused arg | David Baker | 2016-04-29 | 1 | -2/+2 |
| | |||||
* | Use the constant | David Baker | 2016-04-29 | 1 | -1/+2 |
| | |||||
* | Avoid `vars` builtin | David Baker | 2016-04-29 | 1 | -15/+15 |
| | |||||
* | Use named parameter format | David Baker | 2016-04-29 | 1 | -14/+22 |
| | |||||
* | pep8 | David Baker | 2016-04-29 | 1 | -1/+0 |
| | |||||
* | Remove rather pointless get function | David Baker | 2016-04-29 | 1 | -9/+1 |
| | |||||
* | Stop processing notifs once we've sent a mail | David Baker | 2016-04-29 | 1 | -0/+1 |
| | |||||
* | Send mail notifs with a plaintext part too | David Baker | 2016-04-29 | 1 | -7/+20 |
| | |||||
* | Add an email pusher for new users | David Baker | 2016-04-29 | 1 | -0/+1 |
| | | | | If they registered with an email address and email notifs are enabled on the HS | ||||
* | Merge remote-tracking branch 'origin/develop' into dbkr/email_notifs | David Baker | 2016-04-29 | 2 | -1/+5 |
|\ | |||||
| * | Optionally split out the pushers into a separate process | Mark Haines | 2016-04-21 | 2 | -1/+5 |
| | | |||||
* | | May as well always include room link | David Baker | 2016-04-28 | 1 | -4/+3 |
| | | |||||
* | | Put back real delay before mailing | David Baker | 2016-04-28 | 1 | -1/+1 |
| | | |||||
* | | Support file messages & fix plain text | David Baker | 2016-04-28 | 1 | -2/+3 |
| | | |||||
* | | Support image notifs | David Baker | 2016-04-28 | 1 | -11/+25 |
| | | |||||
* | | pep8 | David Baker | 2016-04-28 | 1 | -2/+5 |
| | | |||||
* | | Only import email pusher if email notifs are on | David Baker | 2016-04-28 | 1 | -6/+8 |
| | | |||||
* | | Handle room invites in email notifs | David Baker | 2016-04-28 | 1 | -5/+30 |
| | | |||||
* | | Linkify plain text messages too | David Baker | 2016-04-28 | 1 | -1/+12 |
| | | |||||
* | | Run filter_events_for_client | David Baker | 2016-04-27 | 1 | -7/+11 |
| | | | | | | | | so we don't accidentally mail out events people shouldn't see | ||||
* | | Make html messages work | David Baker | 2016-04-27 | 1 | -10/+21 |
| | | |||||
* | | Better grammar for multiple messages in a room | David Baker | 2016-04-27 | 2 | -10/+33 |
| | | | | | | | | Say who the messages are from if there's no room name, otherwise it's a bit nonsensical | ||||
* | | Hopefully all remaining bits for email notifs | David Baker | 2016-04-27 | 1 | -22/+144 |
| | | | | | | | | Add public facing base url to the server so synapse knows what URL to use when converting mxc to http urls for use in emails | ||||
* | | More variable calculation for email notifs | David Baker | 2016-04-25 | 1 | -2/+55 |
| | | | | | | | | Include name of the person we're sending to and add summary text at the top giving an overview of what's happened. | ||||
* | | Actually do UTF8 correctly | David Baker | 2016-04-22 | 1 | -1/+1 |
| | | |||||
* | | Mime part is binary so encode it first. | David Baker | 2016-04-22 | 1 | -1/+1 |
| | | | | | | | | Doesn't get character enocind right yet but makes it not error. | ||||
* | | No inlineCallbacks necessary on this | David Baker | 2016-04-22 | 1 | -1/+0 |
| | | |||||
* | | Flesh out email templating | David Baker | 2016-04-21 | 2 | -3/+65 |
| | | | | | | | | Mostly WIP porting the room name calculation logic from the web client so our room names in the email mirror the clients. | ||||
* | | Generate mails from a template | David Baker | 2016-04-20 | 2 | -21/+21 |
| | | |||||
* | | more pep8 | David Baker | 2016-04-20 | 1 | -1/+1 |
| | | |||||
* | | Send a rather basic email notif | David Baker | 2016-04-20 | 2 | -9/+71 |
| | | | | | | | | Also pep8 fixes | ||||
* | | Add single instance & logging stuff | David Baker | 2016-04-19 | 1 | -8/+39 |
| | | | | | | | | Copy the stuff over from http pusher that prevents multiple instances of process running at once and sets up logging and measure blocks. | ||||
* | | First bits of emailpusher | David Baker | 2016-04-19 | 2 | -1/+217 |
|/ | | | | Mostly logic of when to send an email | ||||
* | Measure push action generator | Erik Johnston | 2016-04-14 | 1 | -11/+15 |
| | |||||
* | Merge pull request #725 from matrix-org/dbkr/push_only_joined | Erik Johnston | 2016-04-14 | 1 | -1/+13 |
|\ | | | | | Don't push for everyone who ever sent an RR to the room | ||||
| * | Don't push for everyone who ever sent an RR to the room | David Baker | 2016-04-14 | 1 | -1/+13 |
| | | |||||
* | | Add missing yield | Erik Johnston | 2016-04-14 | 1 | -1/+1 |
| | | |||||
* | | Rename log context | Erik Johnston | 2016-04-14 | 1 | -1/+1 |
| | | |||||
* | | Make send_badge private | Erik Johnston | 2016-04-14 | 1 | -2/+2 |
| | | |||||
* | | Add necessary logging contexts | Erik Johnston | 2016-04-14 | 1 | -24/+25 |
| | | |||||
* | | Add extra Measure | Erik Johnston | 2016-04-14 | 1 | -14/+16 |
|/ | |||||
* | Unused member variable | David Baker | 2016-04-12 | 1 | -1/+0 |
| | |||||
* | Fix the rule_id for .m.rule.invite_for_me (#715) | Mark Haines | 2016-04-11 | 1 | -1/+1 |
| | |||||
* | Run unsafe proces in a loop until we've caught up | David Baker | 2016-04-11 | 1 | -4/+11 |
| | | | | and wrap unsafe process in a try block | ||||
* | Actually check if we;re processing | David Baker | 2016-04-08 | 1 | -0/+2 |
| | |||||
* | Fix invite pushes | David Baker | 2016-04-08 | 3 | -39/+51 |
| | | | | | | * If the event is an invite event, add the invitee to list of user we run push rules for (if they have a pusher etc) * Move invite_for_me to be higher prio than member events otherwise member events matches them * Spell override right | ||||
* | Make sure max stream ordering only increases | David Baker | 2016-04-08 | 1 | -1/+1 |
| | |||||
* | Unsafe process should call itself if the max has changed | David Baker | 2016-04-08 | 1 | -0/+3 |
| | |||||
* | Comment why unsafe process is unsafe | David Baker | 2016-04-08 | 1 | -0/+5 |
| | |||||
* | Only pass in what we need | David Baker | 2016-04-07 | 2 | -11/+13 |
| | |||||
* | Wrap process in a flag so we don't process whist already processing. | David Baker | 2016-04-07 | 1 | -0/+9 |
| | |||||
* | indenting | David Baker | 2016-04-07 | 1 | -1/+1 |
| | |||||
* | defer.gatherResults rather than doing all the pokes in series | David Baker | 2016-04-07 | 1 | -2/+16 |
| | |||||
* | Add measure blocks | David Baker | 2016-04-07 | 1 | -5/+10 |
| | |||||
* | Add comments on min_stream_id | David Baker | 2016-04-07 | 1 | -0/+2 |
| | | | | saying that the min stream id won't be completely accurate all the time | ||||
* | Remove code that's now been obsoleted or moved elsewhere | David Baker | 2016-04-07 | 3 | -454/+9 |
| | |||||
* | Send badge count pushes. | David Baker | 2016-04-07 | 2 | -17/+48 |
| | | | | Also fix bugs with retrying. | ||||
* | Make pushers use the event_push_actions table instead of listening on an ↵ | David Baker | 2016-04-06 | 5 | -79/+301 |
| | | | | | | | event stream & running the rules again. Sytest passes, but remaining to do: * Make badges work again * Remove old, unused code | ||||
* | Use google style doc strings. | Mark Haines | 2016-04-01 | 1 | -3/+5 |
| | | | | | | | pycharm supports them so there is no need to use the other format. Might as well convert the existing strings to reduce the risk of people accidentally cargo culting the wrong doc string format. | ||||
* | Don't ignore the obey overlay if the rule has an enabled attribute of False | David Baker | 2016-03-31 | 1 | -2/+3 |
| | | | | Fixes https://github.com/vector-im/vector-web/issues/1244 | ||||
* | Never notify for member events. This fixes ↵ | David Baker | 2016-03-24 | 1 | -20/+21 |
| | | | | https://github.com/vector-im/vector-web/issues/828 | ||||
* | Make get_invites return RoomsForUser | Erik Johnston | 2016-03-23 | 1 | -1/+1 |
| | |||||
* | Use get_users_in_room to count the number of room members rather than using ↵ | Mark Haines | 2016-03-22 | 1 | -1/+3 |
| | | | | read_receipts | ||||
* | Make select more sensible when dseleting access tokens, rename pusher ↵ | David Baker | 2016-03-11 | 1 | -1/+1 |
| | | | | deletion to match access token deletion and make exception arg optional. | ||||
* | Fix cache invalidation so deleting access tokens (which we did when changing ↵ | David Baker | 2016-03-11 | 1 | -4/+4 |
| | | | | password) actually takes effect without HS restart. Reinstate the code to avoid logging out the session that changed the password, removed in 415c2f05491ce65a4fc34326519754cd1edd9c54 | ||||
* | Fix relative imports so they work in both py3 and py27 | Mark Haines | 2016-03-08 | 5 | -10/+10 |
| | |||||
* | Move the code for formatting push rules into a separate function | Mark Haines | 2016-03-03 | 1 | -0/+112 |
| | |||||
* | Add support for changing the actions for default rules | Mark Haines | 2016-02-26 | 1 | -7/+50 |
| | | | | | | See matrix-org/matrix-doc#283 Works by adding dummy rules to the push rules table with a negative priority class and then using those rules to clobber the default rule actions when adding the default rules in ``list_with_base_rules`` | ||||
* | Merge branch 'develop' into rav/guest_access_after_room_join | Richard van der Hoff | 2016-02-19 | 6 | -46/+31 |
|\ | |||||
| * | Remove dead code for setting device specific rules. | Mark Haines | 2016-02-18 | 6 | -46/+31 |
| | | | | | | | | | | | | It wasn't possible to hit the code from the API because of a typo in parsing the request path. Since no-one was using the feature we might as well remove the dead code. | ||||
* | | Allow guest users access to messages in rooms they have joined | Richard van der Hoff | 2016-02-19 | 1 | -1/+1 |
|/ | | | | | | | There should be no difference between guest users and non-guest users in terms of access to messages. Define the semantics of the is_peeking argument to filter_events_for_clients (slightly) better; interpret it appropriately, and set it correctly from /sync. | ||||
* | Atomically persit push actions when we persist the event | Erik Johnston | 2016-02-09 | 1 | -15/+5 |
| | |||||
* | Pass in current state to push action handler | Erik Johnston | 2016-02-09 | 2 | -9/+7 |
| | |||||
* | Revert to putting it around the entire block | Erik Johnston | 2016-02-09 | 1 | -124/+124 |
| | |||||
* | Don't measure across event stream call, as it lasts for a long time. | Erik Johnston | 2016-02-09 | 1 | -124/+124 |
| | |||||
* | Fix up logcontexts | Erik Johnston | 2016-02-08 | 2 | -5/+6 |
| | |||||
* | Add metrics to pushers | Erik Johnston | 2016-02-08 | 1 | -31/+53 |
| | |||||
* | Change event_push_actions_rm_tokens schema | Erik Johnston | 2016-02-03 | 1 | -1/+1 |
| | |||||
* | Fix flake8 warnings for new flake8 | Daniel Wagner-Hall | 2016-02-02 | 1 | -1/+1 |
| | |||||
* | Only notify for messages in one to one rooms, not every event | David Baker | 2016-01-26 | 1 | -0/+6 |
| | | | | Fixes the fact that candidate events and hangups generated notifications. | ||||
* | Push: Use storage apis that are cached | Erik Johnston | 2016-01-25 | 1 | -17/+13 |
| | |||||
* | Merge pull request #523 from matrix-org/dbkr/no_push_unless_notify | David Baker | 2016-01-22 | 1 | -9/+1 |
|\ | | | | | Better fix for actions with both dont_notify and tweaks | ||||
| * | Better fix for actions with both dont_notify and tweaks | David Baker | 2016-01-22 | 1 | -1/+1 |
| | | |||||
| * | Revert b4a41aa542203c03bb8a6c93097b94bc5d167265 as it's just broken. | David Baker | 2016-01-22 | 1 | -9/+1 |
| | | |||||
* | | Merge pull request #522 from matrix-org/dbkr/no_push_unless_notify | David Baker | 2016-01-22 | 1 | -1/+9 |
|\| | | | | | Don't add notifications to the table unless there's actually a 'notify' action | ||||
| * | Don't add notifications to the table unless there's actually a 'notify' action | David Baker | 2016-01-22 | 1 | -1/+9 |
| | | |||||
* | | Don't mutate cached values | Erik Johnston | 2016-01-22 | 1 | -2/+6 |
|/ | |||||
* | Don't crash if a user has no push rule enabled entries | David Baker | 2016-01-22 | 1 | -0/+3 |
| | |||||
* | Overlay the push_rules_enabled map for users, otherwise they won't be able ↵ | David Baker | 2016-01-22 | 1 | -0/+14 |
| | | | | to disable server default rules. | ||||
* | Only fetch events for rooms and receipts | Erik Johnston | 2016-01-21 | 1 | -1/+2 |
| | |||||
* | Only compute badge count when necessary. | Erik Johnston | 2016-01-21 | 1 | -4/+15 |
| | | | | This reverts commit d7265977376eb391007cde55c4b2d9b8f54d452b. | ||||
* | Remove member event rule as per SYN-607 | David Baker | 2016-01-21 | 1 | -17/+19 |
| | |||||
* | Add storage function to get all receipts for a user. Also add some cache ↵ | David Baker | 2016-01-20 | 1 | -22/+6 |
| | | | | invalidation to the receipts storage because there wasn't any, and remove a method that was unused. | ||||
* | Inline membership specifier | David Baker | 2016-01-20 | 1 | -3/+1 |
| | |||||
* | Simplify badge updating code by just updating it every time we get woken up ↵ | David Baker | 2016-01-20 | 1 | -15/+4 |
| | | | | and it's not an event | ||||
* | Merge branch 'develop' into push_badge_counts | David Baker | 2016-01-20 | 1 | -1/+1 |
|\ | |||||
| * | 'filtered' is a list of zero or 1 | David Baker | 2016-01-20 | 1 | -1/+1 |
| | | |||||
* | | Merge remote-tracking branch 'origin/develop' into push_badge_counts | David Baker | 2016-01-20 | 1 | -0/+3 |
|\| | |||||
| * | Don't generate push actions for our own events | David Baker | 2016-01-20 | 1 | -0/+3 |
| | | |||||
* | | Merge branch 'develop' into push_badge_counts | David Baker | 2016-01-19 | 7 | -393/+509 |
|\| | |||||
| * | Change regex cache size to 5000 | Erik Johnston | 2016-01-19 | 1 | -1/+1 |
| | | |||||
| * | Add regex cache. Only caculate push actions for users that have sent read ↵ | Erik Johnston | 2016-01-19 | 3 | -11/+31 |
| | | | | | | | | receipts, and are on that server | ||||
| * | Handle glob -> regex errors | Erik Johnston | 2016-01-19 | 1 | -29/+33 |
| | | |||||
| * | Return don't break | Erik Johnston | 2016-01-19 | 1 | -1/+1 |
| | | |||||
| * | Except truthy values | Erik Johnston | 2016-01-19 | 1 | -1/+1 |
| | | |||||
| * | Merge branch 'develop' of github.com:matrix-org/synapse into default_notify | Erik Johnston | 2016-01-19 | 6 | -385/+478 |
| |\ | |||||
| | * | Update _id | Erik Johnston | 2016-01-19 | 1 | -2/+3 |
| | | | |||||
| | * | Preserve truthiness | Erik Johnston | 2016-01-18 | 1 | -1/+2 |
| | | | |||||
| | * | You need to escape backslashes | Erik Johnston | 2016-01-18 | 1 | -2/+2 |
| | | | |||||
| | * | Fix branch didn't check word_boundary | Erik Johnston | 2016-01-18 | 1 | -3/+9 |
| | | | |||||
| | * | Don't split at word boundaries, actually use regex | Erik Johnston | 2016-01-18 | 2 | -63/+48 |
| | | | |||||
| | * | Drop log levels | Erik Johnston | 2016-01-18 | 1 | -2/+2 |
| | | | |||||
| | * | Add comments and remove dead code | Erik Johnston | 2016-01-18 | 1 | -12/+11 |
| | | | |||||
| | * | Do for loop once at start | Erik Johnston | 2016-01-18 | 1 | -8/+17 |
| | | | |||||
| | * | Use static for const dicts | Erik Johnston | 2016-01-18 | 3 | -197/+202 |
| | | | |||||
| | * | Don't edit ruleset | Erik Johnston | 2016-01-18 | 1 | -5/+4 |
| | | | |||||
| | * | Re-enable urnead notifications | Erik Johnston | 2016-01-18 | 1 | -3/+0 |
| | | | |||||
| | * | Make notifications go quicker | Erik Johnston | 2016-01-18 | 2 | -127/+215 |
| | | | |||||
| | * | Temporarily disable notification branch | Erik Johnston | 2016-01-13 | 1 | -0/+3 |
| | | | |||||
| | * | Rename 'user_name' to 'user_id' in push to make it consistent with the rest ↵ | Mark Haines | 2016-01-13 | 5 | -54/+54 |
| | | | | | | | | | | | | of the code | ||||
| * | | Change default pushrules back to notifying for all messages. | David Baker | 2016-01-13 | 1 | -1/+0 |
| |/ | |||||
* / | Use the unread notification count to send accurate badge counts in push ↵ | David Baker | 2016-01-13 | 3 | -45/+86 |
|/ | | | | notifications. | ||||
* | Merge pull request #456 from matrix-org/store_event_actions | David Baker | 2016-01-08 | 4 | -22/+200 |
|\ | | | | | Send unread notification counts | ||||
| * | more commas | David Baker | 2016-01-07 | 1 | -1/+1 |
| | | |||||
| * | Delete notifications for redacted events | David Baker | 2016-01-06 | 1 | -0/+7 |
| | | |||||
| * | Add is_guest flag to users db to track whether a user is a guest user or ↵ | David Baker | 2016-01-06 | 2 | -8/+25 |
| | | | | | | | | not. Use this so we can run _filter_events_for_client when calculating event_push_actions. | ||||
| * | comment typo | David Baker | 2016-01-05 | 1 | -1/+1 |
| | | |||||
| * | Rename event-actions to event_push_actions as per PR request | David Baker | 2016-01-04 | 2 | -2/+2 |
| | | |||||
| * | Address minor PR issues | David Baker | 2016-01-04 | 2 | -5/+4 |
| | | |||||
| * | pep8 & unused variable | David Baker | 2015-12-22 | 3 | -4/+3 |
| | | |||||
| * | Add some comments to areas that could be optimised. | David Baker | 2015-12-22 | 2 | -1/+11 |
| | | |||||
| * | Insert push actions in a single db query rather than one per user/profile_tag | David Baker | 2015-12-22 | 1 | -4/+6 |
| | | |||||
| * | Add bulk push rule evaluator which actually still evaluates rules one by ↵ | David Baker | 2015-12-22 | 3 | -22/+118 |
| | | | | | | | | one, but does far fewer db queries to fetch the rules | ||||
| * | Only run pushers for users on this hs! | David Baker | 2015-12-21 | 1 | -1/+7 |
| | | |||||
| * | still very WIP, but now sends unread_notifications_count in the room object ↵ | David Baker | 2015-12-16 | 1 | -1/+1 |
| | | | | | | | | on sync (only actually corrrect in a full sync: hardcoded to 0 in incremental syncs). | ||||
| * | Store nothing instead of ['dont_notify'] for events with no notification ↵ | David Baker | 2015-12-10 | 3 | -21/+14 |
| | | | | | | | | required: much as it would be nice to be able to tell between the event not having been processed and there being no notification for it, this isn't worth filling up the table with ['dont_notify'] I think. Consequently treat the empty actions array as dont_notify and filter dont_notify out of the result. | ||||
| * | Save event actions to the db | David Baker | 2015-12-10 | 1 | -2/+4 |
| | | |||||
| * | Very first cut of calculating actions for events as they come in. Doesn't ↵ | David Baker | 2015-12-10 | 1 | -0/+47 |
| | | | | | | | | store them yet. Not very efficient. | ||||
* | | copyrights | Matthew Hodgson | 2016-01-07 | 6 | -6/+6 |
|/ | |||||
* | pep8 | David Baker | 2015-12-09 | 1 | -4/+4 |
| | |||||
* | Split out the push rule evaluator into a separate file so it can be more ↵ | David Baker | 2015-12-09 | 2 | -185/+234 |
| | | | | readily reused. Should be functionally identical. | ||||
* | Merge pull request #413 from matrix-org/markjh/reuse_http_client | Mark Haines | 2015-12-02 | 1 | -4/+3 |
|\ | | | | | Reuse a single http client, rather than creating new ones | ||||
| * | Reuse a single http client, rather than creating new ones | Mark Haines | 2015-12-02 | 1 | -4/+3 |
| | | |||||
* | | Change the m.room.message rule to be disabled by default so we only notify ↵ | David Baker | 2015-12-02 | 1 | -0/+1 |
|/ | | | | for 1:1 rooms / highlights out-of-the-box | ||||
* | Use raw string for regex here, otherwise \b is the backspace character. ↵ | David Baker | 2015-10-08 | 1 | -1/+1 |
| | | | | Fixes displayname matching. | ||||
* | Don't make pushers handle presence/typing events | Erik Johnston | 2015-08-24 | 1 | -3/+5 |
| | |||||
* | Store the 'last_token' in the db, even if we processed no events | Erik Johnston | 2015-08-19 | 1 | -0/+6 |
| | |||||
* | Block on storing the current last_tokens | Erik Johnston | 2015-08-19 | 1 | -5/+5 |
| | |||||
* | Simplify LoginHander and AuthHandler | Daniel Wagner-Hall | 2015-08-12 | 1 | -7/+4 |
| | | | | | | | | | * Merge LoginHandler -> AuthHandler * Add a bunch of documentation * Improve some naming * Remove unused branches I will start merging the actual logic of the two handlers shortly | ||||
* | Oops: underride rule had an identifier with override in it. | David Baker | 2015-06-23 | 1 | -1/+1 |
| | |||||
* | Catch any exceptions in the pusher loop. Use a lower timeout for pushers so ↵ | David Baker | 2015-06-05 | 1 | -112/+133 |
| | | | | we can see if they're actually still running. | ||||
* | Take a dict of the rule, not the rule list | Mark Haines | 2015-05-26 | 1 | -1/+1 |
| | |||||
* | SYN-390: Don't modify the dictionary returned from the database here either | Mark Haines | 2015-05-26 | 1 | -4/+7 |
| |