Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

What is the contributors table link based on? #323

Open
LPeter1997 opened this issue Jan 8, 2022 · 4 comments
Open

What is the contributors table link based on? #323

LPeter1997 opened this issue Jan 8, 2022 · 4 comments

Comments

@LPeter1997
Copy link

I don't really know what the bot inserts as a link for a given entry in the contributors table (I see that it's the "profile" link in the JSON). Many times it's their GitHub profile or a personal page. But recently it managed to insert an NSFW website with a similar link as the name of my contributors username. Can I make sure it's always the GitHub page instead?

@LPeter1997 LPeter1997 added the bug label Jan 8, 2022
@gr2m
Copy link
Contributor

gr2m commented Jan 11, 2022

Can you link to the commit that made the change?

@LPeter1997
Copy link
Author

Sure: LanguageDev/Yoakke@a7a64a3

After this, I'd much prefer a direct link to the GH profile instead of any custom homepage.

@gr2m
Copy link
Contributor

gr2m commented Jan 13, 2022

I actually have no idea where this URL came from, I'd need to dig into the code myself and don't have the time right now. Maybe the user had furesoft.ml configured as their profile website and if the website is present, that's what the all-contributors app is using?

@wkillerud
Copy link

I was wondering the same thing here today, and I think I found it. github.js and its getUserInfo function automatically prefers the blog field over html_url if the former is a valid URL (GitHub API docs for Users). From what I can tell, the GitLab version only does profile links, not other websites (GitLab API docs for Users).

I would have expected these links to only ever point to the GitHub profile (html_url). As it is now, my avatar in fremtind/jokul links to a domain I'm no longer renewing. I did not expect to have to update the URL any other place than my GitHub profile.

My suggestion would be to change the CLI so that the profile field returned by getUserInfo() only ever returns html_url. For users who want the current behavior, the function could be expanded to also return a blog value which can behave like profile does today (fall back to html_url). That way they can use a custom contributorTemplate that uses contributor.blog instead of contributor.profile.

@tenshiAMD tenshiAMD transferred this issue from all-contributors/app Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants