Skip to content

Show download exception message for ps1 helper - #581

Merged
Coby Allred (cobya) merged 2 commits into
microsoft:masterfrom
Saibamen:patch-1
Oct 8, 2025
Merged

Coby Allred (cobya) merged 2 commits into
microsoft:masterfrom
Saibamen:patch-1

Conversation

@Saibamen

@Saibamen Adam Stachowicz (Saibamen) commented Oct 3, 2025 •

Copy link
Copy Markdown
Contributor

Output example:

Unable to download https://github.com/microsoft/artifacts-credprovider/releases/download/v1.4.1/Microsoft.Net6.NuGet.CredentialProvider.zip to the location /tmp/CredProviderZip/Microsoft.Net6.NuGet.CredentialProvider.zip.
Exception calling "DownloadFile" with "2" argument(s): "The SSL connection could not be established, see inner exception."
Inner Exception 1: The SSL connection could not be established, see inner exception.
Inner Exception 2: The SSL connection could not be established, see inner exception.
Inner Exception 3: The remote certificate is invalid because of errors in the certificate chain: PartialChain

@cobya

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@cobya Coby Allred (cobya) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution here, we'll get this prepped for our 2.0.0 release ๐Ÿ˜„

@cobya
Coby Allred (cobya) merged commit ad4cb81 into microsoft:master Oct 8, 2025
Adam Stachowicz (Saibamen) added a commit to Saibamen/artifacts-credprovider that referenced this pull request Oct 9, 2025
Adam Stachowicz (Saibamen) added a commit to Saibamen/artifacts-credprovider that referenced this pull request Oct 9, 2025
@Saibamen

Copy link
Copy Markdown
Contributor Author

Coby Allred (@cobya) ahh sorry, I forgot to push my latest commit to really print all exceptions (the final version should look like in my first comment here).

In current master branch people will see only first inner exception, not all of them.

Please merge this PR: #587

Coby Allred (cobya) pushed a commit that referenced this pull request Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants