ContentHubEntityExportController sets Content-Type request header, yet there is no request body

Created on 26 October 2016, about 8 years ago
Updated 27 February 2023, almost 2 years ago

\Drupal\acquia_contenthub\Controller\ContentHubEntityExportController::getEntityCdfByInternalRequest() sets the Content-Type request header, but that header only makes sense when there is a request body.

AFAICT it's only being set because \Drupal\acquia_contenthub\Client\ClientManager::getRequestSignature() is called later and needs that header to be set. Which would suggest this is a code smell.

🐛 Bug report
Status

Closed: outdated

Version

1.0

Component

Code

Created by

🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024