countradooku opened a new pull request, #3235:
URL: https://github.com/apache/iggy/pull/3235
## Summary
Adds an experimental PHP SDK under `foreign/php`, implemented as a
Rust-backed PHP extension over the in-tree Iggy Rust SDK.
The PR includes:
- synchronous `IggyClient` bindings
- Fiber/Revolt-aware `IggyAsyncClient` bindings
- stream/topic/message/consumer-group coverage
- PHP integration tests and TLS opt-in tests
- Dockerized PHP SDK test image and compose workflow
- CI component detection and pre-merge tasks for PHP SDK changes
- root Cargo workspace exclusion for `foreign/php`, matching the existing
foreign SDK pattern
## Validation
Ran the following locally:
- `cargo fmt --manifest-path foreign/php/Cargo.toml --check`
- `composer validate --working-dir foreign/php --strict`
- PHP syntax lint for all `foreign/php/tests/*.php`
- `cargo metadata --manifest-path foreign/php/Cargo.toml --no-deps
--format-version 1`
- `docker compose -f foreign/php/docker-compose.test.yml config`
- `docker compose -f foreign/php/docker-compose.test.yml build php-tests`
- PHP extension smoke in Docker: `extension_loaded("iggy-php") === true`
- PHP integration smoke against `apache/iggy:latest`: `23 passed, 3 skipped,
0 failed`
## Notes
The full compose test path is present, but I only completed the `php-tests`
image build plus an integration smoke against an already-running healthy Iggy
container. TLS tests are opt-in and skipped unless `IGGY_TLS_CONNECTION_STRING`
/ `IGGY_TLS_PLAINTEXT_ADDRESS` are set.
--
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]
