CBG-5103: DocID for Diagnostic API to be passed in request body instead of query param

[RIT3shSapata]

CBG-5103

Describe your PR here...

• The docID is read from the request body instead of query parameter
• renamed the errors field of Import Filter response to exception
• The OpenAPI specs have been updated in Diagnostic API: Update OpenAPI Specs #7955

Pre-review checklist

• Removed debug logging (fmt.Print, log.Print, ...)
• Logging sensitive data? Make sure it's tagged (e.g. base.UD(docID), base.MD(dbName))
• Updated relevant information in the API specifications (such as endpoint descriptions, schemas, ...) in docs/api

Dependencies (if applicable)

• Link upstream PRs
  • CBG-5089: Add support for user defined metadata in diagnostic api #7954
• Update Go module dependencies when merged

Integration Tests

• GSI=true,xattrs=true https://jenkins.sgwdev.com/job/SyncGatewayIntegration/222/

The base branch was changed.

[Copilot]

Pull request overview

This PR modifies the Diagnostic API endpoints /_sync and /_import_filter to accept the document ID in the request body instead of as a query parameter, improving consistency with REST API best practices.

Changes:

• Document ID (doc_id) is now read from the request body instead of query parameters for both sync function and import filter dry run endpoints
• The Error field in ImportFilterDryRun response has been renamed to Exception for consistency
• All test cases have been updated to reflect the new request body format

Reviewed changes

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

File	Description
rest/diagnostic_doc_api.go	Modified handlers to read doc_id from request body; added DocID field to payload structs; renamed Error to Exception in response struct
rest/diagnostic_doc_api_test.go	Updated all test cases to pass doc_id in request body instead of query parameter; updated expected response field from Error to Exception

[Copilot]

The variable docid uses inconsistent naming. The payload field is DocID (camelCase with capitalized ID), but the variable is docid (lowercase). Consider renaming to docID for consistency with Go naming conventions where acronyms should be consistently capitalized.

[Copilot]

The variable docid uses inconsistent naming. The payload field is DocID (camelCase with capitalized ID), but the variable is docid (lowercase). Consider renaming to docID for consistency with Go naming conventions where acronyms should be consistently capitalized.