cake-build / cake-build/cake

NuGetPush alias should capture NuGet.exe output

Open
#2,354 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
4.2k
Forks
778
Avg merge
1h 15m
Merged PRs (30d)
19

Description

When pushing with NuGetPush and expecting something like a failure because of a disabled package override on the NuGet Feed.

The expected NuGet.exe output is:
`Response status code does not indicate success: 403 (Package Already Exists)`

However it is not captured and instead the actual return in OnError(exception => { ... }); is:
`NuGet: Process returned an error (exit code 1)`

Relevant Gitters:
[November 8, 2018 8:10 AM](https://gitter.im/cake-build/cake?at=5be3e1532138723405d2cbf5)
[November 8, 2018 9:53 AM](https://gitter.im/cake-build/cake?at=5be3f98092b7d1172156dfb8)
[November 8, 2018 11:20 AM](https://gitter.im/cake-build/cake?at=5be40e07d001b917202969c9)

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the NuGetPush failure described in the issue and trace the NuGetPush alias into its OnError(exception => { ... }) handling. Compare the captured error with NuGet.exe's expected 403 output; done means the response text is available to the error callback instead of only the generic exit-code message.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
cli, tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.