Skip to content

Хранение и алгоритмы сжатия данных - ПР 1 - Update swagger with ability to upload binary files - #6

Merged
timermakov merged 2 commits into
mainfrom
SaDCA-lab1-bin-upload
Oct 11, 2025
Merged

timermakov merged 2 commits into
mainfrom
SaDCA-lab1-bin-upload

Conversation

@timermakov

Copy link
Copy Markdown
Collaborator

No description provided.

@timermakov
timermakov requested review from Copilot and teyhd October 11, 2025 06:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Swagger/OpenAPI configuration to support binary file uploads and improves API documentation. The changes enable better testing of binary document endpoints through the Swagger UI by providing multipart/form-data upload capabilities alongside the existing raw binary endpoints.

Key changes:

  • Enhanced Swagger configuration with better schema mapping for complex types and XML documentation support
  • Added structured response models for consistent API documentation
  • Introduced new multipart/form-data endpoints for binary file uploads in Swagger UI

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
PeaceDatabase/WebApi/Program.cs Enhanced Swagger configuration with schema mappings and JSON serialization settings
PeaceDatabase/Storage/InMemoryDocumentService.cs Added TODO comment for future discussion
PeaceDatabase/Controllers/DbAndDocumentsController.cs Added response models, ProducesResponseType attributes, and new binary upload endpoints

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
Comment thread PeaceDatabase/Controllers/DbAndDocumentsController.cs Outdated
@timermakov
timermakov merged commit 43c9a61 into main Oct 11, 2025
2 checks passed
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