stacker.news icon indicating copy to clipboard operation
stacker.news copied to clipboard

Dup prevention should not be case sensitive for Twitter accounts

Open cointastical opened this issue 4 years ago • 2 comments

The duplicate prevention check will see the following as being distinct / unique, and thus not show the DUP alert:

  • https://Twitter.com/stacker_news
  • https://Twitter.com/Stacker_News

I don't know what other URL types would also need to be made case insensitive as well. I can imagine Github being one:

  • https://github.com/stackernews
  • https://github.com/StackerNews

cointastical avatar Apr 28 '22 08:04 cointastical

👍 URLs are generally case insensitive

huumn avatar Apr 28 '22 16:04 huumn

"URLs in general are case-sensitive (with the exception of machine names). There may be URLs, or parts of URLs, where case doesn't matter, but identifying these may not be easy. Users should always consider that URLs are case-sensitive."

https://www.w3.org/TR/WD-html40-970708/htmlweb.html https://stackoverflow.com/questions/7996919/should-url-be-case-sensitive/7996997#7996997

But I can't think of any time where I've seen two different URLs have only a difference in case, and them both being unique, and valid requests (e.g., not giving a 404, or something to that effect). So for duplicate detection, ignoring case shouldn't cause any issue that I can tell.

cointastical avatar Apr 28 '22 22:04 cointastical