While syncing content with the "Physical file" import processor plugin enabled in the client's import config and using "Basic Auth" as the authorization method in the client's remote config, there is a call made to the server using the request() method in entity_share/modules/entity_share_client/src/Plugin/EntityShareClient/Processor/PhysicalFile.php to fetch the file. This call is eventually completing the authentication process via the user_login_form using the provided credentials.
In our case, the authentication fails, since we are using the
password_encrypt →
module, wherein the password needs to be an encrypted one. This led to us wondering if this call from PhysicalFile.php could be updated to jsonApiRequest() instead of request(). This change works fine for our use-case but just need to know if there is any particular reason why this call is using request() instead of jsonApiRequest() from RemoteManager.php
/admin/config/services/entity_share/channel Also enable basic_auth module on this site./admin/config/services/entity_share/import_config and add remote website as http://local.abc.com/ and select Basic Auth as the authorization method and specify the credentials of the user configured in step 2 on server./admin/content/entity_share/pull on http://local.xyz.com/ and see that the newly created content on server is available here to import. Select the content to Import asynchronously and execute cron.Change the following line of code in /entity_share/modules/entity_share_client/src/Plugin/EntityShareClient/Processor/PhysicalFile.php:
$response = $this->remoteManager->request($runtime_import_context->getRemote(), 'GET', $remote_file_url);
to:
$response = $this->remoteManager->jsonApiRequest($runtime_import_context->getRemote(), 'GET', $remote_file_url);
Discuss if there is an alternate approach to handle this.
Needs review
3.0
Entity Share Client
The last submitted patch, 2: 3401508-2.patch, failed testing.
View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.