-
-
Couldn't load subscription status.
- Fork 233
ProxyUrlTransformer #1361
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ProxyUrlTransformer #1361
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces the ProxyUrlTransformer class to centralize URL transformation logic for proxy operations, supporting both simple string replacement and template-based transformations using Handlebars or Scriban.
Key changes:
- Added new
ProxyUrlTransformerclass with support for template-based URL transformations - Enhanced
ProxyUrlReplaceSettingswith transformer capabilities includingUseTransformerproperty and template support - Refactored existing transformer creation logic into a centralized
TransformerFactory
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| ProxyUrlTransformerTests.cs | Comprehensive test coverage for the new proxy URL transformation functionality |
| ProxyUrlReplaceSettings.cs | Enhanced settings class with transformer template support and nullable annotations |
| TransformerFactory.cs | New factory class to centralize transformer instantiation logic |
| Transformer.cs | Added new Transform method overload for simple template/model transformations |
| ITransformerContextFactory.cs | Updated interface visibility and formatting |
| ITransformer.cs | Added new Transform method signature and updated visibility |
| Response.cs | Refactored to use the new TransformerFactory instead of inline transformer creation |
| ProxyUrlTransformer.cs | New core class implementing URL transformation logic |
| ProxyHelper.cs | Refactored to use ProxyUrlTransformer and updated to primary constructor syntax |
| WebhookSender.cs | Refactored to use TransformerFactory and updated to primary constructor syntax |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1361 +/- ##
==========================================
+ Coverage 35.61% 35.70% +0.08%
==========================================
Files 150 152 +2
Lines 34554 34537 -17
==========================================
+ Hits 12308 12332 +24
+ Misses 21841 21780 -61
- Partials 405 425 +20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot <[email protected]>
No description provided.