Skip to content

bind in register#221

Open
yaauie wants to merge 4 commits into
logstash-plugins:mainfrom
yaauie:bind-in-register
Open

bind in register#221
yaauie wants to merge 4 commits into
logstash-plugins:mainfrom
yaauie:bind-in-register

Conversation

@yaauie

@yaauie yaauie commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Binds to the port in #register, so that a failure to bind can be surfaced to the user.

Resolves: #215

@github-actions

Copy link
Copy Markdown
Contributor

Changelog and Version Management

No changelog or version changes detected. You can either update them manually or use a comment command to update them automatically on merge:

/pr_prepare release:<patch|minor|major> — bump version + update changelog
/pr_prepare changelog:add — changelog entry only (no version bump)

/pr_prepare release:patch
- Add the changelog entry here
  - (Additional details can be provided in multiple lines, respecting markdown formatting)

Omit the entry line to use the PR title. If multiple commands are posted, the last one wins.

@donoghuc donoghuc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small question. Otherwise this looks good.

Comment thread lib/logstash/inputs/http.rb
Co-authored-by: Cas Donoghue <cas.donoghue@gmail.com>
@yaauie yaauie requested a review from donoghuc July 2, 2026 21:44
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.

Bind to port during #register

2 participants