[TT-13765] add check for host rewrite to DynamicMiddleware (js plugin) #6792
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Allow hostname rewrite in DynamicMiddleware (mw_js_plugin)
Related Issue
#6791
Motivation and Context
My organization has a use case where we need to route traffic dynamically to a backend hostname based on headers passed into a request. We'd prefer to not manage a GO plugin or it's development environment, and use a custom Javascript plugin instead. However Javascript plugins don't currently allow hostname rewrites.
How This Has Been Tested
Added unit tests provide some coverage. I ran a functional test by building an image and spinning that up as a hybrid data plane node, attached to our tyk hosted control plane. I was able to successfully rewrite the hostname this way.
Types of changes
Checklist