Skip to content

fix: dummy virus scan service#86

Merged
mkilgus-at merged 1 commit intomainfrom
fix/DI
Apr 30, 2026
Merged

fix: dummy virus scan service#86
mkilgus-at merged 1 commit intomainfrom
fix/DI

Conversation

@mkilgus-at
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 30, 2026 15:21
@mkilgus-at mkilgus-at enabled auto-merge (squash) April 30, 2026 15:22
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the receiver package to include a fix for the dummy virus scan implementation used when SkipVirusScan is enabled, and publishes a new patch version.

Changes:

  • Bump Receiver.Publish package version to 1.2.11.
  • Fix DI compatibility for DummyVirusScanService by using ILogger<DummyVirusScanService>.
  • Add 1.2.11 changelog entry describing the fix.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/Publish/Receiver.Publish/Receiver.Publish.csproj Bumps package version to 1.2.11 for the release containing the fix.
src/Infrastructure/src/VirusScan/DummyVirusScanService.cs Switches to typed logger injection to avoid missing service resolution when SkipVirusScan is enabled.
src/CHANGELOG.md Documents the 1.2.11 fix in the changelog.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/CHANGELOG.md
@mkilgus-at mkilgus-at merged commit 23bc243 into main Apr 30, 2026
12 checks passed
@mkilgus-at mkilgus-at disabled auto-merge April 30, 2026 15:26
@mkilgus-at mkilgus-at deleted the fix/DI branch April 30, 2026 15:26
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.

2 participants