Skip to content

Conversation

@pcuenca
Copy link
Member

@pcuenca pcuenca commented Dec 18, 2025

Re: #3234.

@pcuenca pcuenca marked this pull request as draft December 18, 2025 13:27
Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Did you use a script for it?

@pcuenca
Copy link
Member Author

pcuenca commented Dec 18, 2025

script

Yes, I told cursor about the /user/{username}/overview API, instructed it to look for 307s and fix those, and compile a list for 404s. I also told it to be careful with rate limits and gave it a disposable token - it knew how to read the headers but didn't do anything with them lol. It created this script:

check_authors.py

I verified all the 307s and then went through the 404s and found some replacements.

@pcuenca pcuenca marked this pull request as ready for review December 18, 2025 15:27
@pcuenca
Copy link
Member Author

pcuenca commented Dec 18, 2025

Fixed the 307s and some of the 404s I could find. As @Wauplin said, we can't completely avoid 404s unless we add redirects for each user rename, so I will merge this now.

There's an interesting remaining case, some posts use org names as authors, for example, this one uses huggingface, cc @mishig25 just for info.

@pcuenca
Copy link
Member Author

pcuenca commented Dec 18, 2025

This touches a few files, cc @mishig25 @julien-c in case you want to take a quick glance, otherwise I'll merge at the end of the day.

@pcuenca pcuenca merged commit c15f0e6 into main Dec 18, 2025
1 check passed
@pcuenca pcuenca deleted the case-sensitive-authors branch December 18, 2025 23:30
Copy link
Member

@julien-c julien-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants