mmetc
mmetc
I guess so because #355 was merged two years before this issue. The question is more what's missing capi-side, last time we talked about that it was not ready
Hi, was there any progress on this? If it's simpler, support for "COPY
@blotus conflicts are fixed, keep in mind that on master we don't care at all (no warning, no check) about WAL from cscli. I kept the same behavior in this...
wip https://github.com/crowdsecurity/crowdsec/pull/2812
``` diff --git a/pkg/apiclient/client_test.go b/pkg/apiclient/client_test.go index 27210962..cea87bb0 100644 --- a/pkg/apiclient/client_test.go +++ b/pkg/apiclient/client_test.go @@ -115,7 +115,7 @@ func TestNewClientOk_UnixSocket(t *testing.T) { client, err := NewClient(&Config{ MachineID: "test_login", Password: "test_password", - UserAgent:...
I am reviewing, the above snippet is to fix the current version after merging to master. And here is a functional test (09_socket.bats) ``` #!/usr/bin/env bats # vim: ft=bats:list:ts=8:sts=4:sw=4:et:ai:si: set...
@cyberb I'd rather have unix:///path/to/socket instead of a plain path. What do you think?
Sorry for the delay, this will surely be merged but we'll review again and do more tests
I have ported this on top of 1.5.6 changes and allow both TCP and socket at the same time, I'm preparing the PR but will be merged after release to...
Sorry it took so long @cyberb, what do you think of https://github.com/crowdsecurity/crowdsec/pull/2770 Unfortunately I cannot merge your PR directly since the code needed some refactoring first, which had already started...