Skip to content

Backport(v1.19): buffer: resume buffer correctly even though path contains [] (#5300)#5305

Merged
Watson1978 merged 1 commit intov1.19from
backport-to-v1.19/pr5300
Mar 30, 2026
Merged

Backport(v1.19): buffer: resume buffer correctly even though path contains [] (#5300)#5305
Watson1978 merged 1 commit intov1.19from
backport-to-v1.19/pr5300

Conversation

@github-actions
Copy link
Copy Markdown

Which issue(s) this PR fixes:
Backport #5300

Fixes #4407

What this PR does / why we need it:

If buffer path contains [] such as "path test/${tag[0]}", when resuming buffer process can't find them without escaping bracket.
Thus buffer files remains under that directory.

NOTE: recommended tag spec is specified in routing documentation, but it is easily shoot your legs in practical use-case, so it is better to take care of that case.

https://docs.fluentd.org/configuration/config-file#interlude-routing

Docs Changes:

N/A

Release Note:

N/A

**Which issue(s) this PR fixes**:

Fixes #4407

**What this PR does / why we need it**:

If buffer path contains [] such as "path test/${tag[0]}", when resuming
buffer process can't find them without escaping bracket.
Thus buffer files remains under that directory.

NOTE: recommended tag spec is specified in routing documentation, but it
is easily shoot your legs in practical use-case, so it is better to take
care of that case.

https://docs.fluentd.org/configuration/config-file#interlude-routing

**Docs Changes**:

N/A

**Release Note**:

N/A

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Watson1978 Watson1978 merged commit 0648d4b into v1.19 Mar 30, 2026
1 check passed
@Watson1978 Watson1978 deleted the backport-to-v1.19/pr5300 branch March 30, 2026 03:01
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.

2 participants