Skip to content
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

Don't fall thru when v3 feed failed #1253

Closed
feiling opened this issue Aug 25, 2015 · 1 comment
Closed

Don't fall thru when v3 feed failed #1253

feiling opened this issue Aug 25, 2015 · 1 comment
Assignees
Milestone

Comments

@feiling
Copy link

feiling commented Aug 25, 2015

In ServiceIndexResourceV3Provider.TryCreate(), when there are errors connecting to a v3 feed, the method returns (false, null). This allows fall thru and cause the feed to be treated as V2. We should stop the fall thru instead.

@feiling feiling added this to the 3.2 milestone Aug 25, 2015
@feiling feiling self-assigned this Aug 25, 2015
feiling pushed a commit to NuGetArchive/NuGet3 that referenced this issue Aug 28, 2015
Now if there is a failure, an exception will get thrown so GetResource fails immediately so no fall thru will happens. Also the V3 resource is cached only when it is successfully created.
feiling pushed a commit to NuGetArchive/NuGet3 that referenced this issue Aug 28, 2015
Now if there is a failure, an exception will get thrown so GetResource fails immediately so no fall thru will happens. Also the V3 resource is cached only when it is successfully created.
@emgarten
Copy link
Member

Fixed with NuGetArchive/NuGet3@11a75e0

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

No branches or pull requests

4 participants