search

mattwcole / gelf-extensions-logging

GELF provider for Microsoft.Extensions.Logging
MIT License
109 stars 42 forks source link

Added HttpGelfClient #14

Closed mersadk closed 6 years ago

mersadk commented 6 years ago

Added support for HTTP/HTTPS Graylog input endpoints.

mattwcole commented 6 years ago

Thanks @mersadk this is looking good. I'll have a closer look this week and see if I can get the CI build passing before we merge it.

mattwcole commented 6 years ago

I've pushed a branch with some minor changes to the implementation here. If you are happy with these, you should be able to merge them into your PR. I'm not sure why the Travis build is failing, the tests pass locally, I'll have to investigate further.

mersadk commented 6 years ago

Looks great. I've merged branches.

mattwcole commented 6 years ago

OK I managed to get the build green simply by upgrading Graylog and Elasticsearch. If you merge that branch again, this PR should go green.

mersadk commented 6 years ago

Done but it appears that build again failed.

mattwcole commented 6 years ago

I'm going to merge so I don't have to keep sending you changes as I try to fix it. Thanks again for the contribution!