bobbyiliev / bobbyiliev/quick_access_logs_summary

sometimes I'm getting dates instead of IPs in "Top 20 IP addresses that have been accessing your site:"

Open
#2 3 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Shell
Stars
32
Forks
11
PR merge metrics
No merged PRs in 30d

Description

First of all, thank you for the script! :)

### Describe the bug

I have noticed that sometimes I'm getting dates instead of IPs in "Top 20 IP addresses that have been accessing your site:" and "Most recent...".

### To Reproduce

Just ran the script with my log.

### Expected behavior

There should be only IPs there

### Screenshots

```
Most Recent top 20 IP addresses:
303 2021/05/18 <--- here
167 66.249.66.11
58 213.211.43.129
55 66.249.66.13
50 114.119.156.223
50 114.119.128.46
42 114.119.157.61
42 114.119.144.204
41 66.249.66.15
39 114.119.159.210
35 114.119.128.35
33 114.119.150.149
13 66.249.66.6
11 172.105.250.86
5 66.249.66.8
5 66.249.66.4
4 199.111.229.63
3 74.120.14.53
3 66.249.66.7
2 84.0.240.66
```

Input log attached (actually just `grep '2021/05/18' ...` part of it) [my.log](https://github.com/bobbyiliev/quick_access_logs_summary/files/6504382/my.log)

Contributor guide

Open the contributing guide

Research direction

Start by running the script against the attached my.log input and inspect the parsing that produces the “Most Recent top 20 IP addresses” and “Top 20 IP addresses” sections. The fix is done when the reproduced output contains only IP addresses in those lists, while the existing access-log summary still works.

Written by the indexing model from the issue text.

Assessment

Tech stack
bash, shell
Domain
analytics, cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.