alemi/upub
alemi/upub
1
0
Fork 1
Code Issues 4 Pull requests Projects Releases Packages 1 Wiki Activity Actions

fix: get actor id from job object

Browse source
This commit is contained in:
əlemi 2024-08-15 05:39:40 +02:00
parent 817f88e1a4
commit 43a4e70cf4
Signed by: alemi
GPG key ID: A4895B84D311642C
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
upub/worker/src/outbound.rs
View file

@ -14,12 +14,12 @@ pub async fn process(ctx: Context, job: &model::job::Model) -> crate::JobResult<
// notifications as seen is a very internal thing to do and should not be in .process() // notifications as seen is a very internal thing to do and should not be in .process()
// probably. still this feels a bit dirty to do, is there a better place to do it? // probably. still this feels a bit dirty to do, is there a better place to do it?
if matches!(t, apb::ObjectType::Activity(apb::ActivityType::View)) { if matches!(t, apb::ObjectType::Activity(apb::ActivityType::View)) {
let actor = upub::model::actor::Entity::ap_to_internal(activity.actor().id()?, &tx) let actor = upub::model::actor::Entity::ap_to_internal(&job.actor, &tx)
.await? .await?
.ok_or_else(|| DbErr::RecordNotFound(activity.actor().id().unwrap_or_default().to_string()))?; .ok_or_else(|| DbErr::RecordNotFound(job.actor.clone()))?;
let activity = upub::model::activity::Entity::ap_to_internal(activity.object().id()?, &tx) let activity = upub::model::activity::Entity::ap_to_internal(&job.activity, &tx)
.await? .await?
.ok_or_else(|| DbErr::RecordNotFound(activity.actor().id().unwrap_or_default().to_string()))?; .ok_or_else(|| DbErr::RecordNotFound(job.activity.clone()))?;
let notif_model = upub::model::notification::ActiveModel { let notif_model = upub::model::notification::ActiveModel {
internal: sea_orm::ActiveValue::NotSet, internal: sea_orm::ActiveValue::NotSet,
activity: sea_orm::ActiveValue::Unchanged(activity), activity: sea_orm::ActiveValue::Unchanged(activity),