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

redone enable-abuseipdb reporting #114

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Bananz0
Copy link

@Bananz0 Bananz0 commented Jul 12, 2024

I have redone the previous PR following the issues raised and using ristretto instead of go-cache.
I'm still not too sure if I have handled the docker secrets well

@shizunge
Copy link
Owner

I want to build a test framework before any new features. The review could be slow.

@shizunge
Copy link
Owner

shizunge commented Oct 7, 2024

Current roadmap:

  1. Create integration tests, including tests of the CLI arguments.
  2. Update github actions
  3. Read config file from yaml and/or ENV. traefik is doing it, we may borrow some codes there. (Note: traefik is under MIT license)
  4. Build a new data processing pipeline, updating Prometheus metrics will be one stage of the pipeline.
  5. enable-abuseipdb reporting, which should a parallel stage of updating Prometheus metrics.

I will review this CL when we finish previous steps.

This CL is at step 5 above.

Step 4 and 5 could be independent from 1-3. Based on my bandwidth, I can only work on them in serial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants