-
Notifications
You must be signed in to change notification settings - Fork 342
[Enhancement] Verify package ID received was package ID requested #557
Comments
Perhaps the client should do a better job with feeds, but the problem here is with the feed doing the wrong thing, not with chocolatey doing the wrong thing. Let's update this issue a bit to something more workable regarding the client doing an id match versus what was requested. |
Thanks for the quick reply. private.feed.ca is just a name I used to hide my private feed server from the world. I totally trust it as I made all the nuget listings on it. Sorry for any confusion. Here is a test with v0.9.8.27.
Again the wrong package is selected. It appears the version comparsion was a red herring, just happened to sort of match up the first time. |
What I'd need from you is a -debug output and fiddler output. |
As debug and fiddler output have sensitive information, files have been shared with ferventcoder, via g+. |
Might want to make sure it is https://plus.google.com/+RobReynolds_FerventCoder/ |
I've tried to alert +RobReynolds_FerventCoder gain in G+. If my G+ skills lacking, and if there's a better way to get these files to you, please let me know. |
I ran into this error:
Note the request for greenfoot, and the attempt to install swimmygui.
I believe the issue to be two fold.
One: PhpNuget (the only feed I got working with my systems) is broken, in that when a FindPackagesById() request comes in, it does not return all 'packagename' entries, but it returns all the entries in the feed. In this case the FindPackagesById()?id='greenfoot', gets non-greenfoot packages, as well as greenfoot packages. Chocolatey should not trust that a feed gives correct information, and it should do better checks on the package IDs returned to make sure they match the request.
Two: There may be a bug in the version matching algorithm. The greenfoot package version was 2.4.0 and swimmygui had version 24.0. That may be why swimmygui was chosen over other packages listed in the feed.
Thanks and keep up the good work.
The text was updated successfully, but these errors were encountered: