Use Libravatar instead #92

Merged

yarmo merged 1 commits from :feature/libravatar into main

2021-06-01 09:53:13 +00:00

ilpianista commented

2021-05-05 11:51:46 +00:00

We push for decentralization, thus we shouldn't rely over Gravatar.

👍 1

ilpianista force-pushed feature/libravatar from e83faf0953 to d7259aa825

2021-05-05 21:45:05 +00:00

Compare

wiktor commented

2021-05-06 13:40:33 +00:00

Seems simple enough 👍

@yarmo please test with various e-mails. When I did OpenPGP proofs years ago I also evaluated libravatar but the performance characteristics were sub-par. (Sometimes libravatar servers timeouted or took long time to reply). If this is federated with no single point of failure then it's double-cool!

Seems simple enough 👍 @yarmo please test with various e-mails. When I did [OpenPGP proofs](https://github.com/wiktor-k/openpgp-proofs) years ago I also evaluated libravatar but the performance characteristics were sub-par. (Sometimes libravatar servers timeouted or took long time to reply). If this is federated with no single point of failure then it's double-cool!

yarmo added the

enhancement

label 2021-05-31 09:09:38 +00:00

yarmo commented

2021-05-31 09:10:17 +00:00

Owner

Looks good, will test today.

yarmo commented

2021-06-01 09:53:01 +00:00

Owner

Tested today, works great.

Avatars take slightly longer to load than was the case gravatar but this IMO does not outweigh the benefits of using a "federatable" and FOSS solution,
I did not test the federation aspect but trust that the libravatar library will handle it just fine.

With that, PR is merged. Thanks @ilpianista for a great contribution!

Tested today, works great. - Avatars take slightly longer to load than was the case gravatar but this IMO does not outweigh the benefits of using a "federatable" and FOSS solution, - I did not test the federation aspect but trust that the libravatar library will handle it just fine. With that, PR is merged. Thanks @ilpianista for a great contribution!

🎉 1

yarmo referenced this issue from a commit

2021-06-01 09:53:13 +00:00

Merge pull request 'Use Libravatar instead' (#92) from ilpianista/keyoxide-web:feature/libravatar into main

yarmo merged commit 6cfcc37910 into main

2021-06-01 09:53:13 +00:00