Issue 1228: return error 400 rather than error 500 on anonymous user file download

[kuchtiak-ufal]

Resolving multiple problems with file-download, that requires License agreement:

• check whether manageUserMetadata payload contains all required metadata keys. Return error 400 otherwise
• handle empty payload in manageUserMetadata POST request - return 400 rather than Server error (500)
  (this issue is described here: Clarin Licenses: Server error when anonymous user is downloading file #1228)
• filter the payload and store only metadata keys related to Clarin license (ignore other fields from the payload)
• handle specially the "required" metadata keys that are optional, e.g. ORGANIZATION(optional)

[Copilot]

Pull request overview

This PR addresses Issue 1228 by ensuring requests to the CLARIN user-metadata “manage” endpoint return HTTP 400 (Bad Request) instead of HTTP 500 when an anonymous user submits an empty/invalid JSON body, and adds an integration test to verify that behavior.

Changes:

• Inject and reuse a Spring-managed ObjectMapper in ClarinUserMetadataRestController.
• Convert Jackson mapping errors in the bitstream “manage” endpoint into DSpaceBadRequestException (400).
• Add an IT case asserting that an empty request body returns 400 for anonymous access when anonymous downloads are allowed.

Reviewed changes

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

File	Description
dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinUserMetadataRestController.java	Adds ObjectMapper injection and maps certain JSON-body errors to 400 for the bitstream manage endpoint.
dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinUserMetadataRestControllerIT.java	Adds integration test coverage for empty JSON body returning 400.

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.