delta/zookeeper.git/zookeeper-client, branch master

[ZOOKEEPER-4674] TestReadOnlyClient.cc: Stop/start "normal" server in test setUp/tearDown (#1983)

2023-02-13T15:26:27+00:00

Co-authored-by: Damien Diederen

ZOOKEEPER-4654: Fix C client test compilation error in Util.cc.

2022-12-18T17:37:53+00:00

Author: Chris Nauroth 

Reviewers: Enrico Olivelli , Mate Szalay-Beko 

Closes #1967 from cnauroth/ZOOKEEPER-4654

ZOOKEEPER-4641: GH CI fails with error: implicit declaration of function FIPS_mode

2022-12-15T12:15:52+00:00

Modifications:
- remove the block, as it is used only for debug

More details here:
https://issues.apache.org/jira/browse/ZOOKEEPER-4641

Author: Enrico Olivelli 

Reviewers: Andor Molnar , Chris Nauroth , Mate Szalay-Beko 

Closes #1951 from eolivelli/fix/ZOOKEEPER-4641

ZOOKEEPER-4479: C tests: Avoid some jitter which results in flaky tests

2022-03-01T16:13:50+00:00

With these commits, the various tests in `TestOperations.cc` set the `last_sent` and `last_recv` fields of the "force-connected" ZooKeeper handle to deterministic values (which match the initial value of the time mock).

When that value is not specified, these fields get set to the "real" time at the end of the  `forceConnected` function, which can be sufficiently different from the value held in the mock for some of the tests to fail.

Author: Damien Diederen 

Reviewers: Enrico Olivelli 

Closes #1827 from ztzg/ZOOKEEPER-4479-flaky-c-test-operations

[maven-release-plugin] prepare for next development iteration

2022-02-03T07:55:04+00:00

ZOOKEEPER-4461: Use logback in zookeeper-contrib project

2022-02-01T10:39:04+00:00

I've only found Ant references in the `-recipes` projects which I don't necesseraly want to migrate, so this patch is focusing on `-contrib` projects.

Please review.

Author: Andor Molnar 

Reviewers: Enrico Olivelli 

Closes #1807 from anmolnar/ZOOKEEPER-4461

fix a typo in zookeeper.h

2021-11-21T04:39:22+00:00

Typo fix

Author: fengpan0403 

Reviewers: maoling 

Closes #1778 from fengpan0403/master

ZOOKEEPER-4342: Fix: Robustify C client against errors during SASL negotiation

2021-08-25T12:14:22+00:00

Before this, the client was ignoring the error field of the response header, and only considering SASL-level errors.

This commit makes it consider `hdr.err`.  It also zeroes the `res` data structure, to avoid a crash in `deallocate` if `deserialize` is skipped, and sets `input_buffer` to `NULL` to avoid a double-free.

(I looked into adding a non-regression test, but doing so requires adding quite a bit of infrastructure to the mocks so that the SASL library can be correctly initialized.  Punting for now.)

Author: Damien Diederen 

Reviewers: Enrico Olivelli

Closes #1733 from ztzg/ZOOKEEPER-4342-robustify-c-client-sasl-errors

ZOOKEEPER-4210: Preserve return code from nonblocking send

2021-03-09T20:29:08+00:00

Async API calls attempt to flush the send buffer, which
calls flush_send_queue(); and can report
  ZOPERATIONTIMEOUT
  ZSYSTEMERROR
  ZCONNECTIONLOSS

Specifically: send_buffer() calls send(2) with MSG_NOSIGNAL,
which can return EPIPE; then send_buffer return -1, causing
ZCONNECTIONLOSS from flush_send_queue().

Current async API calls drop the return value from flush_send_queue(),
as below:

    adaptor_send_queue(zh, 0);
    return (rc < 0)?ZMARSHALLINGERROR:ZOK;

The async API then returns ZOK instead of ZCONNECTIONLOSS.

Author: Sam Mikes 

Reviewers: Enrico Olivelli , Damien Diederen 

Closes #1602 from smikes/asyncsend-returncode-3.6

ZOOKEEPER-4217: add new arg 'func' to handle_socket_error_msg()

2021-03-07T09:49:58+00:00

Add argument `const char *func` typically initialized with __func__ to include current function name in log message.

Author: Sam Mikes 

Reviewers: Enrico Olivelli , Mate Szalay-Beko , Damien Diederen 

Closes #1609 from smikes/handle-msg-report-func