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

Adds support for NuGet V2 upstream package sources #630

Merged
merged 4 commits into from
Mar 3, 2021
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/BaGet.Core/BaGet.Core.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="$(MicrosoftExtensionsPackageVersion)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(MicrosoftExtensionsPackageVersion)" />
<PackageReference Include="NuGet.Packaging" Version="$(NuGetPackageVersion)" />
<PackageReference Include="NuGet.Protocol" Version="$(NuGetPackageVersion)" />
patriksvensson marked this conversation as resolved.
Show resolved Hide resolved
<PackageReference Include="System.Reflection.Metadata" Version="1.6.0" />
</ItemGroup>

Expand Down
5 changes: 5 additions & 0 deletions src/BaGet.Core/Configuration/MirrorOptions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,11 @@ public class MirrorOptions : IValidatableObject
/// </summary>
public Uri PackageSource { get; set; }
Copy link
Owner

Choose a reason for hiding this comment

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

Hey this is a tiny bit of feature creep, but I think you can leverage the options validation here by adding attributes on the PackageSource property:

/// <summary>
/// The v3 index that will be mirrored.
/// </summary>
[RequiredIf(nameof(Enabled))]
[RequiredIf(nameof(Legacy))]
public Uri PackageSource { get; set; }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Excellent idea. However when trying this it complains that only one IfRequired attribute can be used.

As I workaround, we can do the following, that will work but will show the error message Invalid 'Mirror' configs: 'PackageSource' is required because 'EnabledOrLegacy' has a value of 'True'. which does not reflect the settings properly, but maybe this is OK?

/// <summary>
/// If true, packages that aren't found locally will be indexed
/// using the upstream source.
/// </summary>
public bool Enabled { get; set; }

/// <summary>
/// The v3 index that will be mirrored.
/// </summary>
[RequiredIf(nameof(EnabledOrLegacy), true)]
public Uri PackageSource { get; set; }

/// <summary>
/// Whether or not the package source is a v2 package source feed.
/// </summary>
public bool Legacy { get; set; }

[NotMapped]
public bool EnabledOrLegacy => Enabled || Legacy;

Copy link
Owner

@loic-sharma loic-sharma Mar 3, 2021

Choose a reason for hiding this comment

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

Ah good catch! It looks like you can support multiple attribute usages if we:

  1. Set the attribute's AttributeUsageAttribute.AllowMultiple to true
  2. Override the TypeId property (see this)

For an example of a validation attribute that supports multiple uses, see CustomValidationAttribute.

I'll do this in a follow-up PR since this is minor and I've blocked your changes for a little too long already!


/// <summary>
/// Whether or not the package source is a v2 package source feed.
/// </summary>
public bool Legacy { get; set; }

/// <summary>
/// The time before a download from the package source times out.
/// </summary>
Expand Down
13 changes: 12 additions & 1 deletion src/BaGet.Core/Extensions/DependencyInjectionExtensions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -99,11 +99,14 @@ private static void AddBaGetServices(this IServiceCollection services)
services.TryAddTransient<DatabaseSearchService>();
services.TryAddTransient<FileStorageService>();
services.TryAddTransient<MirrorService>();
services.TryAddTransient<MirrorV2Client>();
services.TryAddTransient<MirrorV3Client>();
services.TryAddTransient<NullMirrorService>();
services.TryAddSingleton<NullStorageService>();
services.TryAddTransient<PackageService>();

services.TryAddTransient(IMirrorServiceFactory);
services.TryAddTransient(IMirrorNuGetClientFactory);
}

private static void AddDefaultProviders(this IServiceCollection services)
Expand Down Expand Up @@ -195,8 +198,16 @@ private static IMirrorService IMirrorServiceFactory(IServiceProvider provider)
{
var options = provider.GetRequiredService<IOptionsSnapshot<MirrorOptions>>();
var service = options.Value.Enabled ? typeof(MirrorService) : typeof(NullMirrorService);

return (IMirrorService)provider.GetRequiredService(service);
}

private static IMirrorNuGetClient IMirrorNuGetClientFactory(IServiceProvider provider)
{
var options = provider.GetRequiredService<IOptionsSnapshot<MirrorOptions>>();
var service = options.Value.Legacy ? typeof(MirrorV2Client) : typeof(MirrorV3Client);

return (IMirrorNuGetClient)provider.GetRequiredService(service);
}
}
}
114 changes: 114 additions & 0 deletions src/BaGet.Core/Mirror/Clients/MirrorV2Client.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,114 @@
using BaGet.Protocol.Models;
using Microsoft.Extensions.Options;
using NuGet.Common;
using NuGet.Configuration;
using NuGet.Protocol;
using NuGet.Protocol.Core.Types;
using NuGet.Versioning;
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Threading;
using System.Threading.Tasks;

namespace BaGet.Core
{
internal sealed class MirrorV2Client : IMirrorNuGetClient
{
private readonly ILogger _logger;
private readonly SourceCacheContext _cache;
private readonly SourceRepository _repository;

public MirrorV2Client(IOptionsSnapshot<MirrorOptions> options)
{
if (options is null)
{
throw new ArgumentNullException(nameof(options));
}

if (options.Value?.PackageSource?.AbsolutePath == null)
{
throw new ArgumentException("No mirror package source has been set.");
Copy link
Owner

Choose a reason for hiding this comment

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

Ideally this should be done at startup by ASP.NET Core's options validation. See my comment up above: https://github.com/loic-sharma/BaGet/pull/630/files#r582447197

If that idea works, you should be able to remove this check.

}

_logger = NullLogger.Instance;
_cache = new SourceCacheContext();
_repository = Repository.Factory.GetCoreV2(new PackageSource(options.Value.PackageSource.AbsoluteUri));
}

public async Task<IReadOnlyList<NuGetVersion>> ListPackageVersionsAsync(string id, bool includeUnlisted, CancellationToken cancellationToken)
{
var resource = await _repository.GetResourceAsync<FindPackageByIdResource>();
var versions = await resource.GetAllVersionsAsync(id, _cache, _logger, cancellationToken);

return versions.ToList();
}

public async Task<IReadOnlyList<PackageMetadata>> GetPackageMetadataAsync(string id, CancellationToken cancellationToken)
{
var resource = await _repository.GetResourceAsync<PackageMetadataResource>();
var packages = await resource.GetMetadataAsync(id, includePrerelease: true, includeUnlisted: false, _cache, _logger, cancellationToken);

var result = new List<PackageMetadata>();
foreach (var package in packages)
{
result.Add(new PackageMetadata
{
Authors = package.Authors,
Description = package.Description,
IconUrl = package.IconUrl?.AbsoluteUri,
LicenseUrl = package.LicenseUrl?.AbsoluteUri,
Listed = package.IsListed,
PackageId = id,
Summary = package.Summary,
Version = package.Identity.Version.ToString(),
Tags = package.Tags?.Split(new[] { ';' }, StringSplitOptions.RemoveEmptyEntries),
Title = package.Title,
RequireLicenseAcceptance = package.RequireLicenseAcceptance,
Published = package.Published?.UtcDateTime ?? DateTimeOffset.MinValue,
ProjectUrl = package.ProjectUrl?.AbsoluteUri,
DependencyGroups = GetDependencies(package),
});
}

return result;
}

public async Task<Stream> DownloadPackageAsync(string id, NuGetVersion version, CancellationToken cancellationToken)
{
var packageStream = new MemoryStream();
var resource = await _repository.GetResourceAsync<FindPackageByIdResource>();
await resource.CopyNupkgToStreamAsync(id, version, packageStream, _cache, _logger, cancellationToken);
packageStream.Seek(0, SeekOrigin.Begin);

return packageStream;
}

private IReadOnlyList<DependencyGroupItem> GetDependencies(IPackageSearchMetadata package)
{
var groupItems = new List<DependencyGroupItem>();
foreach (var set in package.DependencySets)
{
var item = new DependencyGroupItem
{
TargetFramework = set.TargetFramework.Framework,
Dependencies = new List<DependencyItem>()
};

foreach (var dependency in set.Packages)
{
item.Dependencies.Add(new DependencyItem
{
Id = dependency.Id,
Range = dependency.VersionRange.ToNormalizedString(),
});
}

groupItems.Add(item);
}

return groupItems;
}
}
}
36 changes: 36 additions & 0 deletions src/BaGet.Core/Mirror/Clients/MirrorV3Client.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
using BaGet.Protocol;
using BaGet.Protocol.Models;
using NuGet.Versioning;
using System;
using System.Collections.Generic;
using System.IO;
using System.Threading;
using System.Threading.Tasks;

namespace BaGet.Core
{
internal sealed class MirrorV3Client : IMirrorNuGetClient
{
private readonly NuGetClient _client;

public MirrorV3Client(NuGetClient client)
{
_client = client ?? throw new ArgumentNullException(nameof(client));
}

public Task<Stream> DownloadPackageAsync(string id, NuGetVersion version, CancellationToken cancellationToken)
{
return _client.DownloadPackageAsync(id, version, cancellationToken);
patriksvensson marked this conversation as resolved.
Show resolved Hide resolved
}

public Task<IReadOnlyList<PackageMetadata>> GetPackageMetadataAsync(string id, CancellationToken cancellationToken)
{
return _client.GetPackageMetadataAsync(id, cancellationToken);
}

public Task<IReadOnlyList<NuGetVersion>> ListPackageVersionsAsync(string id, bool includeUnlisted, CancellationToken cancellationToken)
{
return _client.ListPackageVersionsAsync(id, includeUnlisted, cancellationToken);
}
}
}
16 changes: 16 additions & 0 deletions src/BaGet.Core/Mirror/IMirrorNuGetClient.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
using BaGet.Protocol.Models;
using NuGet.Versioning;
using System.Collections.Generic;
using System.IO;
using System.Threading;
using System.Threading.Tasks;

namespace BaGet.Core
{
public interface IMirrorNuGetClient
{
Task<IReadOnlyList<NuGetVersion>> ListPackageVersionsAsync(string id, bool includeUnlisted, CancellationToken cancellationToken);
Task<IReadOnlyList<PackageMetadata>> GetPackageMetadataAsync(string id, CancellationToken cancellationToken);
Task<Stream> DownloadPackageAsync(string id, NuGetVersion version, CancellationToken cancellationToken);
}
}
8 changes: 8 additions & 0 deletions src/BaGet.Core/Mirror/MirrorNuGetClientAdapter.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
using System;
using System.Collections.Generic;
using System.Text;

namespace BaGet.Core.Mirror
{
patriksvensson marked this conversation as resolved.
Show resolved Hide resolved

}
17 changes: 15 additions & 2 deletions src/BaGet.Core/Mirror/MirrorService.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,13 @@ namespace BaGet.Core
public class MirrorService : IMirrorService
{
private readonly IPackageService _localPackages;
private readonly NuGetClient _upstreamClient;
private readonly IMirrorNuGetClient _upstreamClient;
private readonly IPackageIndexingService _indexer;
private readonly ILogger<MirrorService> _logger;

public MirrorService(
IPackageService localPackages,
NuGetClient upstreamClient,
IMirrorNuGetClient upstreamClient,
IPackageIndexingService indexer,
ILogger<MirrorService> logger)
{
Expand All @@ -32,6 +32,19 @@ public MirrorService(
_logger = logger ?? throw new ArgumentNullException(nameof(logger));
}

public static MirrorService Create(
IPackageService localPackages,
NuGetClient client,
IPackageIndexingService indexer,
ILogger<MirrorService> logger)
{
return new MirrorService(
localPackages,
new MirrorV3Client(client),
indexer,
logger);
}

public async Task<IReadOnlyList<NuGetVersion>> FindPackageVersionsOrNullAsync(
string id,
CancellationToken cancellationToken)
Expand Down
2 changes: 2 additions & 0 deletions src/BaGet/appsettings.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@

"Mirror": {
"Enabled": false,
// Uncomment this to use the NuGet v2 protocol
//"Legacy": true,
"PackageSource": "https://api.nuget.org/v3/index.json"
},

Expand Down
2 changes: 1 addition & 1 deletion tests/BaGet.Core.Tests/Mirror/MirrorServiceTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public FactsBase()
_upstream = new Mock<NuGetClient>();
_indexer = new Mock<IPackageIndexingService>();

_target = new MirrorService(
_target = MirrorService.Create(
_packages.Object,
_upstream.Object,
_indexer.Object,
Expand Down