wcmio-content-package-maven-plugin swallows JSON upload error

Description

The code specifically checks the "success"-attribute in the JSON response. If that is not true, it checks the message for package existence, but then if that is not true, it reports an error. That error comes from the JSON object; which is not correct.

I would rather see the full JSON that was returned + the HTTP status code.

Activity

Show:
Stefan Seifert
July 11, 2018, 5:25 PM

this is fixed in wcm.io Content Package Maven Plugin 1.6.14

Henry Kuijpers
June 12, 2018, 9:28 AM

I found out the problem: The configuration we received contained <userId>${cq.dev.user</userId> (notice the missing }).

So you can test it with an invalid username/password. You will receive a 401, and then the "Package upload failed: " message will be shown. It should indicate something about not getting a response (and should actually show an authorization error or similar).

Stefan Seifert
June 12, 2018, 8:31 AM

how can we reproduce the issue (or test the fix if we apply one)?

do you have a simple example - e.g. a corrupt AEM package or a command line call to reproduce the problem you encountered?

Fixed

Assignee

Unassigned

Reporter

Henry Kuijpers

Labels

None

Components

Fix versions

Priority

Major