zaaarf/upub
1
0
Fork 0
forked from alemi/upub
Code Pull requests Activity

fix: maybe get context properly again?

Browse source
This commit is contained in:
əlemi 2024-06-06 15:54:19 +02:00
parent 1ce89aa6f9
commit a6015a32ed
Signed by: alemi
GPG key ID: A4895B84D311642C
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
upub/core/src/traits/normalize.rs
View file

@ -34,17 +34,15 @@ impl Normalizer for crate::Context {
let mut object_active_model = AP::object_q(&object)?; let mut object_active_model = AP::object_q(&object)?;
// make sure content only contains a safe subset of html // make sure content only contains a safe subset of html
if let Set(Some(content)) = object_active_model.content { if let Ok(content) = object.content() {
object_active_model.content = Set(Some(mdhtml::safe_html(&content))); object_active_model.content = Set(Some(mdhtml::safe_html(content)));
} }
// fix context for remote posts // fix context for remote posts
// > note that this will effectively recursively try to fetch the parent object, in order to find // > if any link is broken or we get rate limited, the whole insertion fails which is
// > the context (which is id of topmost object). there's a recursion limit of 16 hidden inside
// > btw! also if any link is broken or we get rate limited, the whole insertion fails which is
// > kind of dumb. there should be a job system so this can be done in waves. or maybe there's // > kind of dumb. there should be a job system so this can be done in waves. or maybe there's
// > some whole other way to do this?? im thinking but misskey aaaa!! TODO // > some whole other way to do this?? im thinking but misskey aaaa!! TODO
if let Set(Some(ref reply)) = object_active_model.in_reply_to { if let Ok(reply) = object.in_reply_to().id() {
if let Some(o) = crate::model::object::Entity::find_by_ap_id(reply).one(tx).await? { if let Some(o) = crate::model::object::Entity::find_by_ap_id(reply).one(tx).await? {
object_active_model.context = Set(o.context); object_active_model.context = Set(o.context);
} else { } else {