Browse Source

refactor: simplify payload construction

The code changes in this commit refactor the payload construction in the `info_stats.sh` file. The changes simplify the process by removing repetitive if statements and consolidating the payload parameters. This improves readability and maintainability of the code.
pull/4295/head
Daniel Gibbs 2 years ago
parent
commit
640cd0e125
  1. 55
      lgsm/modules/info_stats.sh

55
lgsm/modules/info_stats.sh

@ -66,56 +66,57 @@ payload="{
\"events\": [ \"events\": [
{ {
\"name\": \"LinuxGSM\", \"name\": \"LinuxGSM\",
\"params\": { \"params\": {"
\"cpuusedmhzroundup\": \"${cpuusedmhzroundup}MHz\",
\"diskused\": \"${serverfilesdu}\",
\"distro\": \"${distroname}\",
\"game\": \"${gamename}\",
\"memusedroundup\": \"${memusedroundup}MB\",
\"ramused\": \"${memusedroundup}MB\",
\"servercpu\": \"${cpumodel} ${cpucores} cores\",
\"servercpufreq\": \"${cpufreqency} x${cpucores}\",
\"serverdisk\": \"${totalspace}\",
\"serverfilesdu\": \"${serverfilesdu}\",
\"serverram\": \"${physmemtotal}\",
\"uuidhardware\": \"${uuidhardware}\",
\"uuidinstall\": \"${uuidinstall}\",
\"uuidinstance\": \"${uuidinstance}\",
\"version\": \"${version}\",
\"virtualenvironment\": \"${virtualenvironment}\","
if [ "${discordalert}" == "on" ]; then if [ "${discordalert}" == "on" ]; then
payload="${payload}, \"discordalert\": \"${discordalert}\"" payload="${payload} \"discordalert\": \"${discordalert}\","
fi fi
if [ "${emailalert}" == "on" ]; then if [ "${emailalert}" == "on" ]; then
payload="${payload}, \"emailalert\": \"${emailalert}\"" payload="${payload} \"emailalert\": \"${emailalert}\","
fi fi
if [ "${gotifyalert}" == "on" ]; then if [ "${gotifyalert}" == "on" ]; then
payload="${payload}, \"gotifyalert\": \"${gotifyalert}\"" payload="${payload}, \"gotifyalert\": \"${gotifyalert}\","
fi fi
if [ "${iftttalert}" == "on" ]; then if [ "${iftttalert}" == "on" ]; then
payload="${payload}, \"iftttalert\": \"${iftttalert}\"" payload="${payload}, \"iftttalert\": \"${iftttalert}\","
fi fi
if [ "${mailgunalert}" == "on" ]; then if [ "${mailgunalert}" == "on" ]; then
payload="${payload}, \"mailgunalert\": \"${mailgunalert}\"" payload="${payload} \"mailgunalert\": \"${mailgunalert}\","
fi fi
if [ "${pushbulletalert}" == "on" ]; then if [ "${pushbulletalert}" == "on" ]; then
payload="${payload}, \"pushbulletalert\": \"${pushbulletalert}\"" payload="${payload} \"pushbulletalert\": \"${pushbulletalert}\","
fi fi
if [ "${pushoveralert}" == "on" ]; then if [ "${pushoveralert}" == "on" ]; then
payload="${payload}, \"pushoveralert\": \"${pushoveralert}\"" payload="${payload} \"pushoveralert\": \"${pushoveralert}\","
fi fi
if [ "${rocketchatalert}" == "on" ]; then if [ "${rocketchatalert}" == "on" ]; then
payload="${payload}, \"rocketchatalert\": \"${rocketchatalert}\"" payload="${payload} \"rocketchatalert\": \"${rocketchatalert}\","
fi fi
if [ "${slackalert}" == "on" ]; then if [ "${slackalert}" == "on" ]; then
payload="${payload}, \"slackalert\": \"${slackalert}\"" payload="${payload} \"slackalert\": \"${slackalert}\","
fi fi
if [ "${telegramalert}" == "on" ]; then if [ "${telegramalert}" == "on" ]; then
payload="${payload}, \"telegramalert\": \"${telegramalert}\"" payload="${payload} \"telegramalert\": \"${telegramalert}\","
fi fi
payload="${payload} payload="${payload}
\"cpuusedmhzroundup\": \"${cpuusedmhzroundup}MHz\",
\"diskused\": \"${serverfilesdu}\",
\"distro\": \"${distroname}\",
\"game\": \"${gamename}\",
\"memusedroundup\": \"${memusedroundup}MB\",
\"ramused\": \"${memusedroundup}MB\",
\"servercpu\": \"${cpumodel} ${cpucores} cores\",
\"servercpufreq\": \"${cpufreqency} x${cpucores}\",
\"serverdisk\": \"${totalspace}\",
\"serverfilesdu\": \"${serverfilesdu}\",
\"serverram\": \"${physmemtotal}\",
\"uuidhardware\": \"${uuidhardware}\",
\"uuidinstall\": \"${uuidinstall}\",
\"uuidinstance\": \"${uuidinstance}\",
\"version\": \"${version}\",
\"virtualenvironment\": \"${virtualenvironment}\"
} }
} }
] ]

Loading…
Cancel
Save