-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.4] Expose finalization errors in autoupdate API response #46493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 5.4-dev
Are you sure you want to change the base?
[5.4] Expose finalization errors in autoupdate API response #46493
Conversation
api/components/com_joomlaupdate/src/View/Updates/JsonapiView.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Richard Fath <[email protected]>
|
I have tested this item ✅ successfully on f4b53c8 Hint for other testers how I have testedOn a Linux (or if Windows: WSL) command line, I've called curl to post the request: The Of course you have to replace the Your site should be on Joomla 5.4.1, and automated updates should be enabled. Result without the PR:Result without the PR:I.e. with the PR there is an additional attribute In case of success (like here) it is an empty array. For producing an error I have crated a new update SQL script After that I've repeated the API call. (with the absolute path to the files removed). I.e. the new errors attribute contains the exception details and the call stack. Of course you should remove that bad update SQL script after the test. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46493. |
|
Thanks for the great testing instructions @richard67, unfortunately I have tried on 3 different sites and I am not getting past curl --user-agent "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:89.0) Gecko/20100101 Firefox/89.0" -H "X-JUpdate-Token: (with of course updating a. curl: (28) Failed to connect to www.domain.com port 443: Connection timed out for another site, I get Method Not Allowed and for a 4th, I get a 403 forbidden: so I'm not having much fun :/ lol |
|
I assume you added your token and just removed it from your post? |
Hi @brianteeman, yes absolutely. |
|
@exlemor SSL is ok for your site? I.e. not a self signed certificate? curl is a bit strict with that. |
These are all live sites (some testing installations, some not) all on the internet with Let's Encrypt SSL certs not self-signed. |
|
@exlemor I've allowed myself to add the missing tickmarks for quoting code (or markup) on Github to your comment with your result.
Sure that this site was reachable with https from that client where you have used curl?
Do these sites have an Do they maybe have also an |
Summary of Changes
So far, the update finalization endpoint of the autoupdate feature has not been exposing any potential cause of a issue during that execution step. This PR adds output.
Testing Instructions
Actual result BEFORE applying this Pull Request
Errors missing
Expected result AFTER applying this Pull Request
Errors present
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed