Just another simple console repro:

using System.Data.Common;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Diagnostics;

await using TestContext context = new();

await context.Database.EnsureDeletedAsync();
await context.Database.EnsureCreatedAsync();

public class TestContext : DbContext
{
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        optionsBuilder.UseNpgsql();
        optionsBuilder.AddInterceptors(new ConnectionStringInitializationInterceptor(string.Empty));
        base.OnConfiguring(optionsBuilder);
    }
}

public class ConnectionStringInitializationInterceptor : DbConnectionInterceptor
{
    private readonly string _connectionString;

    internal ConnectionStringInitializationInterceptor(string connectionString)
    {
        _connectionString = connectionString;
    }

    public override InterceptionResult ConnectionOpening(DbConnection connection, ConnectionEventData eventData, InterceptionResult result)
    {
        if (string.IsNullOrEmpty(connection.ConnectionString))
        {
            connection.ConnectionString = _connectionString;
        }

        return result;
    }

    public async override ValueTask<InterceptionResult> ConnectionOpeningAsync(DbConnection connection, ConnectionEventData eventData, InterceptionResult result, CancellationToken cancellationToken = default)
    {
        if (string.IsNullOrEmpty(connection.ConnectionString))
        {
            connection.ConnectionString = _connectionString;
        }

        return result;
    }
}

I'm not sure how far you've looked into this already, but to me it looks like there's some conflict of interest at this point.

When calling Exists inside of NpgsqlDatabaseCreator, the connection is enforced to be non-pooled.
This overwrites the empty string and lets the NpgsqlConnection throw when being cloned due to missing values.

So on one hand the ConnectionString has to be correctly configured, but on the other it should be empty to be passed to the interceptors.

Changing the connection string from an interceptor doesn't work from migrations #2880

Description

Activity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions