🐛(back) validate document content in serializer

[lunika]

Purpose

We recently extract images url in the content. For this, we assume that the document content is always in base64. We enforce this assumptim by checking if it's a valide base64 in the serializer.

Proposal

• 🐛(back) validate document content in serializer

[Copilot]

Pull Request Overview

This PR aims to enforce that document content is always in valid base64 format by adding a new serializer validation and corresponding tests.

• Added a new test case to validate that non-base64 content is rejected.
• Implemented validate_content in the serializer to check for valid base64 using b64decode.
• Updated the CHANGELOG with information about the bug fix.

Reviewed Changes

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

File	Description
src/backend/core/tests/documents/test_api_documents_update.py	Added test to ensure documents with invalid base64 content raise a validation error.
src/backend/core/api/serializers.py	Added validation logic for the content field to enforce base64 encoding.
CHANGELOG.md	Updated changelog to reflect the bug fix.

Comments suppressed due to low confidence (1)

src/backend/core/api/serializers.py:309

• Consider using b64decode(value, validate=True) to enforce strict base64 validation and prevent potentially accepting invalid characters.

b64decode(value)

[Copilot]

Pull Request Overview

This PR adds validation for document content to ensure it is correctly encoded in base64 before processing.

• Adds a new test to verify that a document update with invalid base64 content is rejected
• Implements a new validator in the serializer to enforce valid base64 encoding
• Updates the CHANGELOG to reference the fix

Reviewed Changes

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

File	Description
src/backend/core/tests/documents/test_api_documents_update.py	Adds a test case for invalid base64 document content
src/backend/core/api/serializers.py	Introduces a validate_content method to check base64 content
CHANGELOG.md	Updates changelog with the bug fix entry

[Copilot]

Pull Request Overview

This pull request adds server-side validation for document content by ensuring that input strings are valid base64 encoded data. Key changes include:

• Adding a new serializer method (validate_content) in the API serializer to enforce base64 validation.
• Creating a corresponding test (test_api_documents_update_invalid_content) to confirm that invalid content returns a 400 error with the proper message.
• Updating the changelog to document the fix.

Reviewed Changes

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

File	Description
src/backend/core/tests/documents/test_api_documents_update.py	Added test to check behavior for non-base64 encoded content
src/backend/core/api/serializers.py	Introduced validate_content method to enforce base64 content format
CHANGELOG.md	Updated changelog to document the base64 validation fix

[Copilot]

Pull Request Overview

This PR implements server-side validation for document content to ensure that it is valid base64-encoded data and includes an associated test case and changelog updates.

• Add a new serializer method (validate_content) to check base64 validity
• Introduce a test case to trigger the validation error when non-base64 content is provided
• Update the changelog to reflect the fix

Reviewed Changes

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

File	Description
src/backend/core/tests/documents/test_api_documents_update.py	Added a test case for invalid base64 document content
src/backend/core/api/serializers.py	Added validate_content method to enforce base64 validation on document content
CHANGELOG.md	Updated changelog with the base64 validation fix