Skip to content

Conversation

@aletourneau
Copy link

Corresponds to: fluent/fluent-bit#11080

Copy link
Contributor

@esmerel esmerel left a comment

Choose a reason for hiding this comment

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

My suggestions are stylistic for page consistency. They're fairly minor. This looks good to me otherwise, but should have a technical review for final approval.

Signed-off-by: Alexandre Létourneau <[email protected]>
@aletourneau aletourneau force-pushed the in_http/add_remote_addr branch from bc93bab to 867a1fb Compare November 3, 2025 21:07
@eschabell eschabell added the waiting-on-review Waiting on a review from mainteners label Nov 4, 2025
@eschabell eschabell self-assigned this Nov 4, 2025
Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

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

Seems ok, waiting on code merge.

| `successful_response_code` | Allows setting successful response code. Supported values: `200`, `201`, and `204` | `201` |
| `success_header` | Add an HTTP header key/value pair on success. Multiple headers can be set. For example, `X-Custom custom-answer` | _none_ |
| `threaded` | Indicates whether to run this input in its own [thread](../../administration/multithreading.md#inputs). | `false` |
| Key | Description | Default |
Copy link
Contributor

Choose a reason for hiding this comment

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

It's harder to review when we change the whole table rather than just the extra line.

Copy link
Author

Choose a reason for hiding this comment

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

Apologies for that,

@patrick-stephens patrick-stephens removed the waiting-on-review Waiting on a review from mainteners label Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants