CLD-48-organization-content #43
Conversation
cartridges/int_cloudinary/cartridge/scripts/jobs/steps/cloudinaryUploadOrgContent.js
Outdated
Show resolved
Hide resolved
cartridges/int_cloudinary/cartridge/scripts/util/cloudinaryOrgUtils.js
Outdated
Show resolved
Hide resolved
cartridges/int_cloudinary/cartridge/scripts/util/cloudinaryOrgUtils.js
Outdated
Show resolved
Hide resolved
cartridges/int_cloudinary/cartridge/scripts/util/cloudinaryOrgConstants.js
Outdated
Show resolved
Hide resolved
cartridges/int_cloudinary/cartridge/scripts/jobs/steps/cloudinaryUploadOrgContent.js
Outdated
Show resolved
Hide resolved
cartridges/int_cloudinary/cartridge/scripts/jobs/steps/cloudinaryUploadOrgContent.js
Outdated
Show resolved
Hide resolved
| var changedAssetIds = jobStepHelpers.changePublicIdAndCloudFolder(assetPublicID, folder); | ||
| assetPublicID = changedAssetIds.assetPublicID; | ||
| var svcArgs = jobStepHelpers.getCldUploadSvcArgs(); | ||
| jobStepHelpers.uploadFile(cloudinaryConstants, url, null, folder, assetPublicID, null, svcArgs, null); |
There was a problem hiding this comment.
Is this where we get 401? Can you share the log line, filename and the timestamp so I can triage it in the Cloudinary logs?
There was a problem hiding this comment.
@asad-rafter Thanks for the code change. Please share the output of it
There was a problem hiding this comment.
@yuval-cloudinary output means the logs you want?
There was a problem hiding this comment.
yes @asad-rafter , the new logs with the 401
There was a problem hiding this comment.
There was a problem hiding this comment.
@yuval-cloudinary we are still getting the 401, cloudinary is still trying to access the content and it returns the 401. I have attached the latest logs.
Job-custom.cloudinary.UploadOrgStaticContent-a90b2e248d351a08de8a632441.log
There was a problem hiding this comment.
@asad-rafter Was the 401 issue resolved after removing the environment protection?
|
@yuval-cloudinary the requested changes have been made |
| @@ -178,7 +178,7 @@ var uploadFile = function (cloudinaryConstants, asset, tags, assignedFolder, ass | |||
| if (uploadResult.errorCode === cloudinaryConstants.ERROR_CODES.UNAUTHORIZED) { | |||
| jobLogger.error('Error occurred while connecting with service due to invalid credentials, message: {0}', uploadResult.message); | |||
| } else { | |||
| jobLogger.error('Error uploading file: {0}, message: {1}', asset, uploadResult.message); | |||
| jobLogger.error('jobStepHelpers~uploadFile: Error uploading file: {0}, message: {1}, line number 177', asset, uploadResult.message); | |||
There was a problem hiding this comment.
@asad-rafter Let's revert this line for log message consistency
No description provided.