use crate::PerformCrud;
use actix_web::web::Data;
use lemmy_api_common::{
+ build_response::build_post_response,
+ context::LemmyContext,
post::{CreatePost, PostResponse},
request::fetch_site_data,
utils::{
- blocking,
check_community_ban,
check_community_deleted_or_removed,
- get_local_user_view_from_jwt,
+ generate_local_apub_endpoint,
honeypot_check,
local_site_to_slur_regex,
+ local_user_view_from_jwt,
mark_post_as_read,
+ EndpointType,
},
};
-use lemmy_apub::{
- generate_local_apub_endpoint,
- objects::post::ApubPost,
- protocol::activities::{create_or_update::post::CreateOrUpdatePost, CreateOrUpdateType},
- EndpointType,
-};
use lemmy_db_schema::{
impls::actor_language::default_post_language,
source::{
};
use lemmy_db_views_actor::structs::CommunityView;
use lemmy_utils::{
- error::LemmyError,
- utils::{check_slurs, check_slurs_opt, clean_url_params, is_valid_post_title},
- ConnectionId,
+ error::{LemmyError, LemmyErrorExt, LemmyErrorType},
+ spawn_try_task,
+ utils::{
+ slurs::{check_slurs, check_slurs_opt},
+ validation::{check_url_scheme, clean_url_params, is_valid_body_field, is_valid_post_title},
+ },
+ SYNCHRONOUS_FEDERATION,
};
-use lemmy_websocket::{send::send_post_ws_message, LemmyContext, UserOperationCrud};
-use tracing::{warn, Instrument};
+use tracing::Instrument;
use url::Url;
use webmention::{Webmention, WebmentionError};
impl PerformCrud for CreatePost {
type Response = PostResponse;
- #[tracing::instrument(skip(context, websocket_id))]
- async fn perform(
- &self,
- context: &Data<LemmyContext>,
- websocket_id: Option<ConnectionId>,
- ) -> Result<PostResponse, LemmyError> {
+ #[tracing::instrument(skip(context))]
+ async fn perform(&self, context: &Data<LemmyContext>) -> Result<PostResponse, LemmyError> {
let data: &CreatePost = self;
- let local_user_view =
- get_local_user_view_from_jwt(&data.auth, context.pool(), context.secret()).await?;
- let local_site = blocking(context.pool(), LocalSite::read).await??;
+ let local_user_view = local_user_view_from_jwt(&data.auth, context).await?;
+ let local_site = LocalSite::read(&mut context.pool()).await?;
let slur_regex = local_site_to_slur_regex(&local_site);
check_slurs(&data.name, &slur_regex)?;
let data_url = data.url.as_ref();
let url = data_url.map(clean_url_params).map(Into::into); // TODO no good way to handle a "clear"
- if !is_valid_post_title(&data.name) {
- return Err(LemmyError::from_message("invalid_post_title"));
- }
+ is_valid_post_title(&data.name)?;
+ is_valid_body_field(&data.body, true)?;
+ check_url_scheme(&data.url)?;
- check_community_ban(local_user_view.person.id, data.community_id, context.pool()).await?;
- check_community_deleted_or_removed(data.community_id, context.pool()).await?;
+ check_community_ban(
+ local_user_view.person.id,
+ data.community_id,
+ &mut context.pool(),
+ )
+ .await?;
+ check_community_deleted_or_removed(data.community_id, &mut context.pool()).await?;
let community_id = data.community_id;
- let community = blocking(context.pool(), move |conn| {
- Community::read(conn, community_id)
- })
- .await??;
+ let community = Community::read(&mut context.pool(), community_id).await?;
if community.posting_restricted_to_mods {
let community_id = data.community_id;
- let is_mod = blocking(context.pool(), move |conn| {
- CommunityView::is_mod_or_admin(conn, local_user_view.local_user.person_id, community_id)
- })
+ let is_mod = CommunityView::is_mod_or_admin(
+ &mut context.pool(),
+ local_user_view.local_user.person_id,
+ community_id,
+ )
.await?;
if !is_mod {
- return Err(LemmyError::from_message("only_mods_can_post_in_community"));
+ return Err(LemmyErrorType::OnlyModsCanPostInCommunity)?;
}
}
// Fetch post links and pictrs cached image
let (metadata_res, thumbnail_url) =
- fetch_site_data(context.client(), context.settings(), data_url).await;
+ fetch_site_data(context.client(), context.settings(), data_url, true).await;
let (embed_title, embed_description, embed_video_url) = metadata_res
.map(|u| (u.title, u.description, u.embed_video_url))
.unwrap_or_default();
let language_id = match data.language_id {
Some(lid) => Some(lid),
None => {
- blocking(context.pool(), move |conn| {
- default_post_language(conn, community_id, local_user_view.local_user.id)
- })
- .await??
+ default_post_language(
+ &mut context.pool(),
+ community_id,
+ local_user_view.local_user.id,
+ )
+ .await?
}
};
- blocking(context.pool(), move |conn| {
- CommunityLanguage::is_allowed_community_language(conn, language_id, community_id)
- })
- .await??;
+ CommunityLanguage::is_allowed_community_language(
+ &mut context.pool(),
+ language_id,
+ community_id,
+ )
+ .await?;
let post_form = PostInsertForm::builder()
.name(data.name.trim().to_owned())
.url(url)
- .body(data.body.to_owned())
+ .body(data.body.clone())
.community_id(data.community_id)
.creator_id(local_user_view.person.id)
.nsfw(data.nsfw)
.thumbnail_url(thumbnail_url)
.build();
- let inserted_post =
- match blocking(context.pool(), move |conn| Post::create(conn, &post_form)).await? {
- Ok(post) => post,
- Err(e) => {
- let err_type = if e.to_string() == "value too long for type character varying(200)" {
- "post_title_too_long"
- } else {
- "couldnt_create_post"
- };
-
- return Err(LemmyError::from_error_message(e, err_type));
- }
- };
+ let inserted_post = Post::create(&mut context.pool(), &post_form)
+ .await
+ .with_lemmy_type(LemmyErrorType::CouldntCreatePost)?;
let inserted_post_id = inserted_post.id;
let protocol_and_hostname = context.settings().get_protocol_and_hostname();
- let updated_post = blocking(context.pool(), move |conn| -> Result<Post, LemmyError> {
- let apub_id = generate_local_apub_endpoint(
- EndpointType::Post,
- &inserted_post_id.to_string(),
- &protocol_and_hostname,
- )?;
- Ok(Post::update(
- conn,
- inserted_post_id,
- &PostUpdateForm::builder().ap_id(Some(apub_id)).build(),
- )?)
- })
- .await?
- .map_err(|e| e.with_message("couldnt_create_post"))?;
+ let apub_id = generate_local_apub_endpoint(
+ EndpointType::Post,
+ &inserted_post_id.to_string(),
+ &protocol_and_hostname,
+ )?;
+ let updated_post = Post::update(
+ &mut context.pool(),
+ inserted_post_id,
+ &PostUpdateForm::builder().ap_id(Some(apub_id)).build(),
+ )
+ .await
+ .with_lemmy_type(LemmyErrorType::CouldntCreatePost)?;
// They like their own post by default
let person_id = local_user_view.person.id;
score: 1,
};
- let like = move |conn: &mut _| PostLike::like(conn, &like_form);
- blocking(context.pool(), like)
- .await?
- .map_err(|e| LemmyError::from_error_message(e, "couldnt_like_post"))?;
+ PostLike::like(&mut context.pool(), &like_form)
+ .await
+ .with_lemmy_type(LemmyErrorType::CouldntLikePost)?;
// Mark the post as read
- mark_post_as_read(person_id, post_id, context.pool()).await?;
-
- if let Some(url) = &updated_post.url {
- let mut webmention =
- Webmention::new::<Url>(updated_post.ap_id.clone().into(), url.clone().into())?;
- webmention.set_checked(true);
- match webmention
- .send()
- .instrument(tracing::info_span!("Sending webmention"))
- .await
- {
- Ok(_) => {}
- Err(WebmentionError::NoEndpointDiscovered(_)) => {}
- Err(e) => warn!("Failed to send webmention: {}", e),
+ mark_post_as_read(person_id, post_id, &mut context.pool()).await?;
+
+ if let Some(url) = updated_post.url.clone() {
+ let task = async move {
+ let mut webmention =
+ Webmention::new::<Url>(updated_post.ap_id.clone().into(), url.clone().into())?;
+ webmention.set_checked(true);
+ match webmention
+ .send()
+ .instrument(tracing::info_span!("Sending webmention"))
+ .await
+ {
+ Err(WebmentionError::NoEndpointDiscovered(_)) => Ok(()),
+ Ok(_) => Ok(()),
+ Err(e) => Err(e).with_lemmy_type(LemmyErrorType::CouldntSendWebmention),
+ }
+ };
+ if *SYNCHRONOUS_FEDERATION {
+ task.await?;
+ } else {
+ spawn_try_task(task);
}
- }
-
- let apub_post: ApubPost = updated_post.into();
- CreateOrUpdatePost::send(
- apub_post.clone(),
- &local_user_view.person.clone().into(),
- CreateOrUpdateType::Create,
- context,
- )
- .await?;
+ };
- send_post_ws_message(
- inserted_post.id,
- UserOperationCrud::CreatePost,
- websocket_id,
- Some(local_user_view.person.id),
- context,
- )
- .await
+ build_post_response(context, community_id, person_id, post_id).await
}
}