Dont swallow API errors (fixes #1834)
34 files changed:
// Verify that only the recipient can mark as read
if local_user_view.person.id != orig_comment.get_recipient_id() {
// Verify that only the recipient can mark as read
if local_user_view.person.id != orig_comment.get_recipient_id() {
- return Err(ApiError::err("no_comment_edit_allowed").into());
+ return Err(ApiError::err_plain("no_comment_edit_allowed").into());
Comment::update_read(conn, comment_id, read)
})
.await?
Comment::update_read(conn, comment_id, read)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_comment"))?;
+ .map_err(|_| ApiError::err_plain("couldnt_update_comment"))?;
// Refetch it
let comment_id = data.comment_id;
// Refetch it
let comment_id = data.comment_id;
if data.save {
let save_comment = move |conn: &'_ _| CommentSaved::save(conn, &comment_saved_form);
if data.save {
let save_comment = move |conn: &'_ _| CommentSaved::save(conn, &comment_saved_form);
- if blocking(context.pool(), save_comment).await?.is_err() {
- return Err(ApiError::err("couldnt_save_comment").into());
- }
+ blocking(context.pool(), save_comment)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_save_comment", e))?;
} else {
let unsave_comment = move |conn: &'_ _| CommentSaved::unsave(conn, &comment_saved_form);
} else {
let unsave_comment = move |conn: &'_ _| CommentSaved::unsave(conn, &comment_saved_form);
- if blocking(context.pool(), unsave_comment).await?.is_err() {
- return Err(ApiError::err("couldnt_save_comment").into());
- }
+ blocking(context.pool(), unsave_comment)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_save_comment", e))?;
}
let comment_id = data.comment_id;
}
let comment_id = data.comment_id;
if do_add {
let like_form2 = like_form.clone();
let like = move |conn: &'_ _| CommentLike::like(conn, &like_form2);
if do_add {
let like_form2 = like_form.clone();
let like = move |conn: &'_ _| CommentLike::like(conn, &like_form2);
- if blocking(context.pool(), like).await?.is_err() {
- return Err(ApiError::err("couldnt_like_comment").into());
- }
+ blocking(context.pool(), like)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_like_comment", e))?;
// check size of report and check for whitespace
let reason = data.reason.trim();
if reason.is_empty() {
// check size of report and check for whitespace
let reason = data.reason.trim();
if reason.is_empty() {
- return Err(ApiError::err("report_reason_required").into());
+ return Err(ApiError::err_plain("report_reason_required").into());
}
if reason.chars().count() > 1000 {
}
if reason.chars().count() > 1000 {
- return Err(ApiError::err("report_too_long").into());
+ return Err(ApiError::err_plain("report_too_long").into());
}
let person_id = local_user_view.person.id;
}
let person_id = local_user_view.person.id;
CommentReport::report(conn, &report_form)
})
.await?
CommentReport::report(conn, &report_form)
})
.await?
- .map_err(|_| ApiError::err("couldnt_create_report"))?;
+ .map_err(|e| ApiError::err("couldnt_create_report", e))?;
let comment_report_view = blocking(context.pool(), move |conn| {
CommentReportView::read(conn, report.id, person_id)
let comment_report_view = blocking(context.pool(), move |conn| {
CommentReportView::read(conn, report.id, person_id)
- if blocking(context.pool(), resolve_fun).await?.is_err() {
- return Err(ApiError::err("couldnt_resolve_report").into());
- };
+ blocking(context.pool(), resolve_fun)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_resolve_report", e))?;
let report_id = data.report_id;
let comment_report_view = blocking(context.pool(), move |conn| {
let report_id = data.report_id;
let comment_report_view = blocking(context.pool(), move |conn| {
check_community_ban(local_user_view.person.id, community_id, context.pool()).await?;
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
check_community_ban(local_user_view.person.id, community_id, context.pool()).await?;
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
- if blocking(context.pool(), follow).await?.is_err() {
- return Err(ApiError::err("community_follower_already_exists").into());
- }
+ blocking(context.pool(), follow)
+ .await?
+ .map_err(|e| ApiError::err("community_follower_already_exists", e))?;
} else {
let unfollow =
move |conn: &'_ _| CommunityFollower::unfollow(conn, &community_follower_form);
} else {
let unfollow =
move |conn: &'_ _| CommunityFollower::unfollow(conn, &community_follower_form);
- if blocking(context.pool(), unfollow).await?.is_err() {
- return Err(ApiError::err("community_follower_already_exists").into());
- }
+ blocking(context.pool(), unfollow)
+ .await?
+ .map_err(|e| ApiError::err("community_follower_already_exists", e))?;
}
} else if data.follow {
// Dont actually add to the community followers here, because you need
}
} else if data.follow {
// Dont actually add to the community followers here, because you need
} else {
UndoFollowCommunity::send(&local_user_view.person, &community, context).await?;
let unfollow = move |conn: &'_ _| CommunityFollower::unfollow(conn, &community_follower_form);
} else {
UndoFollowCommunity::send(&local_user_view.person, &community, context).await?;
let unfollow = move |conn: &'_ _| CommunityFollower::unfollow(conn, &community_follower_form);
- if blocking(context.pool(), unfollow).await?.is_err() {
- return Err(ApiError::err("community_follower_already_exists").into());
- }
+ blocking(context.pool(), unfollow)
+ .await?
+ .map_err(|e| ApiError::err("community_follower_already_exists", e))?;
}
let community_id = data.community_id;
}
let community_id = data.community_id;
if data.block {
let block = move |conn: &'_ _| CommunityBlock::block(conn, &community_block_form);
if data.block {
let block = move |conn: &'_ _| CommunityBlock::block(conn, &community_block_form);
- if blocking(context.pool(), block).await?.is_err() {
- return Err(ApiError::err("community_block_already_exists").into());
- }
+ blocking(context.pool(), block)
+ .await?
+ .map_err(|e| ApiError::err("community_block_already_exists", e))?;
// Also, unfollow the community, and send a federated unfollow
let community_follower_form = CommunityFollowerForm {
// Also, unfollow the community, and send a federated unfollow
let community_follower_form = CommunityFollowerForm {
UndoFollowCommunity::send(&local_user_view.person, &community, context).await?;
} else {
let unblock = move |conn: &'_ _| CommunityBlock::unblock(conn, &community_block_form);
UndoFollowCommunity::send(&local_user_view.person, &community, context).await?;
} else {
let unblock = move |conn: &'_ _| CommunityBlock::unblock(conn, &community_block_form);
- if blocking(context.pool(), unblock).await?.is_err() {
- return Err(ApiError::err("community_block_already_exists").into());
- }
+ blocking(context.pool(), unblock)
+ .await?
+ .map_err(|e| ApiError::err("community_block_already_exists", e))?;
}
let community_view = blocking(context.pool(), move |conn| {
}
let community_view = blocking(context.pool(), move |conn| {
if data.ban {
let ban = move |conn: &'_ _| CommunityPersonBan::ban(conn, &community_user_ban_form);
if data.ban {
let ban = move |conn: &'_ _| CommunityPersonBan::ban(conn, &community_user_ban_form);
- if blocking(context.pool(), ban).await?.is_err() {
- return Err(ApiError::err("community_user_already_banned").into());
- }
+ blocking(context.pool(), ban)
+ .await?
+ .map_err(|e| ApiError::err("community_user_already_banned", e))?;
// Also unsubscribe them from the community, if they are subscribed
let community_follower_form = CommunityFollowerForm {
// Also unsubscribe them from the community, if they are subscribed
let community_follower_form = CommunityFollowerForm {
.await?;
} else {
let unban = move |conn: &'_ _| CommunityPersonBan::unban(conn, &community_user_ban_form);
.await?;
} else {
let unban = move |conn: &'_ _| CommunityPersonBan::unban(conn, &community_user_ban_form);
- if blocking(context.pool(), unban).await?.is_err() {
- return Err(ApiError::err("community_user_already_banned").into());
- }
+ blocking(context.pool(), unban)
+ .await?
+ .map_err(|e| ApiError::err("community_user_already_banned", e))?;
UndoBlockUserFromCommunity::send(
&community,
&banned_person,
UndoBlockUserFromCommunity::send(
&community,
&banned_person,
};
if data.added {
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
};
if data.added {
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
- if blocking(context.pool(), join).await?.is_err() {
- return Err(ApiError::err("community_moderator_already_exists").into());
- }
+ blocking(context.pool(), join)
+ .await?
+ .map_err(|e| ApiError::err("community_moderator_already_exists", e))?;
} else {
let leave = move |conn: &'_ _| CommunityModerator::leave(conn, &community_moderator_form);
} else {
let leave = move |conn: &'_ _| CommunityModerator::leave(conn, &community_moderator_form);
- if blocking(context.pool(), leave).await?.is_err() {
- return Err(ApiError::err("community_moderator_already_exists").into());
- }
+ blocking(context.pool(), leave)
+ .await?
+ .map_err(|e| ApiError::err("community_moderator_already_exists", e))?;
.map(|a| a.person.id)
.any(|x| x == local_user_view.person.id)
{
.map(|a| a.person.id)
.any(|x| x == local_user_view.person.id)
{
- return Err(ApiError::err("not_an_admin").into());
+ return Err(ApiError::err_plain("not_an_admin").into());
}
// You have to re-do the community_moderator table, reordering it.
}
// You have to re-do the community_moderator table, reordering it.
};
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
};
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
- if blocking(context.pool(), join).await?.is_err() {
- return Err(ApiError::err("community_moderator_already_exists").into());
- }
+ blocking(context.pool(), join)
+ .await?
+ .map_err(|e| ApiError::err("community_moderator_already_exists", e))?;
CommunityView::read(conn, community_id, Some(person_id))
})
.await?
CommunityView::read(conn, community_id, Some(person_id))
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_community"))?;
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?;
let community_id = data.community_id;
let moderators = blocking(context.pool(), move |conn| {
CommunityModeratorView::for_community(conn, community_id)
})
.await?
let community_id = data.community_id;
let moderators = blocking(context.pool(), move |conn| {
CommunityModeratorView::for_community(conn, community_id)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_community"))?;
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?;
// Return the jwt
Ok(GetCommunityResponse {
// Return the jwt
Ok(GetCommunityResponse {
LocalUserView::find_by_email_or_name(conn, &username_or_email)
})
.await?
LocalUserView::find_by_email_or_name(conn, &username_or_email)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_that_username_or_email"))?;
+ .map_err(|e| ApiError::err("couldnt_find_that_username_or_email", e))?;
// Verify the password
let valid: bool = verify(
// Verify the password
let valid: bool = verify(
)
.unwrap_or(false);
if !valid {
)
.unwrap_or(false);
if !valid {
- return Err(ApiError::err("password_incorrect").into());
+ return Err(ApiError::err_plain("password_incorrect").into());
if let Some(Some(bio)) = &bio {
if bio.chars().count() > 300 {
if let Some(Some(bio)) = &bio {
if bio.chars().count() > 300 {
- return Err(ApiError::err("bio_length_overflow").into());
+ return Err(ApiError::err_plain("bio_length_overflow").into());
display_name.trim(),
context.settings().actor_name_max_length,
) {
display_name.trim(),
context.settings().actor_name_max_length,
) {
- return Err(ApiError::err("invalid_username").into());
+ return Err(ApiError::err_plain("invalid_username").into());
}
}
if let Some(Some(matrix_user_id)) = &matrix_user_id {
if !is_valid_matrix_id(matrix_user_id) {
}
}
if let Some(Some(matrix_user_id)) = &matrix_user_id {
if !is_valid_matrix_id(matrix_user_id) {
- return Err(ApiError::err("invalid_matrix_id").into());
+ return Err(ApiError::err_plain("invalid_matrix_id").into());
- let person_res = blocking(context.pool(), move |conn| {
+ blocking(context.pool(), move |conn| {
Person::update(conn, person_id, &person_form)
})
Person::update(conn, person_id, &person_form)
})
- .await?;
- let _updated_person: Person = match person_res {
- Ok(p) => p,
- Err(_) => {
- return Err(ApiError::err("user_already_exists").into());
- }
- };
+ .await?
+ .map_err(|e| ApiError::err("user_already_exists", e))?;
let local_user_form = LocalUserForm {
person_id,
let local_user_form = LocalUserForm {
person_id,
- return Err(ApiError::err(err_type).into());
+ return Err(ApiError::err(err_type, e).into());
// Make sure passwords match
if data.new_password != data.new_password_verify {
// Make sure passwords match
if data.new_password != data.new_password_verify {
- return Err(ApiError::err("passwords_dont_match").into());
+ return Err(ApiError::err_plain("passwords_dont_match").into());
}
// Check the old password
}
// Check the old password
)
.unwrap_or(false);
if !valid {
)
.unwrap_or(false);
if !valid {
- return Err(ApiError::err("password_incorrect").into());
+ return Err(ApiError::err_plain("password_incorrect").into());
}
let local_user_id = local_user_view.local_user.id;
}
let local_user_id = local_user_view.local_user.id;
let added = data.added;
let added_person_id = data.person_id;
let added = data.added;
let added_person_id = data.person_id;
- let added_admin = match blocking(context.pool(), move |conn| {
+ let added_admin = blocking(context.pool(), move |conn| {
Person::add_admin(conn, added_person_id, added)
})
.await?
Person::add_admin(conn, added_person_id, added)
})
.await?
- {
- Ok(a) => a,
- Err(_) => {
- return Err(ApiError::err("couldnt_update_user").into());
- }
- };
+ .map_err(|e| ApiError::err("couldnt_update_user", e))?;
// Mod tables
let form = ModAddForm {
// Mod tables
let form = ModAddForm {
let ban = data.ban;
let banned_person_id = data.person_id;
let ban_person = move |conn: &'_ _| Person::ban_person(conn, banned_person_id, ban);
let ban = data.ban;
let banned_person_id = data.person_id;
let ban_person = move |conn: &'_ _| Person::ban_person(conn, banned_person_id, ban);
- if blocking(context.pool(), ban_person).await?.is_err() {
- return Err(ApiError::err("couldnt_update_user").into());
- }
+ blocking(context.pool(), ban_person)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_user", e))?;
// Remove their data if that's desired
if data.remove_data.unwrap_or(false) {
// Remove their data if that's desired
if data.remove_data.unwrap_or(false) {
// Don't let a person block themselves
if target_id == person_id {
// Don't let a person block themselves
if target_id == person_id {
- return Err(ApiError::err("cant_block_yourself").into());
+ return Err(ApiError::err_plain("cant_block_yourself").into());
}
let person_block_form = PersonBlockForm {
}
let person_block_form = PersonBlockForm {
if data.block {
let block = move |conn: &'_ _| PersonBlock::block(conn, &person_block_form);
if data.block {
let block = move |conn: &'_ _| PersonBlock::block(conn, &person_block_form);
- if blocking(context.pool(), block).await?.is_err() {
- return Err(ApiError::err("person_block_already_exists").into());
- }
+ blocking(context.pool(), block)
+ .await?
+ .map_err(|e| ApiError::err("person_block_already_exists", e))?;
} else {
let unblock = move |conn: &'_ _| PersonBlock::unblock(conn, &person_block_form);
} else {
let unblock = move |conn: &'_ _| PersonBlock::unblock(conn, &person_block_form);
- if blocking(context.pool(), unblock).await?.is_err() {
- return Err(ApiError::err("person_block_already_exists").into());
- }
+ blocking(context.pool(), unblock)
+ .await?
+ .map_err(|e| ApiError::err("person_block_already_exists", e))?;
}
// TODO does any federated stuff need to be done here?
}
// TODO does any federated stuff need to be done here?
.await??;
if local_user_view.person.id != read_person_mention.recipient_id {
.await??;
if local_user_view.person.id != read_person_mention.recipient_id {
- return Err(ApiError::err("couldnt_update_comment").into());
+ return Err(ApiError::err_plain("couldnt_update_comment").into());
}
let person_mention_id = read_person_mention.id;
let read = data.read;
let update_mention =
move |conn: &'_ _| PersonMention::update_read(conn, person_mention_id, read);
}
let person_mention_id = read_person_mention.id;
let read = data.read;
let update_mention =
move |conn: &'_ _| PersonMention::update_read(conn, person_mention_id, read);
- if blocking(context.pool(), update_mention).await?.is_err() {
- return Err(ApiError::err("couldnt_update_comment").into());
- };
+ blocking(context.pool(), update_mention)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
let person_mention_id = read_person_mention.id;
let person_id = local_user_view.person.id;
let person_mention_id = read_person_mention.id;
let person_id = local_user_view.person.id;
for comment_view in &replies {
let reply_id = comment_view.comment.id;
let mark_as_read = move |conn: &'_ _| Comment::update_read(conn, reply_id, true);
for comment_view in &replies {
let reply_id = comment_view.comment.id;
let mark_as_read = move |conn: &'_ _| Comment::update_read(conn, reply_id, true);
- if blocking(context.pool(), mark_as_read).await?.is_err() {
- return Err(ApiError::err("couldnt_update_comment").into());
- }
+ blocking(context.pool(), mark_as_read)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
}
// Mark all user mentions as read
let update_person_mentions =
move |conn: &'_ _| PersonMention::mark_all_as_read(conn, person_id);
}
// Mark all user mentions as read
let update_person_mentions =
move |conn: &'_ _| PersonMention::mark_all_as_read(conn, person_id);
- if blocking(context.pool(), update_person_mentions)
+ blocking(context.pool(), update_person_mentions)
- .is_err()
- {
- return Err(ApiError::err("couldnt_update_comment").into());
- }
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
// Mark all private_messages as read
let update_pm = move |conn: &'_ _| PrivateMessage::mark_all_as_read(conn, person_id);
// Mark all private_messages as read
let update_pm = move |conn: &'_ _| PrivateMessage::mark_all_as_read(conn, person_id);
- if blocking(context.pool(), update_pm).await?.is_err() {
- return Err(ApiError::err("couldnt_update_private_message").into());
- }
+ blocking(context.pool(), update_pm)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_private_message", e))?;
Ok(GetRepliesResponse { replies: vec![] })
}
Ok(GetRepliesResponse { replies: vec![] })
}
LocalUserView::find_by_email(conn, &email)
})
.await?
LocalUserView::find_by_email(conn, &email)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_that_username_or_email"))?;
+ .map_err(|e| ApiError::err("couldnt_find_that_username_or_email", e))?;
// Generate a random token
let token = generate_random_string();
// Generate a random token
let token = generate_random_string();
html,
&context.settings(),
)
html,
&context.settings(),
)
- .map_err(|e| ApiError::err(&e))?;
+ .map_err(|e| ApiError::err("email_send_failed", e))?;
Ok(PasswordResetResponse {})
}
Ok(PasswordResetResponse {})
}
// Make sure passwords match
if data.password != data.password_verify {
// Make sure passwords match
if data.password != data.password_verify {
- return Err(ApiError::err("passwords_dont_match").into());
+ return Err(ApiError::err_plain("passwords_dont_match").into());
}
// Update the user with the new password
}
// Update the user with the new password
LocalUser::update_password(conn, local_user_id, &password)
})
.await?
LocalUser::update_password(conn, local_user_id, &password)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_user"))?;
+ .map_err(|e| ApiError::err("couldnt_update_user", e))?;
// Return the jwt
Ok(LoginResponse {
// Return the jwt
Ok(LoginResponse {
if do_add {
let like_form2 = like_form.clone();
let like = move |conn: &'_ _| PostLike::like(conn, &like_form2);
if do_add {
let like_form2 = like_form.clone();
let like = move |conn: &'_ _| PostLike::like(conn, &like_form2);
- if blocking(context.pool(), like).await?.is_err() {
- return Err(ApiError::err("couldnt_like_post").into());
- }
+ blocking(context.pool(), like)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_like_post", e))?;
if data.save {
let save = move |conn: &'_ _| PostSaved::save(conn, &post_saved_form);
if data.save {
let save = move |conn: &'_ _| PostSaved::save(conn, &post_saved_form);
- if blocking(context.pool(), save).await?.is_err() {
- return Err(ApiError::err("couldnt_save_post").into());
- }
+ blocking(context.pool(), save)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_save_post", e))?;
} else {
let unsave = move |conn: &'_ _| PostSaved::unsave(conn, &post_saved_form);
} else {
let unsave = move |conn: &'_ _| PostSaved::unsave(conn, &post_saved_form);
- if blocking(context.pool(), unsave).await?.is_err() {
- return Err(ApiError::err("couldnt_save_post").into());
- }
+ blocking(context.pool(), unsave)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_save_post", e))?;
}
let post_id = data.post_id;
}
let post_id = data.post_id;
// check size of report and check for whitespace
let reason = data.reason.trim();
if reason.is_empty() {
// check size of report and check for whitespace
let reason = data.reason.trim();
if reason.is_empty() {
- return Err(ApiError::err("report_reason_required").into());
+ return Err(ApiError::err_plain("report_reason_required").into());
}
if reason.chars().count() > 1000 {
}
if reason.chars().count() > 1000 {
- return Err(ApiError::err("report_too_long").into());
+ return Err(ApiError::err_plain("report_too_long").into());
}
let person_id = local_user_view.person.id;
}
let person_id = local_user_view.person.id;
PostReport::report(conn, &report_form)
})
.await?
PostReport::report(conn, &report_form)
})
.await?
- .map_err(|_| ApiError::err("couldnt_create_report"))?;
+ .map_err(|e| ApiError::err("couldnt_create_report", e))?;
let post_report_view = blocking(context.pool(), move |conn| {
PostReportView::read(conn, report.id, person_id)
let post_report_view = blocking(context.pool(), move |conn| {
PostReportView::read(conn, report.id, person_id)
- if blocking(context.pool(), resolve_fun).await?.is_err() {
- return Err(ApiError::err("couldnt_resolve_report").into());
- };
+ blocking(context.pool(), resolve_fun)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_resolve_report", e))?;
let post_report_view = blocking(context.pool(), move |conn| {
PostReportView::read(conn, report_id, person_id)
let post_report_view = blocking(context.pool(), move |conn| {
PostReportView::read(conn, report_id, person_id)
})
.await??;
if local_user_view.person.id != orig_private_message.recipient_id {
})
.await??;
if local_user_view.person.id != orig_private_message.recipient_id {
- return Err(ApiError::err("couldnt_update_private_message").into());
+ return Err(ApiError::err_plain("couldnt_update_private_message").into());
PrivateMessage::update_read(conn, private_message_id, read)
})
.await?
PrivateMessage::update_read(conn, private_message_id, read)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_private_message"))?;
+ .map_err(|e| ApiError::err("couldnt_update_private_message", e))?;
// No need to send an apub update
let op = UserOperation::MarkPrivateMessageAsRead;
// No need to send an apub update
let op = UserOperation::MarkPrivateMessageAsRead;
get_local_user_view_from_jwt_opt(&self.auth, context.pool(), context.secret()).await?;
let res = search_by_apub_id(&self.q, context)
.await
get_local_user_view_from_jwt_opt(&self.auth, context.pool(), context.secret()).await?;
let res = search_by_apub_id(&self.q, context)
.await
- .map_err(|_| ApiError::err("couldnt_find_object"))?;
+ .map_err(|e| ApiError::err("couldnt_find_object", e))?;
convert_response(res, local_user_view.map(|l| l.person.id), context.pool())
.await
convert_response(res, local_user_view.map(|l| l.person.id), context.pool())
.await
- .map_err(|_| ApiError::err("couldnt_find_object").into())
+ .map_err(|e| ApiError::err("couldnt_find_object", e).into())
// Make sure user is the creator
if read_site.creator_id != local_user_view.person.id {
// Make sure user is the creator
if read_site.creator_id != local_user_view.person.id {
- return Err(ApiError::err("not_an_admin").into());
+ return Err(ApiError::err_plain("not_an_admin").into());
}
let new_creator_id = data.person_id;
let transfer_site = move |conn: &'_ _| Site::transfer(conn, new_creator_id);
}
let new_creator_id = data.person_id;
let transfer_site = move |conn: &'_ _| Site::transfer(conn, new_creator_id);
- if blocking(context.pool(), transfer_site).await?.is_err() {
- return Err(ApiError::err("couldnt_update_site").into());
- };
+ blocking(context.pool(), transfer_site)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_site", e))?;
// Mod tables
let form = ModAddForm {
// Mod tables
let form = ModAddForm {
// Make sure docker doesn't have :ro at the end of the volume, so its not a read-only filesystem
let config_hjson = Settings::save_config_file(&data.config_hjson)
// Make sure docker doesn't have :ro at the end of the volume, so its not a read-only filesystem
let config_hjson = Settings::save_config_file(&data.config_hjson)
- .map_err(|_| ApiError::err("couldnt_update_site"))?;
+ .map_err(|e| ApiError::err("couldnt_update_site", e))?;
Ok(GetSiteConfigResponse { config_hjson })
}
Ok(GetSiteConfigResponse { config_hjson })
}
})
.await?;
if !is_mod_or_admin {
})
.await?;
if !is_mod_or_admin {
- return Err(ApiError::err("not_a_mod_or_admin").into());
+ return Err(ApiError::err_plain("not_a_mod_or_admin").into());
}
Ok(())
}
pub fn is_admin(local_user_view: &LocalUserView) -> Result<(), LemmyError> {
if !local_user_view.person.admin {
}
Ok(())
}
pub fn is_admin(local_user_view: &LocalUserView) -> Result<(), LemmyError> {
if !local_user_view.person.admin {
- return Err(ApiError::err("not_an_admin").into());
+ return Err(ApiError::err_plain("not_an_admin").into());
pub async fn get_post(post_id: PostId, pool: &DbPool) -> Result<Post, LemmyError> {
blocking(pool, move |conn| Post::read(conn, post_id))
.await?
pub async fn get_post(post_id: PostId, pool: &DbPool) -> Result<Post, LemmyError> {
blocking(pool, move |conn| Post::read(conn, post_id))
.await?
- .map_err(|_| ApiError::err("couldnt_find_post").into())
+ .map_err(|_| ApiError::err_plain("couldnt_find_post").into())
}
pub async fn mark_post_as_read(
}
pub async fn mark_post_as_read(
PostRead::mark_as_read(conn, &post_read_form)
})
.await?
PostRead::mark_as_read(conn, &post_read_form)
})
.await?
- .map_err(|_| ApiError::err("couldnt_mark_post_as_read").into())
+ .map_err(|_| ApiError::err_plain("couldnt_mark_post_as_read").into())
}
pub async fn get_local_user_view_from_jwt(
}
pub async fn get_local_user_view_from_jwt(
secret: &Secret,
) -> Result<LocalUserView, LemmyError> {
let claims = Claims::decode(jwt, &secret.jwt_secret)
secret: &Secret,
) -> Result<LocalUserView, LemmyError> {
let claims = Claims::decode(jwt, &secret.jwt_secret)
- .map_err(|_| ApiError::err("not_logged_in"))?
+ .map_err(|e| ApiError::err("not_logged_in", e))?
.claims;
let local_user_id = LocalUserId(claims.sub);
let local_user_view =
blocking(pool, move |conn| LocalUserView::read(conn, local_user_id)).await??;
// Check for a site ban
if local_user_view.person.banned {
.claims;
let local_user_id = LocalUserId(claims.sub);
let local_user_view =
blocking(pool, move |conn| LocalUserView::read(conn, local_user_id)).await??;
// Check for a site ban
if local_user_view.person.banned {
- return Err(ApiError::err("site_ban").into());
+ return Err(ApiError::err_plain("site_ban").into());
}
// Check for user deletion
if local_user_view.person.deleted {
}
// Check for user deletion
if local_user_view.person.deleted {
- return Err(ApiError::err("deleted").into());
+ return Err(ApiError::err_plain("deleted").into());
}
check_validator_time(&local_user_view.local_user.validator_time, &claims)?;
}
check_validator_time(&local_user_view.local_user.validator_time, &claims)?;
) -> Result<(), LemmyError> {
let user_validation_time = validator_time.timestamp();
if user_validation_time > claims.iat {
) -> Result<(), LemmyError> {
let user_validation_time = validator_time.timestamp();
if user_validation_time > claims.iat {
- Err(ApiError::err("not_logged_in").into())
+ Err(ApiError::err_plain("not_logged_in").into())
secret: &Secret,
) -> Result<LocalUserSettingsView, LemmyError> {
let claims = Claims::decode(jwt, &secret.jwt_secret)
secret: &Secret,
) -> Result<LocalUserSettingsView, LemmyError> {
let claims = Claims::decode(jwt, &secret.jwt_secret)
- .map_err(|_| ApiError::err("not_logged_in"))?
+ .map_err(|e| ApiError::err("not_logged_in", e))?
.claims;
let local_user_id = LocalUserId(claims.sub);
let local_user_view = blocking(pool, move |conn| {
.claims;
let local_user_id = LocalUserId(claims.sub);
let local_user_view = blocking(pool, move |conn| {
.await??;
// Check for a site ban
if local_user_view.person.banned {
.await??;
// Check for a site ban
if local_user_view.person.banned {
- return Err(ApiError::err("site_ban").into());
+ return Err(ApiError::err_plain("site_ban").into());
}
check_validator_time(&local_user_view.local_user.validator_time, &claims)?;
}
check_validator_time(&local_user_view.local_user.validator_time, &claims)?;
let is_banned =
move |conn: &'_ _| CommunityPersonBanView::get(conn, person_id, community_id).is_ok();
if blocking(pool, is_banned).await? {
let is_banned =
move |conn: &'_ _| CommunityPersonBanView::get(conn, person_id, community_id).is_ok();
if blocking(pool, is_banned).await? {
- Err(ApiError::err("community_ban").into())
+ Err(ApiError::err_plain("community_ban").into())
) -> Result<(), LemmyError> {
let is_blocked = move |conn: &'_ _| PersonBlock::read(conn, potential_blocker_id, my_id).is_ok();
if blocking(pool, is_blocked).await? {
) -> Result<(), LemmyError> {
let is_blocked = move |conn: &'_ _| PersonBlock::read(conn, potential_blocker_id, my_id).is_ok();
if blocking(pool, is_blocked).await? {
- Err(ApiError::err("person_block").into())
+ Err(ApiError::err_plain("person_block").into())
if score == -1 {
let site = blocking(pool, move |conn| Site::read_simple(conn)).await??;
if !site.enable_downvotes {
if score == -1 {
let site = blocking(pool, move |conn| Site::read_simple(conn)).await??;
if !site.enable_downvotes {
- return Err(ApiError::err("downvotes_disabled").into());
+ return Err(ApiError::err_plain("downvotes_disabled").into());
/// Checks the password length
pub fn password_length_check(pass: &str) -> Result<(), LemmyError> {
if !(10..=60).contains(&pass.len()) {
/// Checks the password length
pub fn password_length_check(pass: &str) -> Result<(), LemmyError> {
if !(10..=60).contains(&pass.len()) {
- Err(ApiError::err("invalid_password").into())
+ Err(ApiError::err_plain("invalid_password").into())
/// Checks the site description length
pub fn site_description_length_check(description: &str) -> Result<(), LemmyError> {
if description.len() > 150 {
/// Checks the site description length
pub fn site_description_length_check(description: &str) -> Result<(), LemmyError> {
if description.len() > 150 {
- Err(ApiError::err("site_description_length_overflow").into())
+ Err(ApiError::err_plain("site_description_length_overflow").into())
/// Checks for a honeypot. If this field is filled, fail the rest of the function
pub fn honeypot_check(honeypot: &Option<String>) -> Result<(), LemmyError> {
if honeypot.is_some() {
/// Checks for a honeypot. If this field is filled, fail the rest of the function
pub fn honeypot_check(honeypot: &Option<String>) -> Result<(), LemmyError> {
if honeypot.is_some() {
- Err(ApiError::err("honeypot_fail").into())
+ Err(ApiError::err_plain("honeypot_fail").into())
// Check if post is locked, no new comments
if post.locked {
// Check if post is locked, no new comments
if post.locked {
- return Err(ApiError::err("locked").into());
+ return Err(ApiError::err_plain("locked").into());
}
// If there's a parent_id, check to make sure that comment is in that post
}
// If there's a parent_id, check to make sure that comment is in that post
// Make sure the parent comment exists
let parent = blocking(context.pool(), move |conn| Comment::read(conn, parent_id))
.await?
// Make sure the parent comment exists
let parent = blocking(context.pool(), move |conn| Comment::read(conn, parent_id))
.await?
- .map_err(|_| ApiError::err("couldnt_create_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_create_comment", e))?;
check_person_block(local_user_view.person.id, parent.creator_id, context.pool()).await?;
// Strange issue where sometimes the post ID is incorrect
if parent.post_id != post_id {
check_person_block(local_user_view.person.id, parent.creator_id, context.pool()).await?;
// Strange issue where sometimes the post ID is incorrect
if parent.post_id != post_id {
- return Err(ApiError::err("couldnt_create_comment").into());
+ return Err(ApiError::err_plain("couldnt_create_comment").into());
Comment::create(conn, &comment_form2)
})
.await?
Comment::create(conn, &comment_form2)
})
.await?
- .map_err(|_| ApiError::err("couldnt_create_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_create_comment", e))?;
// Necessary to update the ap_id
let inserted_comment_id = inserted_comment.id;
// Necessary to update the ap_id
let inserted_comment_id = inserted_comment.id;
Ok(Comment::update_ap_id(conn, inserted_comment_id, apub_id)?)
})
.await?
Ok(Comment::update_ap_id(conn, inserted_comment_id, apub_id)?)
})
.await?
- .map_err(|_| ApiError::err("couldnt_create_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_create_comment", e))?;
CreateOrUpdateComment::send(
&updated_comment,
CreateOrUpdateComment::send(
&updated_comment,
};
let like = move |conn: &'_ _| CommentLike::like(conn, &like_form);
};
let like = move |conn: &'_ _| CommentLike::like(conn, &like_form);
- if blocking(context.pool(), like).await?.is_err() {
- return Err(ApiError::err("couldnt_like_comment").into());
- }
+ blocking(context.pool(), like)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_like_comment", e))?;
let object = PostOrComment::Comment(updated_comment);
Vote::send(
let object = PostOrComment::Comment(updated_comment);
Vote::send(
Comment::update_read(conn, comment_id, true)
})
.await?
Comment::update_read(conn, comment_id, true)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
}
// If its a reply, mark the parent as read
if let Some(parent_id) = data.parent_id {
}
// If its a reply, mark the parent as read
if let Some(parent_id) = data.parent_id {
Comment::update_read(conn, parent_id, true)
})
.await?
Comment::update_read(conn, parent_id, true)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_parent_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_update_parent_comment", e))?;
}
// If the parent has PersonMentions mark them as read too
let person_id = local_user_view.person.id;
}
// If the parent has PersonMentions mark them as read too
let person_id = local_user_view.person.id;
PersonMention::update_read(conn, mention.id, true)
})
.await?
PersonMention::update_read(conn, mention.id, true)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_person_mentions"))?;
+ .map_err(|e| ApiError::err("couldnt_update_person_mentions", e))?;
// Verify that only the creator can delete
if local_user_view.person.id != orig_comment.creator.id {
// Verify that only the creator can delete
if local_user_view.person.id != orig_comment.creator.id {
- return Err(ApiError::err("no_comment_edit_allowed").into());
+ return Err(ApiError::err_plain("no_comment_edit_allowed").into());
Comment::update_deleted(conn, comment_id, deleted)
})
.await?
Comment::update_deleted(conn, comment_id, deleted)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
// Send the apub message
let community = blocking(context.pool(), move |conn| {
// Send the apub message
let community = blocking(context.pool(), move |conn| {
Comment::update_removed(conn, comment_id, removed)
})
.await?
Comment::update_removed(conn, comment_id, removed)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
// Mod tables
let form = ModRemoveCommentForm {
// Mod tables
let form = ModRemoveCommentForm {
- .map_err(|_| ApiError::err("couldnt_get_comments"))?;
+ .map_err(|e| ApiError::err("couldnt_get_comments", e))?;
// Blank out deleted or removed info
for cv in comments
// Blank out deleted or removed info
for cv in comments
// Verify that only the creator can edit
if local_user_view.person.id != orig_comment.creator.id {
// Verify that only the creator can edit
if local_user_view.person.id != orig_comment.creator.id {
- return Err(ApiError::err("no_comment_edit_allowed").into());
+ return Err(ApiError::err_plain("no_comment_edit_allowed").into());
Comment::update_content(conn, comment_id, &content_slurs_removed)
})
.await?
Comment::update_content(conn, comment_id, &content_slurs_removed)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_comment"))?;
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
// Send the apub update
CreateOrUpdateComment::send(
// Send the apub update
CreateOrUpdateComment::send(
let site = blocking(context.pool(), move |conn| Site::read(conn, 0)).await??;
if site.community_creation_admin_only && is_admin(&local_user_view).is_err() {
let site = blocking(context.pool(), move |conn| Site::read(conn, 0)).await??;
if site.community_creation_admin_only && is_admin(&local_user_view).is_err() {
- return Err(ApiError::err("only_admins_can_create_communities").into());
+ return Err(ApiError::err_plain("only_admins_can_create_communities").into());
}
check_slurs(&data.name, &context.settings().slur_regex())?;
}
check_slurs(&data.name, &context.settings().slur_regex())?;
check_slurs_opt(&data.description, &context.settings().slur_regex())?;
if !is_valid_actor_name(&data.name, context.settings().actor_name_max_length) {
check_slurs_opt(&data.description, &context.settings().slur_regex())?;
if !is_valid_actor_name(&data.name, context.settings().actor_name_max_length) {
- return Err(ApiError::err("invalid_community_name").into());
+ return Err(ApiError::err_plain("invalid_community_name").into());
}
// Double check for duplicate community actor_ids
}
// Double check for duplicate community actor_ids
let community_actor_id_wrapped = ObjectId::<Community>::new(community_actor_id.clone());
let community_dupe = community_actor_id_wrapped.dereference_local(context).await;
if community_dupe.is_ok() {
let community_actor_id_wrapped = ObjectId::<Community>::new(community_actor_id.clone());
let community_dupe = community_actor_id_wrapped.dereference_local(context).await;
if community_dupe.is_ok() {
- return Err(ApiError::err("community_already_exists").into());
+ return Err(ApiError::err_plain("community_already_exists").into());
}
// Check to make sure the icon and banners are urls
}
// Check to make sure the icon and banners are urls
Community::create(conn, &community_form)
})
.await?
Community::create(conn, &community_form)
})
.await?
- .map_err(|_| ApiError::err("community_already_exists"))?;
+ .map_err(|e| ApiError::err("community_already_exists", e))?;
// The community creator becomes a moderator
let community_moderator_form = CommunityModeratorForm {
// The community creator becomes a moderator
let community_moderator_form = CommunityModeratorForm {
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
if blocking(context.pool(), join).await?.is_err() {
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
if blocking(context.pool(), join).await?.is_err() {
- return Err(ApiError::err("community_moderator_already_exists").into());
+ return Err(ApiError::err_plain("community_moderator_already_exists").into());
}
// Follow your own community
}
// Follow your own community
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
if blocking(context.pool(), follow).await?.is_err() {
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
if blocking(context.pool(), follow).await?.is_err() {
- return Err(ApiError::err("community_follower_already_exists").into());
+ return Err(ApiError::err_plain("community_follower_already_exists").into());
}
let person_id = local_user_view.person.id;
}
let person_id = local_user_view.person.id;
// Make sure deleter is the top mod
if local_user_view.person.id != community_mods[0].moderator.id {
// Make sure deleter is the top mod
if local_user_view.person.id != community_mods[0].moderator.id {
- return Err(ApiError::err("no_community_edit_allowed").into());
+ return Err(ApiError::err_plain("no_community_edit_allowed").into());
Community::update_deleted(conn, community_id, deleted)
})
.await?
Community::update_deleted(conn, community_id, deleted)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_community"))?;
+ .map_err(|e| ApiError::err("couldnt_update_community", e))?;
// Send apub messages
send_apub_delete(
// Send apub messages
send_apub_delete(
Community::update_removed(conn, community_id, removed)
})
.await?
Community::update_removed(conn, community_id, removed)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_community"))?;
+ .map_err(|e| ApiError::err("couldnt_update_community", e))?;
// Mod tables
let expires = data.expires.map(naive_from_unix);
// Mod tables
let expires = data.expires.map(naive_from_unix);
ObjectId::<Community>::new(community_actor_id)
.dereference(context, &mut 0)
.await
ObjectId::<Community>::new(community_actor_id)
.dereference(context, &mut 0)
.await
- .map_err(|_| ApiError::err("couldnt_find_community"))?
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?
CommunityView::read(conn, community_id, person_id)
})
.await?
CommunityView::read(conn, community_id, person_id)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_community"))?;
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?;
// Blank out deleted or removed info
if community_view.community.deleted || community_view.community.removed {
// Blank out deleted or removed info
if community_view.community.deleted || community_view.community.removed {
CommunityModeratorView::for_community(conn, community_id)
})
.await?
CommunityModeratorView::for_community(conn, community_id)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_community"))?;
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?;
let online = context
.chat_server()
let online = context
.chat_server()
})
.await??;
if !mods.contains(&local_user_view.person.id) {
})
.await??;
if !mods.contains(&local_user_view.person.id) {
- return Err(ApiError::err("not_a_moderator").into());
+ return Err(ApiError::err_plain("not_a_moderator").into());
}
let community_id = data.community_id;
}
let community_id = data.community_id;
Community::update(conn, community_id, &community_form)
})
.await?
Community::update(conn, community_id, &community_form)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_community"))?;
+ .map_err(|e| ApiError::err("couldnt_update_community", e))?;
UpdateCommunity::send(&updated_community, &local_user_view.person, context).await?;
UpdateCommunity::send(&updated_community, &local_user_view.person, context).await?;
honeypot_check(&data.honeypot)?;
if !is_valid_post_title(&data.name) {
honeypot_check(&data.honeypot)?;
if !is_valid_post_title(&data.name) {
- return Err(ApiError::err("invalid_post_title").into());
+ return Err(ApiError::err_plain("invalid_post_title").into());
}
check_community_ban(local_user_view.person.id, data.community_id, context.pool()).await?;
}
check_community_ban(local_user_view.person.id, data.community_id, context.pool()).await?;
- return Err(ApiError::err(err_type).into());
+ return Err(ApiError::err(err_type, e).into());
Ok(Post::update_ap_id(conn, inserted_post_id, apub_id)?)
})
.await?
Ok(Post::update_ap_id(conn, inserted_post_id, apub_id)?)
})
.await?
- .map_err(|_| ApiError::err("couldnt_create_post"))?;
+ .map_err(|e| ApiError::err("couldnt_create_post", e))?;
CreateOrUpdatePost::send(
&updated_post,
CreateOrUpdatePost::send(
&updated_post,
let like = move |conn: &'_ _| PostLike::like(conn, &like_form);
if blocking(context.pool(), like).await?.is_err() {
let like = move |conn: &'_ _| PostLike::like(conn, &like_form);
if blocking(context.pool(), like).await?.is_err() {
- return Err(ApiError::err("couldnt_like_post").into());
+ return Err(ApiError::err_plain("couldnt_like_post").into());
}
// Mark the post as read
}
// Mark the post as read
// Verify that only the creator can delete
if !Post::is_post_creator(local_user_view.person.id, orig_post.creator_id) {
// Verify that only the creator can delete
if !Post::is_post_creator(local_user_view.person.id, orig_post.creator_id) {
- return Err(ApiError::err("no_post_edit_allowed").into());
+ return Err(ApiError::err_plain("no_post_edit_allowed").into());
PostView::read(conn, id, person_id)
})
.await?
PostView::read(conn, id, person_id)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_post"))?;
+ .map_err(|e| ApiError::err("couldnt_find_post", e))?;
// Blank out deleted info
if post_view.post.deleted || post_view.post.removed {
// Blank out deleted info
if post_view.post.deleted || post_view.post.removed {
CommunityView::read(conn, community_id, person_id)
})
.await?
CommunityView::read(conn, community_id, person_id)
})
.await?
- .map_err(|_| ApiError::err("couldnt_find_community"))?;
+ .map_err(|e| ApiError::err("couldnt_find_community", e))?;
// Blank out deleted or removed info
if community_view.community.deleted || community_view.community.removed {
// Blank out deleted or removed info
if community_view.community.deleted || community_view.community.removed {
- .map_err(|_| ApiError::err("couldnt_get_posts"))?;
+ .map_err(|e| ApiError::err("couldnt_get_posts", e))?;
// Blank out deleted or removed info
for pv in posts
// Blank out deleted or removed info
for pv in posts
if let Some(name) = &data.name {
if !is_valid_post_title(name) {
if let Some(name) = &data.name {
if !is_valid_post_title(name) {
- return Err(ApiError::err("invalid_post_title").into());
+ return Err(ApiError::err_plain("invalid_post_title").into());
// Verify that only the creator can edit
if !Post::is_post_creator(local_user_view.person.id, orig_post.creator_id) {
// Verify that only the creator can edit
if !Post::is_post_creator(local_user_view.person.id, orig_post.creator_id) {
- return Err(ApiError::err("no_post_edit_allowed").into());
+ return Err(ApiError::err_plain("no_post_edit_allowed").into());
}
// Fetch post links and Pictrs cached image
}
// Fetch post links and Pictrs cached image
- return Err(ApiError::err(err_type).into());
+ return Err(ApiError::err(err_type, e).into());
.await?
{
Ok(private_message) => private_message,
.await?
{
Ok(private_message) => private_message,
- Err(_e) => {
- return Err(ApiError::err("couldnt_create_private_message").into());
+ Err(e) => {
+ return Err(ApiError::err("couldnt_create_private_message", e).into());
- .map_err(|_| ApiError::err("couldnt_create_private_message"))?;
+ .map_err(|e| ApiError::err("couldnt_create_private_message", e))?;
CreateOrUpdatePrivateMessage::send(
&updated_private_message,
CreateOrUpdatePrivateMessage::send(
&updated_private_message,
})
.await??;
if local_user_view.person.id != orig_private_message.creator_id {
})
.await??;
if local_user_view.person.id != orig_private_message.creator_id {
- return Err(ApiError::err("no_private_message_edit_allowed").into());
+ return Err(ApiError::err_plain("no_private_message_edit_allowed").into());
PrivateMessage::update_deleted(conn, private_message_id, deleted)
})
.await?
PrivateMessage::update_deleted(conn, private_message_id, deleted)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_private_message"))?;
+ .map_err(|e| ApiError::err("couldnt_update_private_message", e))?;
// Send the apub update
if data.deleted {
// Send the apub update
if data.deleted {
})
.await??;
if local_user_view.person.id != orig_private_message.creator_id {
})
.await??;
if local_user_view.person.id != orig_private_message.creator_id {
- return Err(ApiError::err("no_private_message_edit_allowed").into());
+ return Err(ApiError::err_plain("no_private_message_edit_allowed").into());
PrivateMessage::update_content(conn, private_message_id, &content_slurs_removed)
})
.await?
PrivateMessage::update_content(conn, private_message_id, &content_slurs_removed)
})
.await?
- .map_err(|_| ApiError::err("couldnt_update_private_message"))?;
+ .map_err(|e| ApiError::err("couldnt_update_private_message", e))?;
// Send the apub update
CreateOrUpdatePrivateMessage::send(
// Send the apub update
CreateOrUpdatePrivateMessage::send(
let read_site = move |conn: &'_ _| Site::read_simple(conn);
if blocking(context.pool(), read_site).await?.is_ok() {
let read_site = move |conn: &'_ _| Site::read_simple(conn);
if blocking(context.pool(), read_site).await?.is_ok() {
- return Err(ApiError::err("site_already_exists").into());
+ return Err(ApiError::err_plain("site_already_exists").into());
let create_site = move |conn: &'_ _| Site::create(conn, &site_form);
if blocking(context.pool(), create_site).await?.is_err() {
let create_site = move |conn: &'_ _| Site::create(conn, &site_form);
if blocking(context.pool(), create_site).await?.is_err() {
- return Err(ApiError::err("site_already_exists").into());
+ return Err(ApiError::err_plain("site_already_exists").into());
}
let site_view = blocking(context.pool(), SiteView::read).await??;
}
let site_view = blocking(context.pool(), SiteView::read).await??;
CommunityFollowerView::for_person(conn, person_id)
})
.await?
CommunityFollowerView::for_person(conn, person_id)
})
.await?
- .map_err(|_| ApiError::err("system_err_login"))?;
+ .map_err(|e| ApiError::err("system_err_login", e))?;
let person_id = local_user_view.person.id;
let community_blocks = blocking(context.pool(), move |conn| {
CommunityBlockView::for_person(conn, person_id)
})
.await?
let person_id = local_user_view.person.id;
let community_blocks = blocking(context.pool(), move |conn| {
CommunityBlockView::for_person(conn, person_id)
})
.await?
- .map_err(|_| ApiError::err("system_err_login"))?;
+ .map_err(|e| ApiError::err("system_err_login", e))?;
let person_id = local_user_view.person.id;
let person_blocks = blocking(context.pool(), move |conn| {
PersonBlockView::for_person(conn, person_id)
})
.await?
let person_id = local_user_view.person.id;
let person_blocks = blocking(context.pool(), move |conn| {
PersonBlockView::for_person(conn, person_id)
})
.await?
- .map_err(|_| ApiError::err("system_err_login"))?;
+ .map_err(|e| ApiError::err("system_err_login", e))?;
let moderates = blocking(context.pool(), move |conn| {
CommunityModeratorView::for_person(conn, person_id)
})
.await?
let moderates = blocking(context.pool(), move |conn| {
CommunityModeratorView::for_person(conn, person_id)
})
.await?
- .map_err(|_| ApiError::err("system_err_login"))?;
+ .map_err(|e| ApiError::err("system_err_login", e))?;
Some(MyUserInfo {
local_user_view,
Some(MyUserInfo {
local_user_view,
};
let update_site = move |conn: &'_ _| Site::update(conn, 1, &site_form);
};
let update_site = move |conn: &'_ _| Site::update(conn, 1, &site_form);
- if blocking(context.pool(), update_site).await?.is_err() {
- return Err(ApiError::err("couldnt_update_site").into());
- }
+ blocking(context.pool(), update_site)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_site", e))?;
let site_view = blocking(context.pool(), SiteView::read).await??;
let site_view = blocking(context.pool(), SiteView::read).await??;
// Make sure site has open registration
if let Ok(site) = blocking(context.pool(), move |conn| Site::read_simple(conn)).await? {
if !site.open_registration {
// Make sure site has open registration
if let Ok(site) = blocking(context.pool(), move |conn| Site::read_simple(conn)).await? {
if !site.open_registration {
- return Err(ApiError::err("registration_closed").into());
+ return Err(ApiError::err_plain("registration_closed").into());
// Make sure passwords match
if data.password != data.password_verify {
// Make sure passwords match
if data.password != data.password_verify {
- return Err(ApiError::err("passwords_dont_match").into());
+ return Err(ApiError::err_plain("passwords_dont_match").into());
}
// Check if there are admins. False if admins exist
}
// Check if there are admins. False if admins exist
- return Err(ApiError::err("captcha_incorrect").into());
+ return Err(ApiError::err_plain("captcha_incorrect").into());
let actor_keypair = generate_actor_keypair()?;
if !is_valid_actor_name(&data.username, context.settings().actor_name_max_length) {
let actor_keypair = generate_actor_keypair()?;
if !is_valid_actor_name(&data.username, context.settings().actor_name_max_length) {
- return Err(ApiError::err("invalid_username").into());
+ return Err(ApiError::err_plain("invalid_username").into());
}
let actor_id = generate_apub_endpoint(
EndpointType::Person,
}
let actor_id = generate_apub_endpoint(
EndpointType::Person,
Person::create(conn, &person_form)
})
.await?
Person::create(conn, &person_form)
})
.await?
- .map_err(|_| ApiError::err("user_already_exists"))?;
+ .map_err(|e| ApiError::err("user_already_exists", e))?;
// Create the local user
// TODO some of these could probably use the DB defaults
// Create the local user
// TODO some of these could probably use the DB defaults
- return Err(ApiError::err(err_type).into());
+ return Err(ApiError::err(err_type, e).into());
};
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
};
let follow = move |conn: &'_ _| CommunityFollower::follow(conn, &community_follower_form);
- if blocking(context.pool(), follow).await?.is_err() {
- return Err(ApiError::err("community_follower_already_exists").into());
- };
+ blocking(context.pool(), follow)
+ .await?
+ .map_err(|e| ApiError::err("community_follower_already_exists", e))?;
// If its an admin, add them as a mod and follower to main
if no_admins {
// If its an admin, add them as a mod and follower to main
if no_admins {
};
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
};
let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
- if blocking(context.pool(), join).await?.is_err() {
- return Err(ApiError::err("community_moderator_already_exists").into());
- }
+ blocking(context.pool(), join)
+ .await?
+ .map_err(|e| ApiError::err("community_moderator_already_exists", e))?;
)
.unwrap_or(false);
if !valid {
)
.unwrap_or(false);
if !valid {
- return Err(ApiError::err("password_incorrect").into());
+ return Err(ApiError::err_plain("password_incorrect").into());
}
// Comments
let person_id = local_user_view.person.id;
let permadelete = move |conn: &'_ _| Comment::permadelete_for_creator(conn, person_id);
}
// Comments
let person_id = local_user_view.person.id;
let permadelete = move |conn: &'_ _| Comment::permadelete_for_creator(conn, person_id);
- if blocking(context.pool(), permadelete).await?.is_err() {
- return Err(ApiError::err("couldnt_update_comment").into());
- }
+ blocking(context.pool(), permadelete)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_comment", e))?;
// Posts
let permadelete = move |conn: &'_ _| Post::permadelete_for_creator(conn, person_id);
// Posts
let permadelete = move |conn: &'_ _| Post::permadelete_for_creator(conn, person_id);
- if blocking(context.pool(), permadelete).await?.is_err() {
- return Err(ApiError::err("couldnt_update_post").into());
- }
+ blocking(context.pool(), permadelete)
+ .await?
+ .map_err(|e| ApiError::err("couldnt_update_post", e))?;
blocking(context.pool(), move |conn| {
Person::delete_account(conn, person_id)
blocking(context.pool(), move |conn| {
Person::delete_account(conn, person_id)
.dereference(context, &mut 0)
.await;
person
.dereference(context, &mut 0)
.await;
person
- .map_err(|_| ApiError::err("couldnt_find_that_username_or_email"))?
+ .map_err(|e| ApiError::err("couldnt_find_that_username_or_email", e))?
Some("") => Ok(Some(None)),
Some(str_url) => match Url::parse(str_url) {
Ok(url) => Ok(Some(Some(url.into()))),
Some("") => Ok(Some(None)),
Some(str_url) => match Url::parse(str_url) {
Ok(url) => Ok(Some(Some(url.into()))),
- Err(_) => Err(ApiError::err("invalid_url")),
+ Err(e) => Err(ApiError::err("invalid_url", e)),
pub mod version;
use http::StatusCode;
pub mod version;
use http::StatusCode;
+use log::warn;
+use std::{fmt, fmt::Display};
use thiserror::Error;
pub type ConnectionId = usize;
use thiserror::Error;
pub type ConnectionId = usize;
#[derive(Debug, Error)]
#[error("{{\"error\":\"{message}\"}}")]
pub struct ApiError {
#[derive(Debug, Error)]
#[error("{{\"error\":\"{message}\"}}")]
pub struct ApiError {
- pub fn err(msg: &str) -> Self {
+ pub fn err_plain(msg: &str) -> Self {
+ ApiError {
+ message: msg.to_string(),
+ }
+ }
+ pub fn err<E: Display>(msg: &str, original_error: E) -> Self {
+ warn!("{}", original_error);
ApiError {
message: msg.to_string(),
}
ApiError {
message: msg.to_string(),
}
-impl std::fmt::Display for LemmyError {
+impl Display for LemmyError {
fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
self.inner.fmt(f)
}
fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
self.inner.fmt(f)
}
}
pub fn check_slurs(text: &str, slur_regex: &Regex) -> Result<(), ApiError> {
}
pub fn check_slurs(text: &str, slur_regex: &Regex) -> Result<(), ApiError> {
- if let Err(slurs) = slur_check(text, slur_regex) {
- Err(ApiError::err(&slurs_vec_to_str(slurs)))
- } else {
- Ok(())
- }
+ slur_check(text, slur_regex)
+ .map_err(|slurs| ApiError::err_plain(&slurs_vec_to_str(slurs.clone())))
}
pub fn check_slurs_opt(text: &Option<String>, slur_regex: &Regex) -> Result<(), ApiError> {
}
pub fn check_slurs_opt(text: &Option<String>, slur_regex: &Regex) -> Result<(), ApiError> {
async move {
let json: Value = serde_json::from_str(&msg.msg)?;
let data = &json["data"].to_string();
async move {
let json: Value = serde_json::from_str(&msg.msg)?;
let data = &json["data"].to_string();
- let op = &json["op"].as_str().ok_or(ApiError {
- message: "Unknown op type".to_string(),
- })?;
+ let op = &json["op"]
+ .as_str()
+ .ok_or_else(|| ApiError::err_plain("missing op"))?;
if let Ok(user_operation_crud) = UserOperationCrud::from_str(op) {
let fut = (message_handler_crud)(context, msg.id, user_operation_crud.clone(), data);
if let Ok(user_operation_crud) = UserOperationCrud::from_str(op) {
let fut = (message_handler_crud)(context, msg.id, user_operation_crud.clone(), data);