logoalt Hacker News

LoganDarktoday at 4:15 AM3 repliesview on HN

I think I like the idea, but the structure of the code doesn't look the best. What most sticks out to me is the "Managers" directory. I've seen similar patterns before, even at my current place of work, but they seem to correlate with less experienced implementations. For instance, I clicked on one of them randomly and already found an issue: https://github.com/nook-browser/Nook/blob/09a4c6957a2e9fd7c5...

I guess `www.` (and only `www.`) is always special, and the only TLDs with two components are `"co.uk", "co.jp", "com.au", "co.nz", "com.br"`?

I don't know how critical this "Manager" is (what even is a "boost"?), but a web browser should absolutely have a proper list of TLDs!


Replies

SoKamiltoday at 9:43 AM

> What most sticks out to me is the "Managers" directory. I've seen similar patterns before, even at my current place of work, but they seem to correlate with less experienced implementations

What is wrong with such structure? How would you structure this code? Genuinely asking

grodriguez100today at 7:18 AM

Uh oh. Looks bad.

normie3000today at 5:30 AM

> the only TLDs with two components are `"co.uk", "co.jp", "com.au", "co.nz", "com.br".

Is this sarcasm? The public suffix list will give some ideas for omissions: https://publicsuffix.org/list/public_suffix_list.dat

show 2 replies