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

please reject this request #67

Closed
wants to merge 34 commits into from
Closed

please reject this request #67

wants to merge 34 commits into from

Conversation

webbob
Copy link

@webbob webbob commented Dec 28, 2011

I'd like to clean up my work and resubmit it. Please close this request.

Bob Webber and others added 30 commits December 21, 2011 11:00
Added new package while working on new code, still learning my way
around.
Moved code from com.nexage.graylog2 package to org.graylog.jsonshortmessage package

Added support for list of key strings of key-value pairs to drop while parsing
a short message. Included update to configuration package, initialization in
the Main function.

Modified the parser hook to look at Drools rules and then JSON-based parsing.
Extending parsing hook to support parsing of JSON formatted
shortMessage part of GELF message.
…ers/webber/src/github/graylog2-server into develop
make m2e ignore the scala lifecycle plugins
can now be forced! fixes #SERVER-23
Conflicts:
	src/main/java/org/graylog2/Main.java
Catch IO and map/array manipulation exceptions and handling them.

Add selective logging with warnings and fatal errors.
@lennartkoopmann
Copy link
Contributor

Great idea - Waiting for the final pull request. :)

joschi added a commit that referenced this pull request Feb 13, 2018
* Migrate from TestNg to JUnit

* Add GeoLite2 Country database for tests

Version: 20180102
Download URL: https://dev.maxmind.com/geoip/geoip2/geolite2/#Downloads

* Improve null-handling in MaxmindDataAdapter

Fix #67

* Use null-safe map (HashMap) in MaxmindDataAdapter#doGet()

* Properly handle absence of CityResponse.location
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.

None yet

2 participants