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

New issue

Closed

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

floatingghost commented

2022-11-24 11:51:42 +00:00

Owner

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

👍 1