search

gdestuynder / audisp-json

42 stars 17 forks source link

Fix small fieldname conflict by renaming top level processname to program #25

Closed Phrozyn closed 5 years ago

Phrozyn commented 5 years ago

Thought I'd put up a PR to indicate what I meant in issue #24 Feel free to close or propose modification. Thanks!

Phrozyn commented 5 years ago

I wanted to add that this change would not break mozdef's ingestion of these events as we look for program: https://github.com/mozilla/MozDef/blob/master/mq/esworker_eventtask.py#L100

gdestuynder commented 5 years ago

@Phrozyn was this made unnecessary by 2.2.5 or is this still needed?

Phrozyn commented 5 years ago

This is no longer needed, thank you!

On Fri, Apr 12, 2019, 10:13 PM Guillaume Destuynder (:kang) < notifications@github.com> wrote:

@Phrozyn https://github.com/Phrozyn was this made unnecessary by 2.2.5 or is this still needed?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gdestuynder/audisp-json/pull/25#issuecomment-482771459, or mute the thread https://github.com/notifications/unsubscribe-auth/AEEf0CVpN3i9gLKJk3liU8AV1rVghGbuks5vgUtJgaJpZM4cqG7b .