Verified Commit 9e285d8b authored by les's avatar les
Browse files

fix AP resource removal

parent cd313ef1
......@@ -59,7 +59,7 @@ module.exports = {
async remove (req, res) {
const resource = await Resource.findOne({
where: { activitypub_id: req.body.object.id },
include: [{ model: APUser, required: false, attributes: ['ap_id'] }]
include: [{ model: APUser, required: true, attributes: ['ap_id'] }]
})
if (!resource) {
log.info(`Comment ${req.body.object.id} not found`)
......@@ -67,8 +67,7 @@ module.exports = {
}
// check if fedi_user that requested resource removal
// is the same that created the resource at first place
log.debug(res.fedi_user.ap_id, resource.ap_user.ap_id)
if (res.fedi_user.ap_id === resource.ap_user.id) {
if (req.fedi_user.ap_id === resource.ap_user.id) {
await resource.destroy()
log.info(`Comment ${req.body.object.id} removed`)
res.sendStatus(201)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment