plugin-comment-widget icon indicating copy to clipboard operation
plugin-comment-widget copied to clipboard

头像服务增强:增加一些国内常用的头像服务商

Open wenjing-xin opened this issue 1 year ago • 4 comments
trafficstars

按照已有头像服务的示例,对头像服务源进行增加。

此外我看了下 AvatarProvider 这个类,只有头像服务源名称和服务地址,没有一些参数配置,比如用户想要配置头像源的默认头像以及图片大小还有其他参数,是否可以在 AvatarProvider 这个类中新加一个参数 _params 让用户自己配置一些头像源请求的参数?如有必要,我在重新修改代码,再次提交代码

wenjing-xin avatar Jul 29 '24 02:07 wenjing-xin

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

f2c-ci-robot[bot] avatar Jul 29 '24 02:07 f2c-ci-robot[bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign johnniang for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

f2c-ci-robot[bot] avatar Jul 29 '24 02:07 f2c-ci-robot[bot]

没有一些参数配置,比如用户想要配置头像源的默认头像以及图片大小还有其他参数,是否可以在 AvatarProvider 这个类中新加一个参数 _params 让用户自己配置一些头像源请求的参数

如果要考虑到支持多个头像源,可能现有的 Gravatar 的配置得改一下,应该将具体的头像源配置用 group(对象)来组织。

image

期望的结构应该是:

{
   provider: "gravatar",
   gravatar: { mirror: "", default: 'monsterid'  },
   cravatar: { ... }
}

ruibaby avatar Jul 31 '24 04:07 ruibaby

没有一些参数配置,比如用户想要配置头像源的默认头像以及图片大小还有其他参数,是否可以在 AvatarProvider 这个类中新加一个参数 _params 让用户自己配置一些头像源请求的参数

如果要考虑到支持多个头像源,可能现有的 Gravatar 的配置得改一下,应该将具体的头像源配置用 group(对象)来组织。

image 期望的结构应该是:
{
   provider: "gravatar",
   gravatar: { mirror: "", default: 'monsterid'  },
   cravatar: { ... }
}

我还是没听明白 请问一下 配置文件的这个yaml组织结构到底需要怎么组织?

wenjing-xin avatar Aug 06 '24 14:08 wenjing-xin

又看了一下这个 PR,看起来 https://cravatar.cn 和 https://weavatar.com 是兼容 Gravatar 的(镜像),并且链接格式一致,所以建议直接配置 头像服务镜像地址 即可,短期内没有必要再提供额外的选项。

ruibaby avatar Mar 27 '25 09:03 ruibaby