Add .NET 10 multi-targeting and CDN-style URL formatting compatibility by Codex · Pull Request #15 · chsword/ResizingServer

Codex · 2026-09-27T00:47:50Z

This change upgrades ResizingClient compatibility to include .NET 10 while preserving existing targets and legacy URL behavior. It also adds first-class URL formatting support for Tencent CDN, Aliyun CDN, and AWS image-processing query styles without breaking existing FormatUrl usage.

Framework compatibility
- Extended ResizingClient target frameworks to include net10.0 alongside net45, netstandard2.0, and net8.0.
- Updated CI matrix targeting so SDK 8/9 jobs build net8.0 and SDK 10 job builds net10.0, avoiding cross-SDK target mismatch.
Backward-compatible client API
- Added ResizingUtil.Format(...) as a compatibility alias to FormatUrl(...) to preserve older calling patterns.
- Kept existing mode token behavior (c/m/p) unchanged for current format-string URL generation.
CDN URL style support
- Added ResizingUtil.FormatTencentCdnUrl(...) for Tencent-style imageMogr2 parameters.
- Added ResizingUtil.FormatAliyunCdnUrl(...) for Aliyun-style x-oss-process=image/resize,....
- Added ResizingUtil.FormatAwsCdnUrl(...) for AWS-style x-amz-process=image/resize,....
- Added shared query-appending behavior that correctly uses ? or & depending on whether the base URL already contains query parameters.

var legacy = ResizingUtil.Format("/u/face/{0}x{1}{2}.jpg", 100, 100, ResizingMode.Pad);
var tencent = ResizingUtil.FormatTencentCdnUrl("https://img.example.com/a.jpg", 100, 100, ResizingMode.Crop);
var aliyun = ResizingUtil.FormatAliyunCdnUrl("https://img.example.com/a.jpg", 100, 100, ResizingMode.Max);
var aws = ResizingUtil.FormatAwsCdnUrl("https://img.example.com/a.jpg", 100, 100, ResizingMode.Pad);

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

ab.chatgpt.com
- Triggering command: /opt/hostedtoolcache/node/24.21.0/x64/lib/node_modules/@openai/codex/node_modules/@openai/codex-linux-x64/vendor/x86_64-REDACTED-linux-musl/codex/codex (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot cloud agent settings (admins only)

Fixes 兼容性升级 #14

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot

Copilot review overview

🟡 Changes recommended

Resolve the three URL-formatting issues identified in ResizingUtil.cs, including the critical AWS syntax issue.

Review effort: Lite
Findings: 1 · 2

Open (3)

Use valid AWS resize mode tokens in x-amz-process · New
Map Tencent Pad resizing to the pad/1 operation · New
Append CDN query parameters before URL fragments · New

What changed in this PR

Adds .NET 10 support and CDN-specific URL formatting while preserving legacy APIs.

Changes:

Adds net10.0 targeting and CI coverage.
Adds Format compatibility alias.
Adds Tencent, Aliyun, and AWS CDN URL helpers, tests, and documentation.
Review identified unresolved Pad mapping, AWS token, and URL-fragment handling issues.

File	Description
`source/ResizingClient/ResizingUtil.cs`	Adds formatting APIs and CDN URL generation.
`source/ResizingClient/ResizingClient.csproj`	Adds the .NET 10 target.
`source/ResizingClient.Tests/UnitTest1.cs`	Tests new formatting APIs.
`README.md`	Documents supported frameworks and APIs.
`.github/workflows/dotnet-ci-release.yml`	Updates framework-specific CI builds.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

+            var fitToken = "fit_cover";
+            switch (mode)
+            {
+                case ResizingMode.Max:
+                    fitToken = "fit_contain";
+                    break;
+                case ResizingMode.Pad:
+                    fitToken = "fit_pad";
+                    break;
+            }


+            var query = mode == ResizingMode.Crop
+                ? $"imageMogr2/thumbnail/{width}x{height}/gravity/center/crop/{width}x{height}"
+                : $"imageMogr2/thumbnail/{width}x{height}";


+        static string AppendQuery(string url, string query)
+        {
+            if (string.IsNullOrWhiteSpace(url)) return url;
+            return url.IndexOf('?', StringComparison.Ordinal) >= 0
+                ? $"{url}&{query}"
+                : $"{url}?{query}";
+        }


chsword · 2026-09-27T01:50:14Z

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, apply the suggestion exactly.

Do not make changes beyond what is described in the linked review thread.

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot · 2026-09-27T01:55:56Z

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, appl...

Addressed in 565368b.

chsword · 2026-09-27T02:34:16Z

@openai-code-agent[agent] 补全单元测试

chsword · 2026-09-27T02:48:41Z

@copilot 补全单元测试

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot · 2026-09-27T02:50:14Z

@copilot 补全单元测试

Addressed in fc2d960.

chsword · 2026-09-27T03:28:14Z

ci 失败是不是需要修改 @copilot

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot · 2026-09-27T03:30:52Z

ci 失败是不是需要修改 @copilot

需要，我已经修复了导致 Windows build-server 失败的 net45 reference assemblies 问题，见 05924b2。

…6-2304-4181-a26f-944951529bcd Add .NET 10 multi-targeting and CDN-style URL formatting compatibility

Initial plan

b6e7470

Codex AI assigned Codex and chsword Sep 27, 2026

Codex AI linked an issue Sep 27, 2026 that may be closed by this pull request

兼容性升级 #14

Closed

Codex started work on behalf of chsword September 27, 2026 00:48

View session

Add .NET 10 target and CDN URL formatting compatibility

9c61952

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Codex AI changed the title ~~[WIP] Update support for .net10 while maintaining compatibility~~ Add .NET 10 multi-targeting and CDN-style URL formatting compatibility Sep 27, 2026

Codex finished work on behalf of chsword September 27, 2026 00:55

Codex AI requested a review from chsword September 27, 2026 00:55

chsword requested a lite review from Copilot September 27, 2026 01:24

Copilot started reviewing on behalf of chsword September 27, 2026 01:24

View session

Copilot AI reviewed Sep 27, 2026

View reviewed changes

Copilot started work on behalf of chsword September 27, 2026 01:50

View session

Copilot AI and others added 3 commits September 27, 2026 01:51

Fix CDN URL formatting review comments

a193f8f

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Add fragment query coverage

02a67b8

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Expand fragment coverage

565368b

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot finished work on behalf of chsword September 27, 2026 01:56

Copilot started work on behalf of chsword September 27, 2026 02:48

View session

Expand CDN unit test coverage

fc2d960

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot finished work on behalf of chsword September 27, 2026 02:50

Copilot started work on behalf of chsword September 27, 2026 03:28

View session

Restore net45 reference assemblies for server build

05924b2

Co-authored-by: chsword <274085+chsword@users.noreply.github.com>

Copilot finished work on behalf of chsword September 27, 2026 03:31

chsword added this pull request to stack #17 September 27, 2026 05:29

chsword marked this pull request as ready for review September 27, 2026 06:04

chsword merged commit f626c09 into master Sep 27, 2026
5 of 6 checks passed

chsword added a commit that referenced this pull request Sep 27, 2026

Merge pull request #15 from chsword/codex/fix-274085-41134369-7985095…

7526f4e

…6-2304-4181-a26f-944951529bcd Add .NET 10 multi-targeting and CDN-style URL formatting compatibility

chsword deleted the codex/fix-274085-41134369-79850956-2304-4181-a26f-944951529bcd branch September 27, 2026 06:05

Navigation Menu

Repository navigation

Add .NET 10 multi-targeting and CDN-style URL formatting compatibility - #15

Add .NET 10 multi-targeting and CDN-style URL formatting compatibility#15
chsword merged 7 commits into
masterfrom
codex/fix-274085-41134369-79850956-2304-4181-a26f-944951529bcd

Codex AI commented Sep 27, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

Codex AI commented Sep 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

chsword commented Sep 27, 2026

Uh oh!

Copilot AI commented Sep 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Codex AI commented Sep 27, 2026 •

edited

Loading