Skip to content

Route markdown images through atc-api and refresh deps#32

Merged
davidkallesen merged 2 commits into
mainfrom
feature/use-atc-api-for-nuget
May 1, 2026
Merged

Route markdown images through atc-api and refresh deps#32
davidkallesen merged 2 commits into
mainfrom
feature/use-atc-api-for-nuget

Conversation

@davidkallesen

Copy link
Copy Markdown
Contributor

Summary

  • Route markdown images through the atc-api proxy where supported
  • Add atc-api URL helpers for the new external-raw and avatar endpoints
  • Refresh ATC coding rules and bump analyzers, runtime, and test deps

Changes

✨ Features

  • Add BuildExternalRawFileUrl, BuildAvatarUrl, IsExternalProxyOwner
  • Route atc-net markdown images through the repo-scoped proxy
  • Route allow-listed external owners through /github/raw proxy
  • Fall back to raw.githubusercontent.com for non-allow-listed owners

📦 Dependencies

  • Bump Atc / Atc.DotNet 3.0.18 → 3.0.67
  • Bump ASP.NET WASM and Caching/Http extensions 10.0.5 → 10.0.7
  • Bump Markdig 1.1.2 → 1.1.3 and MudBlazor 9.2.0 → 9.4.0
  • Bump Microsoft.SourceLink.GitHub 10.0.201 → 10.0.203
  • Bump test SDK 18.5.1, CodeCoverage 18.6.2, TrxReport 2.2.2
  • Bump Atc.Analyzer 0.1.23, Meziantou 3.0.58, Sonar 10.25

🔧 Configuration

  • Update .editorconfig to atc-coding-rules 1.0.1 / Blazor 1.0.2
  • Add Blazor analyzers BL0007–BL0010 to the rule set

🔥 Removal

  • Drop Asyncify and SecurityCodeScan rules from the rule set

Add AtcApiUrlBuilder helpers for the new /github/raw/{owner}/{repo} and
/github/users/{userId}/avatar atc-api endpoints, and switch the markdown
image rewriter to use them: atc-net images use the existing repo-scoped
proxy, allow-listed external owners (microsoft, dotnet, Azure, github)
use the new external proxy, and other owners fall back to direct
raw.githubusercontent.com URLs (the atc-api allow-list excludes them).
  - Update .editorconfig files to atc-coding-rules 1.0.1 (root/src/test) and 1.0.2 (Blazor) — drop Asyncify and SecurityCodeScan
  rules and add BL0007–BL0010 Blazor analyzers
  - Bump analyzers: Atc.Analyzer 0.1.22 → 0.1.23, Meziantou.Analyzer 3.0.29 → 3.0.58, SonarAnalyzer.CSharp 10.22 → 10.25
  - Bump runtime/library deps: Atc(.DotNet) 3.0.18 → 3.0.67, ASP.NET WASM 10.0.5 → 10.0.7, Caching/Http extensions 10.0.5 → 10.0.7,
  Markdig 1.1.2 → 1.1.3, MudBlazor 9.2.0 → 9.4.0, Microsoft.SourceLink.GitHub 10.0.201 → 10.0.203
  - Bump test SDK to 18.5.1 with matching CodeCoverage 18.6.2 and TrxReport 2.2.2
@sonarqubecloud

sonarqubecloud Bot commented May 1, 2026

Copy link
Copy Markdown

@davidkallesen davidkallesen merged commit e3738a7 into main May 1, 2026
6 checks passed
@davidkallesen davidkallesen deleted the feature/use-atc-api-for-nuget branch May 1, 2026 14:30
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.

1 participant