don't store local node info in db

This commit is contained in:
j 2019-02-01 11:57:24 +05:30
parent fe9b0618ed
commit 5a8e08b2c2
5 changed files with 21 additions and 28 deletions

View file

@ -486,9 +486,6 @@ class Nodes(Thread):
self._q = Queue() self._q = Queue()
with db.session(): with db.session():
for u in user.models.User.query.filter_by(peered=True): for u in user.models.User.query.filter_by(peered=True):
if 'local' in u.info:
del u.info['local']
u.save()
self.queue('add', u.id) self.queue('add', u.id)
get_peer(u.id) get_peer(u.id)
for u in user.models.User.query.filter_by(queued=True): for u in user.models.User.query.filter_by(queued=True):

View file

@ -95,4 +95,4 @@ FULLTEXT_SUPPORT = fulltext.platform_supported()
if not FULLTEXT_SUPPORT: if not FULLTEXT_SUPPORT:
config['itemKeys'] = [k for k in config['itemKeys'] if k['id'] != 'fulltext'] config['itemKeys'] = [k for k in config['itemKeys'] if k['id'] != 'fulltext']
DB_VERSION = 14 DB_VERSION = 15

View file

@ -15,7 +15,6 @@ shutdown = False
websockets = [] websockets = []
uisockets = [] uisockets = []
peers = {} peers = {}
changelog_size = None changelog_size = None
activity = {} activity = {}

View file

@ -371,7 +371,9 @@ class Update(Thread):
db_version = migrate_12() db_version = migrate_12()
if db_version < 13: if db_version < 13:
db_version = migrate_13() db_version = migrate_13()
settings.server['db_version'] = settings.DB_VERSION if db_version < 15:
db_version = migrate_15()
settings.server['db_version'] = db_version
def run(self): def run(self):
self.status('Checking for updates...') self.status('Checking for updates...')
@ -618,10 +620,14 @@ def migrate_13():
settings.server['revision'] = revision settings.server['revision'] = revision
return 13 return 13
def migrate_14(): def migrate_15():
from user.models import List from user.models import List, User
with db.session(): with db.session():
l = List.get(':Public') l = List.get(':Public')
if l and not len(l.items): if l and not len(l.items):
l.remove() l.remove()
return 14 for u in User.query:
if 'local' in u.info:
del u.info['local']
u.save()
return 15

View file

@ -57,8 +57,7 @@ class User(db.Model):
user = cls(id=id, peered=False, online=False) user = cls(id=id, peered=False, online=False)
user.info = {} user.info = {}
if state.nodes and state.nodes.local and id in state.nodes.local: if state.nodes and state.nodes.local and id in state.nodes.local:
user.info['local'] = state.nodes.local[id] user.info['username'] = state.nodes.localo[id]['username']
user.info['username'] = user.info['local']['username']
user.update_name() user.update_name()
user.save() user.save()
return user return user
@ -84,6 +83,8 @@ class User(db.Model):
j = {} j = {}
if self.info: if self.info:
j.update(self.info) j.update(self.info)
if state.nodes and self.id in state.nodes.local:
j['local'] = state.nodes.local[self.id].copy()
j['id'] = self.id j['id'] = self.id
if self.pending: if self.pending:
j['pending'] = self.pending j['pending'] = self.pending
@ -603,25 +604,15 @@ def update_user_peering(user_id, peered, username=None):
u.update_peering(peered, username) u.update_peering(peered, username)
def remove_local_info(id): def remove_local_info(id):
with db.session(): if state.nodes:
u = User.get(id, for_update=True) trigger_event('status', {
if u and 'local' in u.info: 'id': id,
del u.info['local'] 'online': state.nodes.is_online(id)
u.save() })
u.trigger_status()
def add_local_info(data): def add_local_info(data):
with db.session():
u = User.get(data['id'], for_update=True)
if u:
if u.info['username'] != data['username']:
u.info['username'] = data['username']
u.update_name()
u.info['local'] = data
u.save()
if state.nodes: if state.nodes:
state.nodes.queue('add', u.id) state.nodes.queue('add', data['id'])
def upload(data): def upload(data):
delay = 60 delay = 60