hubcio opened a new pull request, #3845: URL: https://github.com/apache/iggy/pull/3845
The PHP extension wraps the Rust SDK without the vsr feature, so its suites only ever exercised the legacy wire protocol. Building it with the feature moves the SDK, BDD and examples lanes onto server-ng, and the dev compose harness with them, since a vsr-built extension cannot talk to the legacy server at all. Only one expectation encoded legacy behavior: a send used to assert an empty confirmation list, where the VSR server reports the written partition's offsets. It now asserts one confirmation per partition and that the offset advances by the committed message count. The migration also surfaced a server bug. A username/password body that fails verification falls through to the PAT decode attempt so a credential payload shaped like a token still gets a chance, but when that decode failed too the terminal rejection reported the payload shape rather than the credential failure, and clients saw InvalidFormat for a wrong password. The fall-through now carries the rejection that actually happened, which is what makes the PHP suite raise AuthenticationException instead of the untyped base exception. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
