cschantz/Linux-Server-Management-Toolkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Remove unnecessary process substitution in analyze_time_series()

Browse Source 
Line 1644: Changed from process substitution to direct file input
- From: }' "$TEMP_DIR/attack_vectors_raw.txt" <(cat "$TEMP_DIR/parsed_logs.txt") | sort
- To: }' "$TEMP_DIR/attack_vectors_raw.txt" "$TEMP_DIR/parsed_logs.txt" | sort
- Eliminates unnecessary pipe and subshell for efficiency

This is the final efficiency improvement in the series of bot-analyzer fixes.
This commit is contained in:
Developer
2026-04-23 18:39:17 -04:00
parent adbe5c14d5
commit e1a3b1cf90
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/security/bot-analyzer.sh
+1 -1
View File
@@ -1641,7 +1641,7 @@ analyze_time_series() {
hour = ts[4] hour = ts[4]
print hour print hour
} }
}' "$TEMP_DIR/attack_vectors_raw.txt" <(cat "$TEMP_DIR/parsed_logs.txt") | sort | uniq -c > "$TEMP_DIR/hourly_attack_traffic.txt" }' "$TEMP_DIR/attack_vectors_raw.txt" "$TEMP_DIR/parsed_logs.txt" | sort | uniq -c > "$TEMP_DIR/hourly_attack_traffic.txt"
fi fi
print_success "Time-series analysis complete" print_success "Time-series analysis complete"