remove warning for access log

because you can't see the iplimit when there is no path for access log :D
This commit is contained in:
mhsanaei 2024-09-09 09:52:29 +02:00
parent 272457740f
commit 2e1461e6dc

View file

@ -37,7 +37,7 @@ func (j *CheckClientIpJob) Run() {
shouldClearAccessLog := false shouldClearAccessLog := false
f2bInstalled := j.checkFail2BanInstalled() f2bInstalled := j.checkFail2BanInstalled()
isAccessLogAvailable := j.checkAccessLogAvailable(f2bInstalled) isAccessLogAvailable := j.checkAccessLogAvailable()
if j.hasLimitIp() { if j.hasLimitIp() {
if f2bInstalled && isAccessLogAvailable { if f2bInstalled && isAccessLogAvailable {
@ -49,7 +49,7 @@ func (j *CheckClientIpJob) Run() {
} }
} }
if shouldClearAccessLog || isAccessLogAvailable && time.Now().Unix()-j.lastClear > 3600 { if shouldClearAccessLog || (isAccessLogAvailable && time.Now().Unix()-j.lastClear > 3600) {
j.clearAccessLog() j.clearAccessLog()
} }
} }
@ -174,27 +174,18 @@ func (j *CheckClientIpJob) checkFail2BanInstalled() bool {
return err == nil return err == nil
} }
func (j *CheckClientIpJob) checkAccessLogAvailable(handleWarning bool) bool { func (j *CheckClientIpJob) checkAccessLogAvailable() bool {
isAvailable := true isAvailable := true
warningMsg := ""
accessLogPath, err := xray.GetAccessLogPath() accessLogPath, err := xray.GetAccessLogPath()
if err != nil { if err != nil {
return false return false
} }
// access log is not available if it is set to 'none' or an empty string
switch accessLogPath { switch accessLogPath {
case "none": case "none", "":
warningMsg = "Access log is set to 'none', check your Xray Configs"
isAvailable = false
case "":
warningMsg = "Access log doesn't exist in your Xray Configs"
isAvailable = false isAvailable = false
} }
if handleWarning && warningMsg != "" {
logger.Warning(warningMsg)
}
return isAvailable return isAvailable
} }