fix: align follow event with accepted state; redact internal error details
This commit is contained in:
@@ -37,7 +37,7 @@ pub async fn follow_user(follows: &dyn FollowRepository, events: &dyn EventPubli
|
|||||||
if follower_id == following_id { return Err(DomainError::InvalidInput("cannot follow yourself".into())); }
|
if follower_id == following_id { return Err(DomainError::InvalidInput("cannot follow yourself".into())); }
|
||||||
let follow = Follow { follower_id: follower_id.clone(), following_id: following_id.clone(), state: FollowState::Accepted, ap_id: None, created_at: Utc::now() };
|
let follow = Follow { follower_id: follower_id.clone(), following_id: following_id.clone(), state: FollowState::Accepted, ap_id: None, created_at: Utc::now() };
|
||||||
follows.save(&follow).await?;
|
follows.save(&follow).await?;
|
||||||
events.publish(&DomainEvent::FollowRequested { follower_id: follower_id.clone(), following_id: following_id.clone() }).await?;
|
events.publish(&DomainEvent::FollowAccepted { follower_id: follower_id.clone(), following_id: following_id.clone() }).await?;
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -20,7 +20,7 @@ impl IntoResponse for ApiError {
|
|||||||
Self::Domain(DomainError::Forbidden) => (StatusCode::FORBIDDEN, "forbidden".into()),
|
Self::Domain(DomainError::Forbidden) => (StatusCode::FORBIDDEN, "forbidden".into()),
|
||||||
Self::Domain(DomainError::Conflict(m)) => (StatusCode::CONFLICT, m),
|
Self::Domain(DomainError::Conflict(m)) => (StatusCode::CONFLICT, m),
|
||||||
Self::Domain(DomainError::InvalidInput(m)) => (StatusCode::UNPROCESSABLE_ENTITY, m),
|
Self::Domain(DomainError::InvalidInput(m)) => (StatusCode::UNPROCESSABLE_ENTITY, m),
|
||||||
Self::Domain(DomainError::Internal(m)) => (StatusCode::INTERNAL_SERVER_ERROR, m),
|
Self::Domain(DomainError::Internal(_)) => (StatusCode::INTERNAL_SERVER_ERROR, "internal server error".into()),
|
||||||
Self::Unauthorized => (StatusCode::UNAUTHORIZED, "unauthorized".into()),
|
Self::Unauthorized => (StatusCode::UNAUTHORIZED, "unauthorized".into()),
|
||||||
Self::BadRequest(m) => (StatusCode::BAD_REQUEST, m),
|
Self::BadRequest(m) => (StatusCode::BAD_REQUEST, m),
|
||||||
};
|
};
|
||||||
|
|||||||
Reference in New Issue
Block a user