-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Description
We have 2 BIG PRs need to be review for the full coverage of moving to the Logstash bridge.
Findings:
- GeoIP version conflict:
- Improvements:
- Migrate all Bridges to "interface pattern", using factory methods instead of constructors.
- Extract implementations to package-private (so that we have a clear boundary of what is "externalized" to the logstash plugin