3 ms·
There's a recursion issue somewhere. 1: https://bernard.app/demo/8b13ab64-2440-4520-be05-905b5e2188ff https://bernard.app/demo/8b13ab64-2440-4520-be05-905b5e21
by nickstinemates 3y ago
There's a recursion issue somewhere.
1: https://bernard.app/demo/8b13ab64-2440-4520-be05-905b5e2188ff https://bernard.app/demo/8b13ab64-2440-4520-be05-905b5e2188f...
I like the product and I would use it.
- sph 3y agohttps://img.devrant.com/devrant/rant/r_268897_Geemo.jpg https://img.devrant.com/devrant/rant/r_268897_Geemo.jpg Thanks for the bug report :) to be frank, there is actually a recursive link in your website (that "hooked on computer" link towards the top of https://keeb.dev/2022/05/28/The-beginning/ https://keeb.dev/2022/05/28/The-beginning/)
- codingdave 3y agoRecursive links are not exactly uncommon - there is little reason to unlink the current page if it is a standard item in menus or other navigation on the site.
- sph 3y agoThere is a difference between a having a link to "/page" and a having relative link to "page". The first is fine, the second needs special logic to tell that when we reach "/page/page/[...]/page" we should probably stop. I bet $5 that their web server log shows plenty of naive bots like mine getting lost in that recursion, burning unnecessary CPU time. Thankfully, the solution is not that hard to implement.
- codingdave 3y agoI'm glad the fix is easy, but the web has always been notorious for garbage HTML. You'll go farther with this if you adopt an attitude that you can handle low-quality content vs. getting defensive about someone finding a new edge case that needs to be handled.
- sph 3y agoNot sure why you think I was being defensive, nor annoyed at the problem. I just wanted to clarify for sake of argument. It is on my todo list to fix. Thanks.
- nickstinemates 3y agoThanks for finding the bug in my site. I appreciate it! I have fixed it