The 206 code indicates a partial download which should be counted webmin/webmin#459

jcameron · jcameron · commit af1f38912b11 · 2016-11-19T14:53:15.000-08:00
diff --git a/virtual_feature.pl b/virtual_feature.pl
@@ -1231,7 +1231,7 @@ sub feature_bandwidth
 			open($LOG, "<", $f);
 			}
 		while(<$LOG>) {
-			if (/^(\S+)\s+(\S+)\s+(\S+)\s+\[(\d+)\/(\S+)\/(\d+):(\d+):(\d+):(\d+)\s+(\S+)\]\s+"([^"]*)"\s+(\S+)\s+(\S+)/ && $12 ne "206") {
+			if (/^(\S+)\s+(\S+)\s+(\S+)\s+\[(\d+)\/(\S+)\/(\d+):(\d+):(\d+):(\d+)\s+(\S+)\]\s+"([^"]*)"\s+(\S+)\s+(\S+)/) {
 				# Valid-looking log line .. work out the time
 				no warnings "once";
 				my $ltime = timelocal($9, $8, $7, $4, $virtual_server::apache_mmap{lc($5)}, $6-1900);

Original file line number	Diff line number	Diff line change
`@@ -1231,7 +1231,7 @@ sub feature_bandwidth`
`1231`	`1231`	`open($LOG, "<", $f);`
`1232`	`1232`	`}`
`1233`	`1233`	`while(<$LOG>) {`
`1234`		`- if (/^(\S+)\s+(\S+)\s+(\S+)\s+\[(\d+)\/(\S+)\/(\d+):(\d+):(\d+):(\d+)\s+(\S+)\]\s+"([^"]*)"\s+(\S+)\s+(\S+)/ && $12 ne "206") {`
	`1234`	`+ if (/^(\S+)\s+(\S+)\s+(\S+)\s+\[(\d+)\/(\S+)\/(\d+):(\d+):(\d+):(\d+)\s+(\S+)\]\s+"([^"]*)"\s+(\S+)\s+(\S+)/) {`
`1235`	`1235`	`# Valid-looking log line .. work out the time`
`1236`	`1236`	`no warnings "once";`
`1237`	`1237`	`my $ltime = timelocal($9, $8, $7, $4, $virtual_server::apache_mmap{lc($5)}, $6-1900);`