As part of the devs farewell message on their site, they have included malicious code to make each visitor sends 2,000 requests to the dbzer0 servers in an attempt to DDOS and take the instance offline.
As part of the devs farewell message on their site, they have included malicious code to make each visitor sends 2,000 requests to the dbzer0 servers in an attempt to DDOS and take the instance offline.
I understand what is happening.
I think that breaking a constant evaluated value into an expression that is still a constant comprised entirely of literals is bad.
Extract the constants (100 and 1) to NAMED constants if you wanna do this.
Edit: You also mentioned it might just be a side effect of a minifier. Maybe it is. I’ve got tons of grace if that’s the case, since the dev has little if any control. Be like if I was roasting someone for what a compiler did.
You’re obviously the js expert, so I’ll defer to you on expected minifier behavior, but there are 3 specific reasons that id be surprised (based on what I think a minifier should do)
there is a defined variable that already exists “random”, i would any minifier worth it’s salt to make the variable name smaller
since the expression (100 + 1 - 1) is a constant expression, I similarly would expect a minifier to reduce constant expressions to thier evaluated value (especially if the evaluated value is fewer characters than the expression) aka, just collapse to “100”
there is still a ton of extraneous whitespace