Merge pull request #6544 from aspnetboilerplate/fix/6482

Add max length to TargetNotifiers
aspnetboilerplate · Sep 20, 2022 · 0665b08 · 0665b08
2 parents 2265c53 + 6af109b
commit 0665b08
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/Abp/Notifications/NotificationInfo.cs b/src/Abp/Notifications/NotificationInfo.cs
@@ -72,6 +72,8 @@ public class NotificationInfo : CreationAuditedEntity<Guid>
         /// </summary>
         public const int MaxTenantIdsLength = 128 * 1024;
 
+        public const int MaxTargetNotifiersLength = 1024;
+
         /// <summary>
         /// Notification target list separation character.
         /// </summary>
@@ -149,6 +151,7 @@ public class NotificationInfo : CreationAuditedEntity<Guid>
         /// <summary>
         /// which realtime notifiers should handle this notification
         /// </summary>
+        [StringLength(MaxTargetNotifiersLength)]
         public virtual string TargetNotifiers { get; set; }
 
         [NotMapped]

diff --git a/src/Abp/Notifications/UserNotificationInfo.cs b/src/Abp/Notifications/UserNotificationInfo.cs
@@ -17,6 +17,8 @@ namespace Abp.Notifications
     [Table("AbpUserNotifications")]
     public class UserNotificationInfo : Entity<Guid>, IHasCreationTime, IMayHaveTenant
     {
+        public const int MaxTargetNotifiersLength = 1024;
+
         /// <summary>
         /// Tenant Id.
         /// </summary>
@@ -43,6 +45,7 @@ public class UserNotificationInfo : Entity<Guid>, IHasCreationTime, IMayHaveTena
         /// <summary>
         /// which realtime notifiers should handle this notification
         /// </summary>
+        [StringLength(MaxTargetNotifiersLength)]
         public virtual string TargetNotifiers { get; set; }
 
         [NotMapped]