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

Fix refresh handling in error conditions #49

Merged
merged 3 commits into from Jun 14, 2017

Conversation

@bernd
Copy link
Member

@bernd bernd commented Jun 12, 2017

@bernd bernd added this to the 2.3.0 milestone Jun 12, 2017
@bernd bernd requested a review from kroepke Jun 12, 2017
Copy link
Member

@kroepke kroepke left a comment

The maxmind adapter is missing the validate method in its config, which is confusing because it doesn't validate as the csv file adapter does.

@bernd
Copy link
Member Author

@bernd bernd commented Jun 14, 2017

@kroepke Fixed!

@kroepke kroepke merged commit 8e1d74d into master Jun 14, 2017
1 check passed
1 check passed
@garybot2
graylog-project/pr Jenkins build graylog-project-pr-snapshot 232 has succeeded
Details
@kroepke kroepke deleted the fix-lut-data-adapter-error-handling branch Jun 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants