Browse Source

Set ConcurrencyStamp property max length to constant (40)

Resolve #4361
pull/4479/head
maliming 6 years ago
parent
commit
69aa9b2f03
  1. 7
      framework/src/Volo.Abp.Ddd.Domain/Volo/Abp/Domain/Entities/ConcurrencyStampConsts.cs
  2. 2
      framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/Modeling/AbpEntityTypeBuilderExtensions.cs
  3. 1
      modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityClaimTypeConsts.cs
  4. 1
      modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityRoleConsts.cs
  5. 2
      modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityUserConsts.cs
  6. 6
      modules/identity/src/Volo.Abp.Identity.EntityFrameworkCore/Volo/Abp/Identity/EntityFrameworkCore/IdentityDbContextModelBuilderExtensions.cs

7
framework/src/Volo.Abp.Ddd.Domain/Volo/Abp/Domain/Entities/ConcurrencyStampConsts.cs

@ -0,0 +1,7 @@
namespace Volo.Abp.Domain.Entities
{
public static class ConcurrencyStampConsts
{
public const int MaxLength = 40;
}
}

2
framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/EntityFrameworkCore/Modeling/AbpEntityTypeBuilderExtensions.cs

@ -40,9 +40,9 @@ namespace Volo.Abp.EntityFrameworkCore.Modeling
{
if (b.Metadata.ClrType.IsAssignableTo<IHasConcurrencyStamp>())
{
//TODO: Max length?
b.Property(nameof(IHasConcurrencyStamp.ConcurrencyStamp))
.IsConcurrencyToken()
.HasMaxLength(ConcurrencyStampConsts.MaxLength)
.HasColumnName(nameof(IHasConcurrencyStamp.ConcurrencyStamp));
}
}

1
modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityClaimTypeConsts.cs

@ -6,6 +6,5 @@
public const int MaxRegexLength = 512;
public const int MaxRegexDescriptionLength = 128;
public const int MaxDescriptionLength = 256;
public const int MaxConcurrencyStampLength = 256;
}
}

1
modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityRoleConsts.cs

@ -4,6 +4,5 @@
{
public const int MaxNameLength = 256;
public const int MaxNormalizedNameLength = MaxNameLength;
public const int MaxConcurrencyStampLength = 256;
}
}

2
modules/identity/src/Volo.Abp.Identity.Domain.Shared/Volo/Abp/Identity/IdentityUserConsts.cs

@ -23,7 +23,5 @@ namespace Volo.Abp.Identity
public const int MaxPasswordHashLength = 256;
public const int MaxSecurityStampLength = 256;
public const int MaxConcurrencyStampLength = 256;
}
}

6
modules/identity/src/Volo.Abp.Identity.EntityFrameworkCore/Volo/Abp/Identity/EntityFrameworkCore/IdentityDbContextModelBuilderExtensions.cs

@ -126,9 +126,6 @@ namespace Volo.Abp.Identity.EntityFrameworkCore
b.Property(r => r.Name).IsRequired().HasMaxLength(IdentityRoleConsts.MaxNameLength);
b.Property(r => r.NormalizedName).IsRequired().HasMaxLength(IdentityRoleConsts.MaxNormalizedNameLength);
b.Property(u => u.ConcurrencyStamp).IsRequired().IsConcurrencyToken()
.HasMaxLength(IdentityRoleConsts.MaxConcurrencyStampLength)
.HasColumnName(nameof(IdentityRole.ConcurrencyStamp));
b.Property(r => r.IsDefault).HasColumnName(nameof(IdentityRole.IsDefault));
b.Property(r => r.IsStatic).HasColumnName(nameof(IdentityRole.IsStatic));
b.Property(r => r.IsPublic).HasColumnName(nameof(IdentityRole.IsPublic));
@ -163,9 +160,6 @@ namespace Volo.Abp.Identity.EntityFrameworkCore
b.Property(uc => uc.Regex).HasMaxLength(IdentityClaimTypeConsts.MaxRegexLength);
b.Property(uc => uc.RegexDescription).HasMaxLength(IdentityClaimTypeConsts.MaxRegexDescriptionLength);
b.Property(uc => uc.Description).HasMaxLength(IdentityClaimTypeConsts.MaxDescriptionLength);
b.Property(uc => uc.ConcurrencyStamp).IsRequired().IsConcurrencyToken()
.HasMaxLength(IdentityClaimTypeConsts.MaxConcurrencyStampLength)
.HasColumnName(nameof(IdentityClaimType.ConcurrencyStamp));
});
builder.Entity<OrganizationUnit>(b =>

Loading…
Cancel
Save