Skip to content

/ graylog2-server

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

Jump to bottom

Import extractors returns NPE #942

Closed
edmundoa opened this issue Feb 4, 2015 · 2 comments
Closed

Import extractors returns NPE #942

edmundoa opened this issue Feb 4, 2015 · 2 comments
Assignees
@joschi
Labels
bug
Milestone
1.0.0

Comments

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Feb 4, 2015

Using 1.0.0-rc.2, importing extractors may fail with a NPE.
@edmundoa edmundoa added the bug label Feb 4, 2015
@edmundoa edmundoa added this to the 1.0.0 milestone Feb 4, 2015
@edmundoa edmundoa self-assigned this Feb 4, 2015
@bernd
Copy link
Member

@bernd bernd commented Feb 4, 2015

I was able to import the apache and heroku extractors from graylog.org, but the clavister one did not work. It seems to have a broken converters section. (3 with empty {} and one with a broken date type config)

Nevertheless, our code should somehow handle this or at least provide a better error message.

Tested with 1.0.0-rc.2.
@edmundoa edmundoa modified the milestones: 1.0.1, 1.0.0 Feb 4, 2015
@joschi joschi assigned joschi and unassigned edmundoa Feb 11, 2015
This was referenced Feb 11, 2015
Merged
Merged
@joschi joschi modified the milestones: 1.0.0, 1.0.1 Feb 11, 2015
@edmundoa
Copy link
Member Author

@edmundoa edmundoa commented Feb 13, 2015

Fixed with changes in PRs #960 and graylog-labs/graylog2-web-interface#1115.
@edmundoa edmundoa closed this Feb 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joschi joschi

Labels
bug
Projects
None yet
Milestone
1.0.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@bernd @joschi @edmundoa