notfire/Iceshrimp.NET
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[backend/federation] Make resolvedActorId / authenticatedUserId mismatch a debug message instead of an exception

Browse source
This commit is contained in:
Laura Hausmann 2024-05-03 16:30:03 +02:00
parent db7e51358f
commit 477f300c40
No known key found for this signature in database
GPG key ID: D044E84C5BE01605
1 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
Iceshrimp.Backend/Core/Federation/ActivityPub/ActivityHandlerService.cs
View file

@ -40,11 +40,15 @@ public class ActivityHandlerService(
var resolvedActor = await userResolver.ResolveAsync(activity.Actor.Id);
if (authenticatedUserId == null)
throw GracefulException
.UnprocessableEntity("Refusing to process activity without authenticatedUserId");
if (resolvedActor.Id != authenticatedUserId && authenticatedUserId != null)
throw GracefulException
.UnprocessableEntity($"Authenticated user id {authenticatedUserId} doesn't match resolved actor id {resolvedActor.Id}");
throw GracefulException.UnprocessableEntity("Refusing to process activity without authenticatedUserId");
if (resolvedActor.Id != authenticatedUserId)
{
logger.LogDebug("Authenticated user id {authenticatedUserId} doesn't match resolved actor id {resolvedActorId}, skipping",
authenticatedUserId, resolvedActor.Id);
return;
}
if (new Uri(activity.Actor.Id).Host != new Uri(activity.Id).Host)
throw GracefulException
.UnprocessableEntity($"Activity identifier ({activity.Actor.Id}) host doesn't match actor identifier ({activity.Id}) host");