mirror of
https://github.com/zulip/zulip.git
synced 2025-11-04 05:53:43 +00:00
c57c4cf703
Previously, Zulip did not correctly handle the case of a mobile device being registered with a push device token being registered for multiple accounts on the same server (which is a common case on zulipchat.com). This was because our database `unique` and `unique_together` indexes incorrectly enforced the token being unique on a given server, rather than unique for a given user_id. We fix this gap, and at the same time remove unnecessary (and incorrectly racey) logic deleting and recreating the tokens in the appropriate tables. There's still an open mobile app bug causing repeated re-registrations in a loop, but this should fix the fact that the relevant mobile bug causes the server to 500. Follow-up work that may be of value includes: * Removing `ios_app_id`, which may not have much purpose. * Renaming `last_updated` to `data_created`, since that's what it is now. But none of those are critical to solving the actual bug here. Fixes #8841.
25 lines
637 B
Python
25 lines
637 B
Python
# -*- coding: utf-8 -*-
|
|
# Generated by Django 1.11.14 on 2018-10-10 22:52
|
|
from __future__ import unicode_literals
|
|
|
|
from django.db import migrations, models
|
|
|
|
|
|
class Migration(migrations.Migration):
|
|
|
|
dependencies = [
|
|
('zerver', '0189_userprofile_add_some_emojisets'),
|
|
]
|
|
|
|
operations = [
|
|
migrations.AlterField(
|
|
model_name='pushdevicetoken',
|
|
name='token',
|
|
field=models.CharField(db_index=True, max_length=4096),
|
|
),
|
|
migrations.AlterUniqueTogether(
|
|
name='pushdevicetoken',
|
|
unique_together=set([('user', 'kind', 'token')]),
|
|
),
|
|
]
|