Use futures and tokio to send activities #620

Sapludināts

elegaanz sapludināja 4 revīzijas no async-federation uz master

elegaanz " komentēja " (Migrēts no github.com)

Fixes #252

Publishing an article and having it federated to the whole fediverse only takes a few seconds now (instead of minutes before, on big Plume instances connected to many other ones).

Fixes #252 Publishing an article and having it federated to the whole fediverse only takes a few seconds now (instead of minutes before, on big Plume instances connected to many other ones).

igalic (Migrēts no github.com) recenzēja

igalic (Migrēts no github.com) atstāja komentāru

👀

plume-common/src/activity_pub/mod.rs

igalic (Migrēts no github.com) " komentēja "

is this comment still relevant?

igalic (Migrēts no github.com) " komentēja "

if this fails, we panic with "new rt".??

igalic (Migrēts no github.com) " komentēja "

i think we should talk about our logging strategy sometime 😅

elegaanz (Migrēts no github.com) recenzēja

plume-common/src/activity_pub/mod.rs

elegaanz (Migrēts no github.com) " komentēja "

Not really, but it is for the background jobs in general, for the moment we just spawn them in a separate thread and that's it. I will remove it and create a more general issue.

elegaanz (Migrēts no github.com) recenzēja

plume-common/src/activity_pub/mod.rs

elegaanz (Migrēts no github.com) " komentēja "

Hum… yes, I could probably change that. To be honest I copied this line from reqwest examples without really thinking about it 😅

Hum… yes, I could probably change that. To be honest I copied this line from reqwest examples without really thinking about it :sweat_smile:

elegaanz (Migrēts no github.com) recenzēja

plume-common/src/activity_pub/mod.rs

elegaanz (Migrēts no github.com) " komentēja "

Indeed, we should probably use log and env_logger or something like that. Should I create an issue?

Indeed, we should probably use `log` and `env_logger` or something like that. Should I create an issue?

codecov[bot] " komentēja " (Migrēts no github.com)

Codecov Report

Merging #620 into master will increase coverage by 0.08%.
The diff coverage is 26.66%.

@@            Coverage Diff             @@
##           master     #620      +/-   ##
==========================================
+ Coverage   35.28%   35.37%   +0.08%     
==========================================
  Files          68       68              
  Lines        7907     7904       -3     
  Branches     1893     1887       -6     
==========================================
+ Hits         2790     2796       +6     
+ Misses       4345     4338       -7     
+ Partials      772      770       -2

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/620?src=pr&el=h1) Report > Merging [#620](https://codecov.io/gh/Plume-org/Plume/pull/620?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/6dbc043a7e4d1757851599859683804bf36a3409?src=pr&el=desc) will **increase** coverage by `0.08%`. > The diff coverage is `26.66%`. ```diff @@ Coverage Diff @@ ## master #620 +/- ## ========================================== + Coverage 35.28% 35.37% +0.08% ========================================== Files 68 68 Lines 7907 7904 -3 Branches 1893 1887 -6 ========================================== + Hits 2790 2796 +6 + Misses 4345 4338 -7 + Partials 772 770 -2 ```

trinity-1686a recenzēja

trinity-1686a atstāja komentāru

Īpašnieks

👀

plume-common/src/activity_pub/mod.rs

trinity-1686a " komentēja "

Īpašnieks

I think this print should be done in the map just under, otherwise output could get mixed quickly, making both logs useless

plume-common/src/activity_pub/mod.rs

trinity-1686a " komentēja "

Īpašnieks

(what is that comma for?)

elegaanz (Migrēts no github.com) recenzēja

plume-common/src/activity_pub/mod.rs

elegaanz (Migrēts no github.com) " komentēja "

(no idea, I will remove it)

elegaanz (Migrēts no github.com) recenzēja

plume-common/src/activity_pub/mod.rs

elegaanz (Migrēts no github.com) " komentēja "

Actually, it is added by cargo-fmt, because this whole expression is the argument of a function.

trinity-1686a recenzēja

plume-common/src/activity_pub/mod.rs

trinity-1686a " komentēja "

Īpašnieks

okay...

trinity-1686a apstiprināja izmaiņas

Recenzenti

trinity-1686a apstiprināja izmaiņas

Izmaiņu pieprasījums tika sapludināts ar revīziju 7ea4acc289.

Varat aplūkot arī komandrindas instrukcijas.

Solis 1:

Projekta repozitorijā izveidojiet jaunu jaunu atzaru un pārbaudiet savas izmaiņas.

git checkout -b async-federation master

git pull origin async-federation

Solis 2:

Sapludināt izmaiņas un atjaunot tās Forgejo.

git checkout master

git merge --no-ff async-federation

git push origin master

Pierakstieties, lai pievienotos šai sarunai.

Nav recenzentu

Nav atskaites punktu

Nav atbildīgo

2 dalībnieki

Paziņojumi

Izpildes termiņš

Datums līdz nav korekts. Izmantojiet formātu 'gggg-mm-dd'.

Izpildes termiņš nav uzstādīts.

Atkarības

Nav atkarību.

Notiek ielāde…

Vēl nav satura.

Etiķetes Atskaites punkti

Use futures and tokio to send activities #620

Codecov Report

Recenzenti

Solis 1:

Solis 2: