Skip to content

feat: add stream module api#286

Open
plusls wants to merge 2 commits into
nginx:mainfrom
plusls:main
Open

feat: add stream module api#286
plusls wants to merge 2 commits into
nginx:mainfrom
plusls:main

Conversation

@plusls
Copy link
Copy Markdown

@plusls plusls commented May 7, 2026

Proposed changes

Add stream module api support

impl #108

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have written my commit messages in the Conventional Commits format.
  • I have read the CONTRIBUTING doc
  • I have added tests (when possible) that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

plusls added 2 commits May 7, 2026 10:51
Stream API was compiled whenever NGINX reported stream support, but
nginx-sys only generated stream FFI when its own stream feature was
enabled. Add a stream feature to the root crate that propagates to
nginx-sys/stream and gate the module with both conditions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant