Skip to content

Commit

Permalink
Merge pull request #67 from SixLabors/af/security
Browse files Browse the repository at this point in the history
Security Considerations
  • Loading branch information
JimBobSquarePants authored Apr 15, 2024
2 parents 396bdc3 + 03ece61 commit db74567
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 0 deletions.
19 changes: 19 additions & 0 deletions articles/imagesharp/security.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# Security Considerations

Image processing is a memory-intensive application. Most image processing libraries (including ImageSharp, SkiaSharp, and Magick.NET) decode images into in-memory buffers for further processing. Without additional measures, any publicly facing service that consumes images coming from untrusted sources might be vulnerable to DoS attacks attempting to deplete process memory.

Such measures can be:
- Authentication, for example by using HMAC. See [Securing Processing Commands in ImageSharp.Web](../imagesharp.web/processingcommands.md#securing-processing-commands).
- Offloading to separate services/containers.
- Placing the solution behind a reverse proxy.
- Rate Limiting.
- Imposing conservative allocation limits by configuring a custom `MemoryAllocator`:

```csharp
Configuration.Default.MemoryAllocator = MemoryAllocator.Create(new MemoryAllocatorOptions()
{
// Note that this limits the maximum image size to 64 megapixels of Rgba32.
// Any attempt to create a larger image will throw.
AllocationLimitMegabytes = 256
});
```
1 change: 1 addition & 0 deletions articles/toc.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
### [Working with Pixel Buffers](imagesharp/pixelbuffers.md)
### [Configuration](imagesharp/configuration.md)
### [Memory Management](imagesharp/memorymanagement.md)
### [Security Considerations](imagesharp/security.md)

# [ImageSharp.Drawing](imagesharp.drawing/index.md)
## [Getting Started](imagesharp.drawing/gettingstarted.md)
Expand Down

0 comments on commit db74567

Please sign in to comment.