-
Notifications
You must be signed in to change notification settings - Fork 255
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
Chocolatey GUI not displaying error output when running as non-admin through Background Service #967
Comments
@pauby We've another customer who's come across this issue. I've added a link to your list of tickets, above. |
Chocolatey Agent runs Chocolatey CLI and reports the results back to Chocolatey GUI. As a result, all of the messages come back as "Info" messages. This adds handling to the GrabErrors method to check Info messages for clues that they're actually `Error`, `Warning`, or `Fatal` messages.
Chocolatey Agent runs Chocolatey CLI and reports the results back to Chocolatey GUI. As a result, all of the messages come back as "Info" messages. This adds handling to the GrabErrors method to check Info messages for clues that they're actually `Error`, `Warning`, or `Fatal` messages.
I have put up a PR for this issue to handle that the messages coming from Chocolatey Agent are different than a normal Chocolatey installation. However, I encountered issue #998 which is that after one Install attempt, subsequent installs do not have any log information getting intercepted by Chocolatey GUI, and so failures do not have information to report to the user. |
@corbob thanks for digging into this, and fixing up the problem! |
* hotfix/2.1.1: (build) Switch to a PowerShell step (maint) synced local '.templates/' with remote '.github/GitReleaseManager/.templates/' (build) Bump Recipe package to latest version (maint) Remove unnecessary whitespace (#996) Create config directory if it doesn't exist (maint) Remove unused pragmas (#967) Parse Info for Error, Warn, and Fatal
🎉 This issue has been resolved in version 2.1.1 🎉 The release is available on: Your GitReleaseManager bot 📦 🚀 |
* master: (build) Switch to a PowerShell step (maint) synced local '.templates/' with remote '.github/GitReleaseManager/.templates/' (build) Bump Recipe package to latest version (maint) Remove unnecessary whitespace (#996) Create config directory if it doesn't exist (maint) Remove unused pragmas (#967) Parse Info for Error, Warn, and Fatal (maint) Remove mention of services no longer used (tests) Update Pester tests to be consistent (tests) Move the Pester tests to be consistent (maint) synced local '.templates/' with remote '.github/GitReleaseManager/.templates/' (maint) Update GRM configuration
NOTE: This issue is a duplicate of chocolatey/chocolatey-licensed-issues#327 - adding this here as it potentially needs fixed in multiple places.
What You Are Seeing?
When installing a package through Chocolatey GUI. With the background service enabled and logged into the PC as a non-admin user. Chocolatey GUI says their is an error but does not give you the full content of the error message.
What is Expected?
It would help in troubleshooting if the full error was emitted within Chocolatey GUI instead of having to dig into the chocolatey log to ascertain the contents of the actual error that occurred.
How Did You Get This To Happen? (Steps to Reproduce)
choco new throw-message --version 1.0.0
choco pack
against the .nuspec file to generate the throw-message.1.0.0.nupkgchoco source add -n Local -s C:\temp --allow-self-service
the self-service will be needed for non-admin testing of the package.choco feature enable -n useBackgroundService
, the chocolatey-agent package installed, and the useBackgroundServiceWithNonAdministratorsOnly feature enabled.choco install throw-message -y
. You should get the following output:choco install throw-message -y
now being logged in to the PC as a non-admin user. The output from this:As you can see the GUI output from the 2 users is different. One shows the complete error where as the non-admin does not. Also if you look into the CLI output more you can notice the admin user's is highlighted red to indicate an error. Where the non-admin's same output is in white text, usually indicating standard/info output.
Reproduction Enviornment
OS Version: Windows 10 V 10.0.19045.0
Powershell Version: 5.1.19041.1682
Chocolatey Version: 1.1.0
choco list -lo
output from system:Related Tickets and Issues
The text was updated successfully, but these errors were encountered: