Real-time notifications not working when an EntityIdentifier is set

Platform: ABP 5.1.0 (ASP.NET Core MVC)

Something is wrong with NotificationPublisher when a notification is sent including an EntityIdentifier. When a notification with that parameter set is sent, SignalR disconnects and reconnects, and the notification is never displayed. To see the notification I am forced to refresh the page.

If I simply remove the EntityIdentifier parameter, that same notification works correctly with SignalR.

Sample with the error:

public async Task DataClearedAsync(Guid processStatusId, int tenantId, int userId)
{
	var notificationData = new LocalizableMessageNotificationData(
		new LocalizableString(
			"ClearDataNotificationMessage",
			InAccessConsts.LocalizationSourceName
		)
	);

	await _notificationPublisher
		.PublishAsync(AppNotificationNames.ClearData,
		notificationData,
		new Abp.Domain.Entities.EntityIdentifier(typeof(ProcessStatus), processStatusId),
		userIds: new[] { new UserIdentifier(tenantId, userId) },
		severity: NotificationSeverity.Success);
}

Sample that works correctly:

public async Task DataClearedAsync(Guid processStatusId, int tenantId, int userId)
{
	var notificationData = new LocalizableMessageNotificationData(
		new LocalizableString(
			"ClearDataNotificationMessage",
			InAccessConsts.LocalizationSourceName
		)
	);

	notificationData["processStatusId"] = processStatusId;

	await _notificationPublisher
		.PublishAsync(AppNotificationNames.ClearData,
		notificationData,
		userIds: new[] { new UserIdentifier(tenantId, userId) },
		severity: NotificationSeverity.Success);
}

I suspect this might be a JSON serialization error with the EntityIdentifier that breaks the JSON SignalR generates for the notification.

[acjh]

Reproduced and fixed; see PR #5231.

[nimeshvaghasiya]

@ismcagdas I’m using aspnetzero v11 so it’s abp v7.1, attached screenshots below, only ‘AbpNotificationSubscriptions’ contains records of specific notifications. user id 1 and 2 both are subscribed but won’t get notifications.

Any mistake in code below?