Use Libravatar instead #92

Merged
yarmo merged 1 commits from ilpianista/keyoxide-web:feature/libravatar into main 2 weeks ago

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

We push for decentralization, thus we shouldn't rely over Gravatar.
ilpianista added 1 commit 1 month ago
ilpianista force-pushed feature/libravatar from e83faf0953 to d7259aa825 1 month ago

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 2 weeks ago
Owner

Looks good, will test today.

Looks good, will test today.
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!
yarmo merged commit 6cfcc37910 into main 2 weeks ago
ilpianista deleted branch feature/libravatar 2 weeks ago
The pull request has been merged as 6cfcc37910.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.