Uploaded image for project: 'Artifactory Binary Repository'
  1. Artifactory Binary Repository
  2. RTFACT-9741

Artifactory nuget integration broken with VS2015 Update2

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Affects Version/s: 4.4.0, 4.7.0
    • Fix Version/s: 4.7.1
    • Component/s: NuGet
    • Labels:
    • Severity:
      Critical

      Description

      Nuget is case insensitive and we are not in the way we parse parameters.
      In VS2015 Update2 (which integrate Nuget 3.4) they made small changes which brakes totally our integration of nuget (it is impossible to install any nuget package).

      Details :

      before this version a call to find a package was :

      http://localhost:8081/artifactory/api/nuget/nuget/FindPackagesById()?id='Microsoft.Web.Infrastructure'

      we parsed it correctly and sent following request to nuget gallery :
      https://www.nuget.org/api/v2/FindPackagesById()?includePrerelease=false&id='Microsoft.Web.Infrastructure'&includeAllVersions=false

      ___________________
      Now (from Update 2 released on 30th of March) the request is :

      http://localhost:8081/artifactory/api/nuget/nuget/FindPackagesById()?Id='Microsoft.AspNet.WebPages'

      and we are sending this request to official gallery :

      https://www.nuget.org/api/v2/FindPackagesById()?includePrerelease=false&includeAllVersions=false

      We don't send the Id value, because in class NuGetSearchParameters.java, method NuGetSearchParameters we do

      id = requestParams.getFirst("id");

      so "Id" is not recognized.

      ______________________________

      If you use nuget gallery directly it is the same result with :

      https://www.nuget.org/api/v2/FindPackagesById()?Id='Microsoft.AspNet.WebPages'&includePrerelease=true&includeAllVersions=false

      or

      https://www.nuget.org/api/v2/FindPackagesById()?id='Microsoft.AspNet.WebPages'&iNcLuDePreRelease=true&incLudeAllVersions=false

      This is the same with all parameters. In general nuget is case insensitive (I guess it is because IIS is a case insensitive web server)

        Attachments

          Activity

              People

              Assignee:
              shayy Shay Yaakov (Inactive)
              Reporter:
              jroquelaure Jonathan Roquelaure
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Sync Status

                  Connection: RTFACT Sync
                  RTMID-9741 -
                  SYNCHRONIZED
                  • Last Sync Date: