Expand logging around the exception that is thrown when Chocolatey CLI is unable to communicate with a source #3258
Comments
vexx32
added a commit
to vexx32/choco
that referenced
this issue
Jul 11, 2023
Previously all that was logged was the surface exception, which in quite a few cases we've seen not yielding any useful information. Instead, log the entire exception chain, surfacing just the outermost and innermost exceptions, which we generally expect to contain the most relevant information for users.
10 tasks
corbob
pushed a commit
to vexx32/choco
that referenced
this issue
Jul 11, 2023
Previously all that was logged was the surface exception, which in quite a few cases we've seen not yielding any useful information. Instead, log the entire exception chain, surfacing just the outermost and innermost exceptions, which we generally expect to contain the most relevant information for users.
corbob
pushed a commit
to vexx32/choco
that referenced
this issue
Jul 12, 2023
Previously all that was logged was the surface exception, which in quite a few cases we've seen not yielding any useful information. Instead, log the entire exception chain, surfacing just the outermost and innermost exceptions, which we generally expect to contain the most relevant information for users.
corbob
added a commit
that referenced
this issue
Jul 12, 2023
(#3258) Expand logging for nuget resources errors
gep13
changed the title
gep13
added a commit
that referenced
this issue
Jul 26, 2023
* release/2.2.0: (21 commits) (doc) Update to indicate new package version used (maint) Add helper to split on max line lengths (#3281) Add validation for cache folder permissions (#3264) Update to latest Chocolatey.NuGet.Client (#3264) Ignore lock folders in cache directories (#3186) Remove easter egg (doc) Improve error message for defaultPushSource (tests) Clear HTTP Cache before getting packages (#3258) Expand logging for nuget resources errors (maint) Set file encoding to include BOM (#3237) Reduce number of queries for dependencies (#3231) Add tests to ensure package listing (maint) Remove unnecessary using statements (#3231) Don't refresh local package info during upgrade no-ops (build) Update to latest recipe package (doc) Minor corrections to wording (#3242) Add a script to run Authenticated tests (#3242) Attempt default credentials for sources (maint) Fix incorrect naming style uses (doc) Apply scripting best practices to output ...
|
🎉 This issue has been resolved in version 2.2.0 🎉 The release is available on: Your GitReleaseManager bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist
Is Your Feature Request Related To A Problem? Please describe.
There are times when Chocolatey CLI isn't able to communicate with a source. When this happens, the warning that is shown at the command line is typically the following:
From what we have seen, the reason that this can happen range from:
We need to provide more information in the Chocolatey log file when this happens, so that we can attempt to point people in the right direction.
Describe The Solution. Why is it needed?
When the exception:
Is thrown, there is typically a number of inner exceptions, that provide additional detail on the problem.
We need to enumerate through these exceptions, and record the message from each of them in the log file (NOTE: We aren't planning on emitting these to the CLI), so that we can instruct folks on how to resolve the problem.
In addition to capturing this output in the log file, we will also include a ch0.co short link to the documentation that is linked below, so that folks can try to figure out what is going in.
Additional Context
N/A
Related Issues
The text was updated successfully, but these errors were encountered: