Skip to content

fix: NRE during disposal of stubbed MqttTransport#1475

Merged
jeremydmiller merged 1 commit intoJasperFx:mainfrom
vzam:vipri/mqtt-dispose
Jun 5, 2025
Merged

fix: NRE during disposal of stubbed MqttTransport#1475
jeremydmiller merged 1 commit intoJasperFx:mainfrom
vzam:vipri/mqtt-dispose

Conversation

@vzam
Copy link
Contributor

@vzam vzam commented Jun 5, 2025

When external transports are disabled, the current implementation of MqttTransport would throw a NullReferenceException upon disposal. This PR aims at fixing that.

Minimal reproduction of issue:

using Wolverine;
using Wolverine.MQTT;

var host = Host.CreateDefaultBuilder()
    .UseWolverine(opts =>
    {
        opts.UseMqttWithLocalBroker();
        opts.StubAllExternalTransports();
    });

var app = host.Build();

await app.StartAsync();

app.Dispose();

@jeremydmiller jeremydmiller merged commit 981d187 into JasperFx:main Jun 5, 2025
1 check passed
@jeremydmiller jeremydmiller mentioned this pull request Jun 5, 2025
@vzam vzam deleted the vipri/mqtt-dispose branch June 5, 2025 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants