Closed zenminimalist closed 9 months ago
Hello,
can you check using bash instead of sh please?
bash <(curl -s https://raw.githubusercontent.com/tbaldur/cyberpanel-mods/main/modsec_rules_v_changer.sh) || bash <(wget -O - https://raw.githubusercontent.com/tbaldur/cyberpanel-mods/main/modsec_rules_v_changer.sh)
Thanks so much! That worked without an error. I even tried the 4.0 RC - no errors : ) Will a CP update overwrite any of these mods? Or is it a groundhog thing
with each update?
And I assume the Fix missing acme-challenge context on all vhosts config
is obsolete now, as CP switched to ACME (from whatever they were using before) in one of their latest releases? But that could be related to me not fully understanding the problem it tries to fix.
I will update the documentation to use bash instead. I'm not aware of the current changes done in cyberpanel, at the time of the script making, cyberpanel did not update the rules, so it should not overwrite them.
The "Fix missing acme-challenge context on all vhosts config" was because on one of the cyberpanel updates, when they changed the certificate issuing mode, they did not update the configuration files, so it resulted in cyberpanel not being able to issue the SSL. If your SSL is working properly then you don't need to run that fix. Also if you use external SSL certificate provider like cloudflare, you also don't need it
Thanks for proving these mods! I tried a bunch of them no erros, exept with the Modsec ones. Not sure if they were installed or not as I get a
/dev/fd/63: 43: Syntax error: "(" unexpected
Or can this jsut be ignored?