Rework User.Import to spin off a bunch of jobs instead of single-threading the task #306

Closed
opened 2022-11-24 11:51:42 +00:00 by floatingghost · 0 comments

From a bug raised on IRC

:background worker has a default timeout of 5s, which should be fine, but it handles mute/block/follow imports - this potentially has to make n HTTP requests which will easily exceed the timeout

instead, the 5s should be ok, but Import should instead spin off tasks to handle each entry instead of one big thingy

https://akkoma.dev/AkkomaGang/akkoma/src/branch/develop/lib/pleroma/user/import.ex#L43

From a bug raised on IRC :background worker has a default timeout of 5s, which should be fine, but it handles mute/block/follow imports - this potentially has to make *n* HTTP requests which will easily exceed the timeout instead, the 5s should be ok, but Import should instead spin off tasks to handle each entry instead of one big thingy https://akkoma.dev/AkkomaGang/akkoma/src/branch/develop/lib/pleroma/user/import.ex#L43
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AkkomaGang/akkoma#306
No description provided.