mirror of
https://gitlab.com/famedly/conduit.git
synced 2025-06-27 16:35:59 +00:00
Merge branch 'lazy-loading' into 'rocksdb'
Lazy loading See merge request famedly/conduit!240
This commit is contained in:
commit
2cb51af274
8 changed files with 459 additions and 89 deletions
|
@ -1,5 +1,9 @@
|
||||||
use crate::{database::DatabaseGuard, ConduitResult, Error, Ruma};
|
use crate::{database::DatabaseGuard, ConduitResult, Error, Ruma};
|
||||||
use ruma::api::client::{error::ErrorKind, r0::context::get_context};
|
use ruma::{
|
||||||
|
api::client::{error::ErrorKind, r0::context::get_context},
|
||||||
|
events::EventType,
|
||||||
|
};
|
||||||
|
use std::collections::HashSet;
|
||||||
use std::convert::TryFrom;
|
use std::convert::TryFrom;
|
||||||
|
|
||||||
#[cfg(feature = "conduit_bin")]
|
#[cfg(feature = "conduit_bin")]
|
||||||
|
@ -21,6 +25,7 @@ pub async fn get_context_route(
|
||||||
body: Ruma<get_context::Request<'_>>,
|
body: Ruma<get_context::Request<'_>>,
|
||||||
) -> ConduitResult<get_context::Response> {
|
) -> ConduitResult<get_context::Response> {
|
||||||
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
||||||
|
let sender_device = body.sender_device.as_ref().expect("user is authenticated");
|
||||||
|
|
||||||
if !db.rooms.is_joined(sender_user, &body.room_id)? {
|
if !db.rooms.is_joined(sender_user, &body.room_id)? {
|
||||||
return Err(Error::BadRequest(
|
return Err(Error::BadRequest(
|
||||||
|
@ -29,6 +34,8 @@ pub async fn get_context_route(
|
||||||
));
|
));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
let mut lazy_loaded = HashSet::new();
|
||||||
|
|
||||||
let base_pdu_id = db
|
let base_pdu_id = db
|
||||||
.rooms
|
.rooms
|
||||||
.get_pdu_id(&body.event_id)?
|
.get_pdu_id(&body.event_id)?
|
||||||
|
@ -45,8 +52,18 @@ pub async fn get_context_route(
|
||||||
.ok_or(Error::BadRequest(
|
.ok_or(Error::BadRequest(
|
||||||
ErrorKind::NotFound,
|
ErrorKind::NotFound,
|
||||||
"Base event not found.",
|
"Base event not found.",
|
||||||
))?
|
))?;
|
||||||
.to_room_event();
|
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
&base_event.sender,
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(base_event.sender.clone());
|
||||||
|
}
|
||||||
|
|
||||||
|
let base_event = base_event.to_room_event();
|
||||||
|
|
||||||
let events_before: Vec<_> = db
|
let events_before: Vec<_> = db
|
||||||
.rooms
|
.rooms
|
||||||
|
@ -60,6 +77,17 @@ pub async fn get_context_route(
|
||||||
.filter_map(|r| r.ok()) // Remove buggy events
|
.filter_map(|r| r.ok()) // Remove buggy events
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
|
for (_, event) in &events_before {
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
&event.sender,
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(event.sender.clone());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
let start_token = events_before
|
let start_token = events_before
|
||||||
.last()
|
.last()
|
||||||
.and_then(|(pdu_id, _)| db.rooms.pdu_count(pdu_id).ok())
|
.and_then(|(pdu_id, _)| db.rooms.pdu_count(pdu_id).ok())
|
||||||
|
@ -82,6 +110,17 @@ pub async fn get_context_route(
|
||||||
.filter_map(|r| r.ok()) // Remove buggy events
|
.filter_map(|r| r.ok()) // Remove buggy events
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
|
for (_, event) in &events_after {
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
&event.sender,
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(event.sender.clone());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
let end_token = events_after
|
let end_token = events_after
|
||||||
.last()
|
.last()
|
||||||
.and_then(|(pdu_id, _)| db.rooms.pdu_count(pdu_id).ok())
|
.and_then(|(pdu_id, _)| db.rooms.pdu_count(pdu_id).ok())
|
||||||
|
@ -98,12 +137,15 @@ pub async fn get_context_route(
|
||||||
resp.events_before = events_before;
|
resp.events_before = events_before;
|
||||||
resp.event = Some(base_event);
|
resp.event = Some(base_event);
|
||||||
resp.events_after = events_after;
|
resp.events_after = events_after;
|
||||||
resp.state = db // TODO: State at event
|
resp.state = Vec::new();
|
||||||
.rooms
|
for ll_id in &lazy_loaded {
|
||||||
.room_state_full(&body.room_id)?
|
if let Some(member_event) =
|
||||||
.values()
|
db.rooms
|
||||||
.map(|pdu| pdu.to_state_event())
|
.room_state_get(&body.room_id, &EventType::RoomMember, ll_id.as_str())?
|
||||||
.collect();
|
{
|
||||||
|
resp.state.push(member_event.to_state_event());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
Ok(resp.into())
|
Ok(resp.into())
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,32 +1,47 @@
|
||||||
use crate::{utils, ConduitResult};
|
use crate::{database::DatabaseGuard, ConduitResult, Error, Ruma};
|
||||||
use ruma::api::client::r0::filter::{self, create_filter, get_filter};
|
use ruma::api::client::{
|
||||||
|
error::ErrorKind,
|
||||||
|
r0::filter::{create_filter, get_filter},
|
||||||
|
};
|
||||||
|
|
||||||
#[cfg(feature = "conduit_bin")]
|
#[cfg(feature = "conduit_bin")]
|
||||||
use rocket::{get, post};
|
use rocket::{get, post};
|
||||||
|
|
||||||
/// # `GET /_matrix/client/r0/user/{userId}/filter/{filterId}`
|
/// # `GET /_matrix/client/r0/user/{userId}/filter/{filterId}`
|
||||||
///
|
///
|
||||||
/// TODO: Loads a filter that was previously created.
|
/// Loads a filter that was previously created.
|
||||||
#[cfg_attr(feature = "conduit_bin", get("/_matrix/client/r0/user/<_>/filter/<_>"))]
|
///
|
||||||
#[tracing::instrument]
|
/// - A user can only access their own filters
|
||||||
pub async fn get_filter_route() -> ConduitResult<get_filter::Response> {
|
#[cfg_attr(
|
||||||
// TODO
|
feature = "conduit_bin",
|
||||||
Ok(get_filter::Response::new(filter::IncomingFilterDefinition {
|
get("/_matrix/client/r0/user/<_>/filter/<_>", data = "<body>")
|
||||||
event_fields: None,
|
)]
|
||||||
event_format: filter::EventFormat::default(),
|
#[tracing::instrument(skip(db, body))]
|
||||||
account_data: filter::IncomingFilter::default(),
|
pub async fn get_filter_route(
|
||||||
room: filter::IncomingRoomFilter::default(),
|
db: DatabaseGuard,
|
||||||
presence: filter::IncomingFilter::default(),
|
body: Ruma<get_filter::Request<'_>>,
|
||||||
})
|
) -> ConduitResult<get_filter::Response> {
|
||||||
.into())
|
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
||||||
|
let filter = match db.users.get_filter(sender_user, &body.filter_id)? {
|
||||||
|
Some(filter) => filter,
|
||||||
|
None => return Err(Error::BadRequest(ErrorKind::NotFound, "Filter not found.")),
|
||||||
|
};
|
||||||
|
|
||||||
|
Ok(get_filter::Response::new(filter).into())
|
||||||
}
|
}
|
||||||
|
|
||||||
/// # `PUT /_matrix/client/r0/user/{userId}/filter`
|
/// # `PUT /_matrix/client/r0/user/{userId}/filter`
|
||||||
///
|
///
|
||||||
/// TODO: Creates a new filter to be used by other endpoints.
|
/// Creates a new filter to be used by other endpoints.
|
||||||
#[cfg_attr(feature = "conduit_bin", post("/_matrix/client/r0/user/<_>/filter"))]
|
#[cfg_attr(
|
||||||
#[tracing::instrument]
|
feature = "conduit_bin",
|
||||||
pub async fn create_filter_route() -> ConduitResult<create_filter::Response> {
|
post("/_matrix/client/r0/user/<_>/filter", data = "<body>")
|
||||||
// TODO
|
)]
|
||||||
Ok(create_filter::Response::new(utils::random_string(10)).into())
|
#[tracing::instrument(skip(db, body))]
|
||||||
|
pub async fn create_filter_route(
|
||||||
|
db: DatabaseGuard,
|
||||||
|
body: Ruma<create_filter::Request<'_>>,
|
||||||
|
) -> ConduitResult<create_filter::Response> {
|
||||||
|
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
||||||
|
Ok(create_filter::Response::new(db.users.create_filter(sender_user, &body.filter)?).into())
|
||||||
}
|
}
|
||||||
|
|
|
@ -6,7 +6,11 @@ use ruma::{
|
||||||
},
|
},
|
||||||
events::EventType,
|
events::EventType,
|
||||||
};
|
};
|
||||||
use std::{collections::BTreeMap, convert::TryInto, sync::Arc};
|
use std::{
|
||||||
|
collections::{BTreeMap, HashSet},
|
||||||
|
convert::TryInto,
|
||||||
|
sync::Arc,
|
||||||
|
};
|
||||||
|
|
||||||
#[cfg(feature = "conduit_bin")]
|
#[cfg(feature = "conduit_bin")]
|
||||||
use rocket::{get, put};
|
use rocket::{get, put};
|
||||||
|
@ -117,6 +121,7 @@ pub async fn get_message_events_route(
|
||||||
body: Ruma<get_message_events::Request<'_>>,
|
body: Ruma<get_message_events::Request<'_>>,
|
||||||
) -> ConduitResult<get_message_events::Response> {
|
) -> ConduitResult<get_message_events::Response> {
|
||||||
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
let sender_user = body.sender_user.as_ref().expect("user is authenticated");
|
||||||
|
let sender_device = body.sender_device.as_ref().expect("user is authenticated");
|
||||||
|
|
||||||
if !db.rooms.is_joined(sender_user, &body.room_id)? {
|
if !db.rooms.is_joined(sender_user, &body.room_id)? {
|
||||||
return Err(Error::BadRequest(
|
return Err(Error::BadRequest(
|
||||||
|
@ -133,9 +138,18 @@ pub async fn get_message_events_route(
|
||||||
|
|
||||||
let to = body.to.as_ref().map(|t| t.parse());
|
let to = body.to.as_ref().map(|t| t.parse());
|
||||||
|
|
||||||
|
db.rooms
|
||||||
|
.lazy_load_confirm_delivery(&sender_user, &sender_device, &body.room_id, from)?;
|
||||||
|
|
||||||
// Use limit or else 10
|
// Use limit or else 10
|
||||||
let limit = body.limit.try_into().map_or(10_usize, |l: u32| l as usize);
|
let limit = body.limit.try_into().map_or(10_usize, |l: u32| l as usize);
|
||||||
|
|
||||||
|
let next_token;
|
||||||
|
|
||||||
|
let mut resp = get_message_events::Response::new();
|
||||||
|
|
||||||
|
let mut lazy_loaded = HashSet::new();
|
||||||
|
|
||||||
match body.dir {
|
match body.dir {
|
||||||
get_message_events::Direction::Forward => {
|
get_message_events::Direction::Forward => {
|
||||||
let events_after: Vec<_> = db
|
let events_after: Vec<_> = db
|
||||||
|
@ -152,7 +166,18 @@ pub async fn get_message_events_route(
|
||||||
.take_while(|&(k, _)| Some(Ok(k)) != to) // Stop at `to`
|
.take_while(|&(k, _)| Some(Ok(k)) != to) // Stop at `to`
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
let end_token = events_after.last().map(|(count, _)| count.to_string());
|
for (_, event) in &events_after {
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
&event.sender,
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(event.sender.clone());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
next_token = events_after.last().map(|(count, _)| count).copied();
|
||||||
|
|
||||||
let events_after: Vec<_> = events_after
|
let events_after: Vec<_> = events_after
|
||||||
.into_iter()
|
.into_iter()
|
||||||
|
@ -161,11 +186,8 @@ pub async fn get_message_events_route(
|
||||||
|
|
||||||
let mut resp = get_message_events::Response::new();
|
let mut resp = get_message_events::Response::new();
|
||||||
resp.start = Some(body.from.to_owned());
|
resp.start = Some(body.from.to_owned());
|
||||||
resp.end = end_token;
|
resp.end = next_token.map(|count| count.to_string());
|
||||||
resp.chunk = events_after;
|
resp.chunk = events_after;
|
||||||
resp.state = Vec::new();
|
|
||||||
|
|
||||||
Ok(resp.into())
|
|
||||||
}
|
}
|
||||||
get_message_events::Direction::Backward => {
|
get_message_events::Direction::Backward => {
|
||||||
let events_before: Vec<_> = db
|
let events_before: Vec<_> = db
|
||||||
|
@ -182,20 +204,49 @@ pub async fn get_message_events_route(
|
||||||
.take_while(|&(k, _)| Some(Ok(k)) != to) // Stop at `to`
|
.take_while(|&(k, _)| Some(Ok(k)) != to) // Stop at `to`
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
let start_token = events_before.last().map(|(count, _)| count.to_string());
|
for (_, event) in &events_before {
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
&event.sender,
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(event.sender.clone());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
next_token = events_before.last().map(|(count, _)| count).copied();
|
||||||
|
|
||||||
let events_before: Vec<_> = events_before
|
let events_before: Vec<_> = events_before
|
||||||
.into_iter()
|
.into_iter()
|
||||||
.map(|(_, pdu)| pdu.to_room_event())
|
.map(|(_, pdu)| pdu.to_room_event())
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
let mut resp = get_message_events::Response::new();
|
|
||||||
resp.start = Some(body.from.to_owned());
|
resp.start = Some(body.from.to_owned());
|
||||||
resp.end = start_token;
|
resp.end = next_token.map(|count| count.to_string());
|
||||||
resp.chunk = events_before;
|
resp.chunk = events_before;
|
||||||
resp.state = Vec::new();
|
|
||||||
|
|
||||||
Ok(resp.into())
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
resp.state = Vec::new();
|
||||||
|
for ll_id in &lazy_loaded {
|
||||||
|
if let Some(member_event) =
|
||||||
|
db.rooms
|
||||||
|
.room_state_get(&body.room_id, &EventType::RoomMember, ll_id.as_str())?
|
||||||
|
{
|
||||||
|
resp.state.push(member_event.to_state_event());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if let Some(next_token) = next_token {
|
||||||
|
db.rooms.lazy_load_mark_sent(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&body.room_id,
|
||||||
|
lazy_loaded,
|
||||||
|
next_token,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
Ok(resp.into())
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,6 +1,10 @@
|
||||||
use crate::{database::DatabaseGuard, ConduitResult, Database, Error, Result, Ruma, RumaResponse};
|
use crate::{database::DatabaseGuard, ConduitResult, Database, Error, Result, Ruma, RumaResponse};
|
||||||
use ruma::{
|
use ruma::{
|
||||||
api::client::r0::{sync::sync_events, uiaa::UiaaResponse},
|
api::client::r0::{
|
||||||
|
filter::{IncomingFilterDefinition, LazyLoadOptions},
|
||||||
|
sync::sync_events,
|
||||||
|
uiaa::UiaaResponse,
|
||||||
|
},
|
||||||
events::{
|
events::{
|
||||||
room::member::{MembershipState, RoomMemberEventContent},
|
room::member::{MembershipState, RoomMemberEventContent},
|
||||||
AnySyncEphemeralRoomEvent, EventType,
|
AnySyncEphemeralRoomEvent, EventType,
|
||||||
|
@ -36,13 +40,15 @@ use rocket::{get, tokio};
|
||||||
/// Calling this endpoint with a `since` parameter from a previous `next_batch` returns:
|
/// Calling this endpoint with a `since` parameter from a previous `next_batch` returns:
|
||||||
/// For joined rooms:
|
/// For joined rooms:
|
||||||
/// - Some of the most recent events of each timeline that happened after since
|
/// - Some of the most recent events of each timeline that happened after since
|
||||||
/// - If user joined the room after since: All state events and device list updates in that room
|
/// - If user joined the room after since: All state events (unless lazy loading is activated) and
|
||||||
|
/// all device list updates in that room
|
||||||
/// - If the user was already in the room: A list of all events that are in the state now, but were
|
/// - If the user was already in the room: A list of all events that are in the state now, but were
|
||||||
/// not in the state at `since`
|
/// not in the state at `since`
|
||||||
/// - If the state we send contains a member event: Joined and invited member counts, heroes
|
/// - If the state we send contains a member event: Joined and invited member counts, heroes
|
||||||
/// - Device list updates that happened after `since`
|
/// - Device list updates that happened after `since`
|
||||||
/// - If there are events in the timeline we send or the user send updated his read mark: Notification counts
|
/// - If there are events in the timeline we send or the user send updated his read mark: Notification counts
|
||||||
/// - EDUs that are active now (read receipts, typing updates, presence)
|
/// - EDUs that are active now (read receipts, typing updates, presence)
|
||||||
|
/// - TODO: Allow multiple sync streams to support Pantalaimon
|
||||||
///
|
///
|
||||||
/// For invited rooms:
|
/// For invited rooms:
|
||||||
/// - If the user was invited after `since`: A subset of the state of the room at the point of the invite
|
/// - If the user was invited after `since`: A subset of the state of the room at the point of the invite
|
||||||
|
@ -77,34 +83,32 @@ pub async fn sync_events_route(
|
||||||
Entry::Vacant(v) => {
|
Entry::Vacant(v) => {
|
||||||
let (tx, rx) = tokio::sync::watch::channel(None);
|
let (tx, rx) = tokio::sync::watch::channel(None);
|
||||||
|
|
||||||
|
v.insert((body.since.clone(), rx.clone()));
|
||||||
|
|
||||||
tokio::spawn(sync_helper_wrapper(
|
tokio::spawn(sync_helper_wrapper(
|
||||||
Arc::clone(&arc_db),
|
Arc::clone(&arc_db),
|
||||||
sender_user.clone(),
|
sender_user.clone(),
|
||||||
sender_device.clone(),
|
sender_device.clone(),
|
||||||
body.since.clone(),
|
body,
|
||||||
body.full_state,
|
|
||||||
body.timeout,
|
|
||||||
tx,
|
tx,
|
||||||
));
|
));
|
||||||
|
|
||||||
v.insert((body.since.clone(), rx)).1.clone()
|
rx
|
||||||
}
|
}
|
||||||
Entry::Occupied(mut o) => {
|
Entry::Occupied(mut o) => {
|
||||||
if o.get().0 != body.since {
|
if o.get().0 != body.since {
|
||||||
let (tx, rx) = tokio::sync::watch::channel(None);
|
let (tx, rx) = tokio::sync::watch::channel(None);
|
||||||
|
|
||||||
|
o.insert((body.since.clone(), rx.clone()));
|
||||||
|
|
||||||
tokio::spawn(sync_helper_wrapper(
|
tokio::spawn(sync_helper_wrapper(
|
||||||
Arc::clone(&arc_db),
|
Arc::clone(&arc_db),
|
||||||
sender_user.clone(),
|
sender_user.clone(),
|
||||||
sender_device.clone(),
|
sender_device.clone(),
|
||||||
body.since.clone(),
|
body,
|
||||||
body.full_state,
|
|
||||||
body.timeout,
|
|
||||||
tx,
|
tx,
|
||||||
));
|
));
|
||||||
|
|
||||||
o.insert((body.since.clone(), rx.clone()));
|
|
||||||
|
|
||||||
rx
|
rx
|
||||||
} else {
|
} else {
|
||||||
o.get().1.clone()
|
o.get().1.clone()
|
||||||
|
@ -135,18 +139,16 @@ async fn sync_helper_wrapper(
|
||||||
db: Arc<DatabaseGuard>,
|
db: Arc<DatabaseGuard>,
|
||||||
sender_user: Box<UserId>,
|
sender_user: Box<UserId>,
|
||||||
sender_device: Box<DeviceId>,
|
sender_device: Box<DeviceId>,
|
||||||
since: Option<String>,
|
body: sync_events::IncomingRequest,
|
||||||
full_state: bool,
|
|
||||||
timeout: Option<Duration>,
|
|
||||||
tx: Sender<Option<ConduitResult<sync_events::Response>>>,
|
tx: Sender<Option<ConduitResult<sync_events::Response>>>,
|
||||||
) {
|
) {
|
||||||
|
let since = body.since.clone();
|
||||||
|
|
||||||
let r = sync_helper(
|
let r = sync_helper(
|
||||||
Arc::clone(&db),
|
Arc::clone(&db),
|
||||||
sender_user.clone(),
|
sender_user.clone(),
|
||||||
sender_device.clone(),
|
sender_device.clone(),
|
||||||
since.clone(),
|
body,
|
||||||
full_state,
|
|
||||||
timeout,
|
|
||||||
)
|
)
|
||||||
.await;
|
.await;
|
||||||
|
|
||||||
|
@ -179,9 +181,7 @@ async fn sync_helper(
|
||||||
db: Arc<DatabaseGuard>,
|
db: Arc<DatabaseGuard>,
|
||||||
sender_user: Box<UserId>,
|
sender_user: Box<UserId>,
|
||||||
sender_device: Box<DeviceId>,
|
sender_device: Box<DeviceId>,
|
||||||
since: Option<String>,
|
body: sync_events::IncomingRequest,
|
||||||
full_state: bool,
|
|
||||||
timeout: Option<Duration>,
|
|
||||||
// bool = caching allowed
|
// bool = caching allowed
|
||||||
) -> Result<(sync_events::Response, bool), Error> {
|
) -> Result<(sync_events::Response, bool), Error> {
|
||||||
// TODO: match body.set_presence {
|
// TODO: match body.set_presence {
|
||||||
|
@ -193,8 +193,26 @@ async fn sync_helper(
|
||||||
let next_batch = db.globals.current_count()?;
|
let next_batch = db.globals.current_count()?;
|
||||||
let next_batch_string = next_batch.to_string();
|
let next_batch_string = next_batch.to_string();
|
||||||
|
|
||||||
|
// Load filter
|
||||||
|
let filter = match body.filter {
|
||||||
|
None => IncomingFilterDefinition::default(),
|
||||||
|
Some(sync_events::IncomingFilter::FilterDefinition(filter)) => filter,
|
||||||
|
Some(sync_events::IncomingFilter::FilterId(filter_id)) => db
|
||||||
|
.users
|
||||||
|
.get_filter(&sender_user, &filter_id)?
|
||||||
|
.unwrap_or_default(),
|
||||||
|
};
|
||||||
|
|
||||||
|
let (lazy_load_enabled, lazy_load_send_redundant) = match filter.room.state.lazy_load_options {
|
||||||
|
LazyLoadOptions::Enabled {
|
||||||
|
include_redundant_members: redundant,
|
||||||
|
} => (true, redundant),
|
||||||
|
_ => (false, false),
|
||||||
|
};
|
||||||
|
|
||||||
let mut joined_rooms = BTreeMap::new();
|
let mut joined_rooms = BTreeMap::new();
|
||||||
let since = since
|
let since = body
|
||||||
|
.since
|
||||||
.clone()
|
.clone()
|
||||||
.and_then(|string| string.parse().ok())
|
.and_then(|string| string.parse().ok())
|
||||||
.unwrap_or(0);
|
.unwrap_or(0);
|
||||||
|
@ -264,6 +282,14 @@ async fn sync_helper(
|
||||||
// limited unless there are events in non_timeline_pdus
|
// limited unless there are events in non_timeline_pdus
|
||||||
let limited = non_timeline_pdus.next().is_some();
|
let limited = non_timeline_pdus.next().is_some();
|
||||||
|
|
||||||
|
let mut timeline_users = HashSet::new();
|
||||||
|
for (_, event) in &timeline_pdus {
|
||||||
|
timeline_users.insert(event.sender.as_str().to_owned());
|
||||||
|
}
|
||||||
|
|
||||||
|
db.rooms
|
||||||
|
.lazy_load_confirm_delivery(&sender_user, &sender_device, &room_id, since)?;
|
||||||
|
|
||||||
// Database queries:
|
// Database queries:
|
||||||
|
|
||||||
let current_shortstatehash = db
|
let current_shortstatehash = db
|
||||||
|
@ -344,14 +370,58 @@ async fn sync_helper(
|
||||||
state_events,
|
state_events,
|
||||||
) = if since_shortstatehash.is_none() {
|
) = if since_shortstatehash.is_none() {
|
||||||
// Probably since = 0, we will do an initial sync
|
// Probably since = 0, we will do an initial sync
|
||||||
|
|
||||||
let (joined_member_count, invited_member_count, heroes) = calculate_counts()?;
|
let (joined_member_count, invited_member_count, heroes) = calculate_counts()?;
|
||||||
|
|
||||||
let current_state_ids = db.rooms.state_full_ids(current_shortstatehash)?;
|
let current_state_ids = db.rooms.state_full_ids(current_shortstatehash)?;
|
||||||
let state_events: Vec<_> = current_state_ids
|
|
||||||
.iter()
|
let mut state_events = Vec::new();
|
||||||
.map(|(_, id)| db.rooms.get_pdu(id))
|
let mut lazy_loaded = HashSet::new();
|
||||||
.filter_map(|r| r.ok().flatten())
|
|
||||||
.collect();
|
for (shortstatekey, id) in current_state_ids {
|
||||||
|
let (event_type, state_key) = db.rooms.get_statekey_from_short(shortstatekey)?;
|
||||||
|
|
||||||
|
if event_type != EventType::RoomMember {
|
||||||
|
let pdu = match db.rooms.get_pdu(&id)? {
|
||||||
|
Some(pdu) => pdu,
|
||||||
|
None => {
|
||||||
|
error!("Pdu in state not found: {}", id);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
};
|
||||||
|
state_events.push(pdu);
|
||||||
|
} else if !lazy_load_enabled
|
||||||
|
|| body.full_state
|
||||||
|
|| timeline_users.contains(&state_key)
|
||||||
|
{
|
||||||
|
let pdu = match db.rooms.get_pdu(&id)? {
|
||||||
|
Some(pdu) => pdu,
|
||||||
|
None => {
|
||||||
|
error!("Pdu in state not found: {}", id);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
};
|
||||||
|
lazy_loaded.insert(
|
||||||
|
UserId::parse(state_key.as_ref())
|
||||||
|
.expect("they are in timeline_users, so they should be correct"),
|
||||||
|
);
|
||||||
|
state_events.push(pdu);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Reset lazy loading because this is an initial sync
|
||||||
|
db.rooms
|
||||||
|
.lazy_load_reset(&sender_user, &sender_device, &room_id)?;
|
||||||
|
|
||||||
|
// The state_events above should contain all timeline_users, let's mark them as lazy
|
||||||
|
// loaded.
|
||||||
|
db.rooms.lazy_load_mark_sent(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&room_id,
|
||||||
|
lazy_loaded,
|
||||||
|
next_batch,
|
||||||
|
);
|
||||||
|
|
||||||
(
|
(
|
||||||
heroes,
|
heroes,
|
||||||
|
@ -387,20 +457,67 @@ async fn sync_helper(
|
||||||
|
|
||||||
let since_state_ids = db.rooms.state_full_ids(since_shortstatehash)?;
|
let since_state_ids = db.rooms.state_full_ids(since_shortstatehash)?;
|
||||||
|
|
||||||
let state_events = if joined_since_last_sync {
|
let mut state_events = Vec::new();
|
||||||
current_state_ids
|
let mut lazy_loaded = HashSet::new();
|
||||||
.iter()
|
|
||||||
.map(|(_, id)| db.rooms.get_pdu(id))
|
for (key, id) in current_state_ids {
|
||||||
.filter_map(|r| r.ok().flatten())
|
if body.full_state || since_state_ids.get(&key) != Some(&id) {
|
||||||
.collect::<Vec<_>>()
|
let pdu = match db.rooms.get_pdu(&id)? {
|
||||||
} else {
|
Some(pdu) => pdu,
|
||||||
current_state_ids
|
None => {
|
||||||
.iter()
|
error!("Pdu in state not found: {}", id);
|
||||||
.filter(|(key, id)| since_state_ids.get(key) != Some(id))
|
continue;
|
||||||
.map(|(_, id)| db.rooms.get_pdu(id))
|
}
|
||||||
.filter_map(|r| r.ok().flatten())
|
};
|
||||||
.collect()
|
|
||||||
};
|
if pdu.kind == EventType::RoomMember {
|
||||||
|
match UserId::parse(
|
||||||
|
pdu.state_key
|
||||||
|
.as_ref()
|
||||||
|
.expect("State event has state key")
|
||||||
|
.clone(),
|
||||||
|
) {
|
||||||
|
Ok(state_key_userid) => {
|
||||||
|
lazy_loaded.insert(state_key_userid);
|
||||||
|
}
|
||||||
|
Err(e) => error!("Invalid state key for member event: {}", e),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
state_events.push(pdu);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
for (_, event) in &timeline_pdus {
|
||||||
|
if lazy_loaded.contains(&event.sender) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if !db.rooms.lazy_load_was_sent_before(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&room_id,
|
||||||
|
&event.sender,
|
||||||
|
)? || lazy_load_send_redundant
|
||||||
|
{
|
||||||
|
if let Some(member_event) = db.rooms.room_state_get(
|
||||||
|
&room_id,
|
||||||
|
&EventType::RoomMember,
|
||||||
|
event.sender.as_str(),
|
||||||
|
)? {
|
||||||
|
lazy_loaded.insert(event.sender.clone());
|
||||||
|
state_events.push(member_event);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
db.rooms.lazy_load_mark_sent(
|
||||||
|
&sender_user,
|
||||||
|
&sender_device,
|
||||||
|
&room_id,
|
||||||
|
lazy_loaded,
|
||||||
|
next_batch,
|
||||||
|
);
|
||||||
|
|
||||||
let encrypted_room = db
|
let encrypted_room = db
|
||||||
.rooms
|
.rooms
|
||||||
|
@ -765,7 +882,7 @@ async fn sync_helper(
|
||||||
};
|
};
|
||||||
|
|
||||||
// TODO: Retry the endpoint instead of returning (waiting for #118)
|
// TODO: Retry the endpoint instead of returning (waiting for #118)
|
||||||
if !full_state
|
if !body.full_state
|
||||||
&& response.rooms.is_empty()
|
&& response.rooms.is_empty()
|
||||||
&& response.presence.is_empty()
|
&& response.presence.is_empty()
|
||||||
&& response.account_data.is_empty()
|
&& response.account_data.is_empty()
|
||||||
|
@ -774,7 +891,7 @@ async fn sync_helper(
|
||||||
{
|
{
|
||||||
// Hang a few seconds so requests are not spammed
|
// Hang a few seconds so requests are not spammed
|
||||||
// Stop hanging if new info arrives
|
// Stop hanging if new info arrives
|
||||||
let mut duration = timeout.unwrap_or_default();
|
let mut duration = body.timeout.unwrap_or_default();
|
||||||
if duration.as_secs() > 30 {
|
if duration.as_secs() > 30 {
|
||||||
duration = Duration::from_secs(30);
|
duration = Duration::from_secs(30);
|
||||||
}
|
}
|
||||||
|
|
|
@ -249,6 +249,7 @@ impl Database {
|
||||||
userid_masterkeyid: builder.open_tree("userid_masterkeyid")?,
|
userid_masterkeyid: builder.open_tree("userid_masterkeyid")?,
|
||||||
userid_selfsigningkeyid: builder.open_tree("userid_selfsigningkeyid")?,
|
userid_selfsigningkeyid: builder.open_tree("userid_selfsigningkeyid")?,
|
||||||
userid_usersigningkeyid: builder.open_tree("userid_usersigningkeyid")?,
|
userid_usersigningkeyid: builder.open_tree("userid_usersigningkeyid")?,
|
||||||
|
userfilterid_filter: builder.open_tree("userfilterid_filter")?,
|
||||||
todeviceid_events: builder.open_tree("todeviceid_events")?,
|
todeviceid_events: builder.open_tree("todeviceid_events")?,
|
||||||
},
|
},
|
||||||
uiaa: uiaa::Uiaa {
|
uiaa: uiaa::Uiaa {
|
||||||
|
@ -289,6 +290,8 @@ impl Database {
|
||||||
userroomid_leftstate: builder.open_tree("userroomid_leftstate")?,
|
userroomid_leftstate: builder.open_tree("userroomid_leftstate")?,
|
||||||
roomuserid_leftcount: builder.open_tree("roomuserid_leftcount")?,
|
roomuserid_leftcount: builder.open_tree("roomuserid_leftcount")?,
|
||||||
|
|
||||||
|
lazyloadedids: builder.open_tree("lazyloadedids")?,
|
||||||
|
|
||||||
userroomid_notificationcount: builder.open_tree("userroomid_notificationcount")?,
|
userroomid_notificationcount: builder.open_tree("userroomid_notificationcount")?,
|
||||||
userroomid_highlightcount: builder.open_tree("userroomid_highlightcount")?,
|
userroomid_highlightcount: builder.open_tree("userroomid_highlightcount")?,
|
||||||
|
|
||||||
|
@ -324,6 +327,7 @@ impl Database {
|
||||||
statekeyshort_cache: Mutex::new(LruCache::new(1_000_000)),
|
statekeyshort_cache: Mutex::new(LruCache::new(1_000_000)),
|
||||||
our_real_users_cache: RwLock::new(HashMap::new()),
|
our_real_users_cache: RwLock::new(HashMap::new()),
|
||||||
appservice_in_room_cache: RwLock::new(HashMap::new()),
|
appservice_in_room_cache: RwLock::new(HashMap::new()),
|
||||||
|
lazy_load_waiting: Mutex::new(HashMap::new()),
|
||||||
stateinfo_cache: Mutex::new(LruCache::new(1000)),
|
stateinfo_cache: Mutex::new(LruCache::new(1000)),
|
||||||
},
|
},
|
||||||
account_data: account_data::AccountData {
|
account_data: account_data::AccountData {
|
||||||
|
|
|
@ -1,10 +1,11 @@
|
||||||
use super::super::Config;
|
use super::super::Config;
|
||||||
|
use super::{DatabaseEngine, Tree};
|
||||||
use crate::{utils, Result};
|
use crate::{utils, Result};
|
||||||
use std::{future::Future, pin::Pin, sync::Arc};
|
|
||||||
use std::{
|
use std::{
|
||||||
collections::{hash_map, HashMap},
|
collections::{hash_map, HashMap},
|
||||||
sync::RwLock};
|
sync::RwLock,
|
||||||
use super::{DatabaseEngine, Tree};
|
};
|
||||||
|
use std::{future::Future, pin::Pin, sync::Arc};
|
||||||
use tokio::sync::watch;
|
use tokio::sync::watch;
|
||||||
|
|
||||||
pub struct Engine {
|
pub struct Engine {
|
||||||
|
|
|
@ -28,7 +28,7 @@ use ruma::{
|
||||||
push::{Action, Ruleset, Tweak},
|
push::{Action, Ruleset, Tweak},
|
||||||
serde::{CanonicalJsonObject, CanonicalJsonValue, Raw},
|
serde::{CanonicalJsonObject, CanonicalJsonValue, Raw},
|
||||||
state_res::{self, RoomVersion, StateMap},
|
state_res::{self, RoomVersion, StateMap},
|
||||||
uint, EventId, RoomAliasId, RoomId, RoomVersionId, ServerName, UserId,
|
uint, DeviceId, EventId, RoomAliasId, RoomId, RoomVersionId, ServerName, UserId,
|
||||||
};
|
};
|
||||||
use serde::Deserialize;
|
use serde::Deserialize;
|
||||||
use serde_json::value::to_raw_value;
|
use serde_json::value::to_raw_value;
|
||||||
|
@ -79,6 +79,8 @@ pub struct Rooms {
|
||||||
pub(super) userroomid_leftstate: Arc<dyn Tree>,
|
pub(super) userroomid_leftstate: Arc<dyn Tree>,
|
||||||
pub(super) roomuserid_leftcount: Arc<dyn Tree>,
|
pub(super) roomuserid_leftcount: Arc<dyn Tree>,
|
||||||
|
|
||||||
|
pub(super) lazyloadedids: Arc<dyn Tree>, // LazyLoadedIds = UserId + DeviceId + RoomId + LazyLoadedUserId
|
||||||
|
|
||||||
pub(super) userroomid_notificationcount: Arc<dyn Tree>, // NotifyCount = u64
|
pub(super) userroomid_notificationcount: Arc<dyn Tree>, // NotifyCount = u64
|
||||||
pub(super) userroomid_highlightcount: Arc<dyn Tree>, // HightlightCount = u64
|
pub(super) userroomid_highlightcount: Arc<dyn Tree>, // HightlightCount = u64
|
||||||
|
|
||||||
|
@ -117,6 +119,8 @@ pub struct Rooms {
|
||||||
pub(super) shortstatekey_cache: Mutex<LruCache<u64, (EventType, String)>>,
|
pub(super) shortstatekey_cache: Mutex<LruCache<u64, (EventType, String)>>,
|
||||||
pub(super) our_real_users_cache: RwLock<HashMap<Box<RoomId>, Arc<HashSet<Box<UserId>>>>>,
|
pub(super) our_real_users_cache: RwLock<HashMap<Box<RoomId>, Arc<HashSet<Box<UserId>>>>>,
|
||||||
pub(super) appservice_in_room_cache: RwLock<HashMap<Box<RoomId>, HashMap<String, bool>>>,
|
pub(super) appservice_in_room_cache: RwLock<HashMap<Box<RoomId>, HashMap<String, bool>>>,
|
||||||
|
pub(super) lazy_load_waiting:
|
||||||
|
Mutex<HashMap<(Box<UserId>, Box<DeviceId>, Box<RoomId>, u64), HashSet<Box<UserId>>>>,
|
||||||
pub(super) stateinfo_cache: Mutex<
|
pub(super) stateinfo_cache: Mutex<
|
||||||
LruCache<
|
LruCache<
|
||||||
u64,
|
u64,
|
||||||
|
@ -3453,4 +3457,94 @@ impl Rooms {
|
||||||
|
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn lazy_load_was_sent_before(
|
||||||
|
&self,
|
||||||
|
user_id: &UserId,
|
||||||
|
device_id: &DeviceId,
|
||||||
|
room_id: &RoomId,
|
||||||
|
ll_user: &UserId,
|
||||||
|
) -> Result<bool> {
|
||||||
|
let mut key = user_id.as_bytes().to_vec();
|
||||||
|
key.push(0xff);
|
||||||
|
key.extend_from_slice(&device_id.as_bytes());
|
||||||
|
key.push(0xff);
|
||||||
|
key.extend_from_slice(&room_id.as_bytes());
|
||||||
|
key.push(0xff);
|
||||||
|
key.extend_from_slice(&ll_user.as_bytes());
|
||||||
|
Ok(self.lazyloadedids.get(&key)?.is_some())
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn lazy_load_mark_sent(
|
||||||
|
&self,
|
||||||
|
user_id: &UserId,
|
||||||
|
device_id: &DeviceId,
|
||||||
|
room_id: &RoomId,
|
||||||
|
lazy_load: HashSet<Box<UserId>>,
|
||||||
|
count: u64,
|
||||||
|
) {
|
||||||
|
self.lazy_load_waiting.lock().unwrap().insert(
|
||||||
|
(
|
||||||
|
user_id.to_owned(),
|
||||||
|
device_id.to_owned(),
|
||||||
|
room_id.to_owned(),
|
||||||
|
count,
|
||||||
|
),
|
||||||
|
lazy_load,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn lazy_load_confirm_delivery(
|
||||||
|
&self,
|
||||||
|
user_id: &UserId,
|
||||||
|
device_id: &DeviceId,
|
||||||
|
room_id: &RoomId,
|
||||||
|
since: u64,
|
||||||
|
) -> Result<()> {
|
||||||
|
if let Some(user_ids) = self.lazy_load_waiting.lock().unwrap().remove(&(
|
||||||
|
user_id.to_owned(),
|
||||||
|
device_id.to_owned(),
|
||||||
|
room_id.to_owned(),
|
||||||
|
since,
|
||||||
|
)) {
|
||||||
|
let mut prefix = user_id.as_bytes().to_vec();
|
||||||
|
prefix.push(0xff);
|
||||||
|
prefix.extend_from_slice(&device_id.as_bytes());
|
||||||
|
prefix.push(0xff);
|
||||||
|
prefix.extend_from_slice(&room_id.as_bytes());
|
||||||
|
prefix.push(0xff);
|
||||||
|
|
||||||
|
for ll_id in user_ids {
|
||||||
|
let mut key = prefix.clone();
|
||||||
|
key.extend_from_slice(&ll_id.as_bytes());
|
||||||
|
self.lazyloadedids.insert(&key, &[])?;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
Ok(())
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn lazy_load_reset(
|
||||||
|
&self,
|
||||||
|
user_id: &Box<UserId>,
|
||||||
|
device_id: &Box<DeviceId>,
|
||||||
|
room_id: &Box<RoomId>,
|
||||||
|
) -> Result<()> {
|
||||||
|
let mut prefix = user_id.as_bytes().to_vec();
|
||||||
|
prefix.push(0xff);
|
||||||
|
prefix.extend_from_slice(&device_id.as_bytes());
|
||||||
|
prefix.push(0xff);
|
||||||
|
prefix.extend_from_slice(&room_id.as_bytes());
|
||||||
|
prefix.push(0xff);
|
||||||
|
|
||||||
|
for (key, _) in self.lazyloadedids.scan_prefix(prefix) {
|
||||||
|
self.lazyloadedids.remove(&key)?;
|
||||||
|
}
|
||||||
|
|
||||||
|
Ok(())
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,6 +1,9 @@
|
||||||
use crate::{utils, Error, Result};
|
use crate::{utils, Error, Result};
|
||||||
use ruma::{
|
use ruma::{
|
||||||
api::client::{error::ErrorKind, r0::device::Device},
|
api::client::{
|
||||||
|
error::ErrorKind,
|
||||||
|
r0::{device::Device, filter::IncomingFilterDefinition},
|
||||||
|
},
|
||||||
encryption::{CrossSigningKey, DeviceKeys, OneTimeKey},
|
encryption::{CrossSigningKey, DeviceKeys, OneTimeKey},
|
||||||
events::{AnyToDeviceEvent, EventType},
|
events::{AnyToDeviceEvent, EventType},
|
||||||
identifiers::MxcUri,
|
identifiers::MxcUri,
|
||||||
|
@ -31,6 +34,8 @@ pub struct Users {
|
||||||
pub(super) userid_selfsigningkeyid: Arc<dyn Tree>,
|
pub(super) userid_selfsigningkeyid: Arc<dyn Tree>,
|
||||||
pub(super) userid_usersigningkeyid: Arc<dyn Tree>,
|
pub(super) userid_usersigningkeyid: Arc<dyn Tree>,
|
||||||
|
|
||||||
|
pub(super) userfilterid_filter: Arc<dyn Tree>, // UserFilterId = UserId + FilterId
|
||||||
|
|
||||||
pub(super) todeviceid_events: Arc<dyn Tree>, // ToDeviceId = UserId + DeviceId + Count
|
pub(super) todeviceid_events: Arc<dyn Tree>, // ToDeviceId = UserId + DeviceId + Count
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -990,4 +995,45 @@ impl Users {
|
||||||
// TODO: Unhook 3PID
|
// TODO: Unhook 3PID
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Creates a new sync filter. Returns the filter id.
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn create_filter(
|
||||||
|
&self,
|
||||||
|
user_id: &UserId,
|
||||||
|
filter: &IncomingFilterDefinition,
|
||||||
|
) -> Result<String> {
|
||||||
|
let filter_id = utils::random_string(4);
|
||||||
|
|
||||||
|
let mut key = user_id.as_bytes().to_vec();
|
||||||
|
key.push(0xff);
|
||||||
|
key.extend_from_slice(filter_id.as_bytes());
|
||||||
|
|
||||||
|
self.userfilterid_filter.insert(
|
||||||
|
&key,
|
||||||
|
&serde_json::to_vec(&filter).expect("filter is valid json"),
|
||||||
|
)?;
|
||||||
|
|
||||||
|
Ok(filter_id)
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tracing::instrument(skip(self))]
|
||||||
|
pub fn get_filter(
|
||||||
|
&self,
|
||||||
|
user_id: &UserId,
|
||||||
|
filter_id: &str,
|
||||||
|
) -> Result<Option<IncomingFilterDefinition>> {
|
||||||
|
let mut key = user_id.as_bytes().to_vec();
|
||||||
|
key.push(0xff);
|
||||||
|
key.extend_from_slice(filter_id.as_bytes());
|
||||||
|
|
||||||
|
let raw = self.userfilterid_filter.get(&key)?;
|
||||||
|
|
||||||
|
if let Some(raw) = raw {
|
||||||
|
serde_json::from_slice(&raw)
|
||||||
|
.map_err(|_| Error::bad_database("Invalid filter event in db."))
|
||||||
|
} else {
|
||||||
|
Ok(None)
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue