Skip to content

fix: applyPathRewrite logs old req.url instead of rewritten path#1157

Merged
chimurai merged 1 commit intochimurai:masterfrom
nicoabie:fixes/1156
Apr 3, 2026
Merged

fix: applyPathRewrite logs old req.url instead of rewritten path#1157
chimurai merged 1 commit intochimurai:masterfrom
nicoabie:fixes/1156

Conversation

@nicoabie
Copy link
Copy Markdown
Contributor

Description

Motivation and Context

  • help debugging

How has this been tested?

  • locally

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@chimurai chimurai added this to the v4 milestone Apr 3, 2026
@chimurai chimurai enabled auto-merge (squash) April 3, 2026 22:34
@chimurai
Copy link
Copy Markdown
Owner

chimurai commented Apr 3, 2026

Thanks!

@chimurai chimurai merged commit a6ebb03 into chimurai:master Apr 3, 2026
10 checks passed
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 96.335%. remained the same
when pulling 1e53336 on nicoabie:fixes/1156
into 9e79695 on chimurai:master.

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.

applyPathRewrite logs old req.url instead of rewritten path

3 participants