Fix: Migration bug on old instances (Could not cast or convert from System.String to BTCPayServer.DerivationSchemeSettings)

This commit is contained in:
nicolas.dorier 2025-01-14 22:23:16 +09:00
parent 6ae36825d5
commit d89fb5ec4e
No known key found for this signature in database
GPG key ID: 6618763EF09186FE
2 changed files with 33 additions and 1 deletions

View file

@ -224,6 +224,12 @@ namespace BTCPayServer.Hosting
settings.MigrateStoreExcludedPaymentMethods = true;
await _Settings.UpdateSetting(settings);
}
if (!settings.MigrateOldDerivationSchemes)
{
await MigrateOldDerivationSchemes();
settings.MigrateOldDerivationSchemes = true;
await _Settings.UpdateSetting(settings);
}
}
catch (Exception ex)
{
@ -232,6 +238,31 @@ namespace BTCPayServer.Hosting
}
}
private async Task MigrateOldDerivationSchemes()
{
await using var ctx = _DBContextFactory.CreateContext();
var stores = await ctx.Stores.ToArrayAsync();
foreach (var store in stores)
{
if (string.IsNullOrEmpty(store.DerivationStrategies))
continue;
JObject strategies = JObject.Parse(store.DerivationStrategies);
if (strategies is null)
continue;
foreach (var prop in strategies.Properties())
{
var h = _handlers.TryGet(new PaymentMethodId(prop.Name));
if (h is not BitcoinLikePaymentHandler bh ||
prop.Value is not JValue { Type: JTokenType.String, Value: string v })
continue;
var settings = DerivationSchemeSettings.Parse(v, bh.Network);
prop.Value = JToken.FromObject(settings, bh.Serializer);
store.DerivationStrategies = strategies.ToString();
}
}
await ctx.SaveChangesAsync();
}
private async Task MigrateStoreExcludedPaymentMethods()
{
await using var ctx = _DBContextFactory.CreateContext();

View file

@ -32,6 +32,7 @@ namespace BTCPayServer.Services
public bool MigrateAppYmlToJson { get; set; }
public bool MigrateToStoreConfig { get; set; }
public bool MigrateBlockExplorerLinks { get; set; }
public bool MigrateStoreExcludedPaymentMethods { get; internal set; }
public bool MigrateStoreExcludedPaymentMethods { get; set; }
public bool MigrateOldDerivationSchemes { get; set; }
}
}