Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log the client IP address and the agent when login fail #16423

Closed
stonezdj opened this issue Feb 26, 2022 · 0 comments · Fixed by #16424
Closed

Log the client IP address and the agent when login fail #16423

stonezdj opened this issue Feb 26, 2022 · 0 comments · Fixed by #16424
Assignees
Labels
area/access-log The work to make access-log handler more extensible and configurable
Milestone

Comments

@stonezdj
Copy link
Contributor

Sometimes some pipeline or malicious tool uses invalid password to log in Harbor server, there is no log information to find out the source of this failed login. need to log the client IP and the agent.

stonezdj added a commit to stonezdj/harbor that referenced this issue Feb 26, 2022
   Fixes goharbor#16423

Signed-off-by: stonezdj <stonezdj@gmail.com>
stonezdj added a commit to stonezdj/harbor that referenced this issue Feb 26, 2022
   Fixes goharbor#16423

Signed-off-by: stonezdj <stonezdj@gmail.com>
@stonezdj stonezdj self-assigned this Feb 28, 2022
@stonezdj stonezdj added the area/access-log The work to make access-log handler more extensible and configurable label Feb 28, 2022
stonezdj added a commit to stonezdj/harbor that referenced this issue Feb 28, 2022
   Fixes goharbor#16423

Signed-off-by: stonezdj <stonezdj@gmail.com>
stonezdj added a commit to stonezdj/harbor that referenced this issue Feb 28, 2022
   Fixes goharbor#16423

Signed-off-by: stonezdj <stonezdj@gmail.com>
stonezdj added a commit to stonezdj/harbor that referenced this issue Mar 2, 2022
   Fixes goharbor#16423

Signed-off-by: stonezdj <stonezdj@gmail.com>
@yanji09 yanji09 added this to the Sprint 132 milestone Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/access-log The work to make access-log handler more extensible and configurable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants