Skip to content

feat(aspnetcore): HTTP endpoints to trigger and manage reports (PR 7) - #32

Merged
thiagoluga merged 14 commits into
masterfrom
feat/aspnetcore
Jun 26, 2026
Merged

feat(aspnetcore): HTTP endpoints to trigger and manage reports (PR 7)#32
thiagoluga merged 14 commits into
masterfrom
feat/aspnetcore

harden(core): sanitize artifact file name; narrow cleanup catch

283735d
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Jun 26, 2026 in 4s

8 new alerts

New alerts in code changed by this pull request

  • 1 warning
  • 7 notes

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 42 in tests/NeoReports.AspNetCore.IntegrationTests/EndpointsTests.cs

See this annotation in the file changed.

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
Object
Loading
- the conversion can be done implicitly.

Check notice on line 14 in src/NeoReports.Core/Artifacts/FileSystemArtifactStore.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.

Check notice on line 40 in src/NeoReports.Core/Artifacts/FileSystemArtifactStore.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.

Check notice on line 69 in src/NeoReports.Core/Artifacts/FileSystemArtifactStore.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop
implicitly filters its target sequence
Loading
- consider filtering the sequence explicitly using '.Where(...)'.

Check notice on line 97 in src/NeoReports.Core/Artifacts/FileSystemArtifactStore.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.

Check notice on line 62 in tests/NeoReports.AspNetCore.IntegrationTests/TestApp.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note test

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.

Check notice on line 9 in tests/NeoReports.Core.UnitTests/FileSystemArtifactStoreTests.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note test

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.

Check notice on line 15 in tests/NeoReports.Core.UnitTests/FileSystemArtifactStoreTests.cs

See this annotation in the file changed.

Code scanning / CodeQL

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments Note test

Call to 'System.IO.Path.Combine' may silently drop its earlier arguments.