From b7185c1be5dfc04a67be0346fd17a1fe44d812ca Mon Sep 17 00:00:00 2001 From: Alper Ebicoglu Date: Mon, 8 Jan 2018 17:27:01 +0300 Subject: [PATCH] Added max lengths for Identity server entities --- .../ApiResources/ApiResourceClaimConsts.cs | 7 ++ .../ApiResources/ApiResourceConsts.cs | 9 ++ .../ApiResources/ApiScopeClaimConsts.cs | 7 ++ .../ApiResources/ApiScopeConsts.cs | 9 ++ .../ApiResources/ApiSecretConsts.cs | 9 ++ .../Clients/ClientClaimConsts.cs | 8 ++ .../IdentityServer/Clients/ClientConsts.cs | 25 ++++++ .../Clients/ClientCorsOriginConsts.cs | 7 ++ .../Clients/ClientGrantTypeConsts.cs | 7 ++ .../Clients/ClientIdPRestrictionConsts.cs | 7 ++ .../ClientPostLogoutRedirectUriConsts.cs | 7 ++ .../Clients/ClientPropertyConsts.cs | 8 ++ .../Clients/ClientRedirectUriConsts.cs | 7 ++ .../Clients/ClientScopeConsts.cs | 7 ++ .../Clients/ClientSecretConsts.cs | 9 ++ .../Grants/PersistedGrantConsts.cs | 10 +++ .../IdentityResources/IdentityClaimConsts.cs | 7 ++ .../IdentityResourceConsts.cs | 9 ++ .../IdentityServerDbContext.cs | 84 +++++++++---------- 19 files changed, 201 insertions(+), 42 deletions(-) create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceClaimConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeClaimConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiSecretConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientClaimConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientCorsOriginConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientGrantTypeConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientIdPRestrictionConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPostLogoutRedirectUriConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPropertyConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientRedirectUriConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientScopeConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientSecretConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Grants/PersistedGrantConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityClaimConsts.cs create mode 100644 src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityResourceConsts.cs diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceClaimConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceClaimConsts.cs new file mode 100644 index 0000000000..a87d3566fc --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceClaimConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.ApiResources +{ + public class ApiResourceClaimConsts + { + public const int TypeMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceConsts.cs new file mode 100644 index 0000000000..1c3aecc531 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiResourceConsts.cs @@ -0,0 +1,9 @@ +namespace Volo.Abp.IdentityServer.ApiResources +{ + public class ApiResourceConsts + { + public const int NameMaxLength = 200; + public const int DisplayNameMaxLength = 200; + public const int DescriptionMaxLength = 1000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeClaimConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeClaimConsts.cs new file mode 100644 index 0000000000..5e06a29402 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeClaimConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.ApiResources +{ + public class ApiScopeClaimConsts + { + public const int TypeMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeConsts.cs new file mode 100644 index 0000000000..d52b953921 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiScopeConsts.cs @@ -0,0 +1,9 @@ +namespace Volo.Abp.IdentityServer.ApiResources +{ + public class ApiScopeConsts + { + public const int NameMaxLength = 200; + public const int DisplayNameMaxLength = 200; + public const int DescriptionMaxLength = 1000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiSecretConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiSecretConsts.cs new file mode 100644 index 0000000000..5008a34afb --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/ApiResources/ApiSecretConsts.cs @@ -0,0 +1,9 @@ +namespace Volo.Abp.IdentityServer.ApiResources +{ + public class ApiSecretConsts + { + public const int DescriptionMaxLength = 1000; + public const int ValueMaxLength = 2000; + public const int TypeMaxLength = 250; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientClaimConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientClaimConsts.cs new file mode 100644 index 0000000000..3a5d1fe907 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientClaimConsts.cs @@ -0,0 +1,8 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientClaimConsts + { + public const int TypeMaxLength = 250; + public const int ValueMaxLength = 250; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientConsts.cs new file mode 100644 index 0000000000..d83138fd2f --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientConsts.cs @@ -0,0 +1,25 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientConsts + { + public const int ClientIdMaxLength = 200; + + public const int ProtocolTypeMaxLength = 200; + + public const int ClientNameMaxLength = 200; + + public const int ClientUriMaxLength = 2000; + + public const int LogoUriMaxLength = 2000; + + public const int DescriptionMaxLength = 1000; + + public const int FrontChannelLogoutUriMaxLength = 2000; + + public const int BackChannelLogoutUriMaxLength = 2000; + + public const int ClientClaimsPrefixMaxLength = 200; + + public const int PairWiseSubjectSaltMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientCorsOriginConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientCorsOriginConsts.cs new file mode 100644 index 0000000000..07ba598d6c --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientCorsOriginConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientCorsOriginConsts + { + public const int OriginMaxLength = 150; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientGrantTypeConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientGrantTypeConsts.cs new file mode 100644 index 0000000000..5f94c51f97 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientGrantTypeConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientGrantTypeConsts + { + public const int GrantTypeMaxLength = 250; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientIdPRestrictionConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientIdPRestrictionConsts.cs new file mode 100644 index 0000000000..c71301871a --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientIdPRestrictionConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientIdPRestrictionConsts + { + public const int ProviderMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPostLogoutRedirectUriConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPostLogoutRedirectUriConsts.cs new file mode 100644 index 0000000000..8fe55ba018 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPostLogoutRedirectUriConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientPostLogoutRedirectUriConsts + { + public const int PostLogoutRedirectUriMaxLength = 2000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPropertyConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPropertyConsts.cs new file mode 100644 index 0000000000..6535304596 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientPropertyConsts.cs @@ -0,0 +1,8 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientPropertyConsts + { + public const int KeyMaxLength = 250; + public const int ValueMaxLength = 2000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientRedirectUriConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientRedirectUriConsts.cs new file mode 100644 index 0000000000..d67bbea509 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientRedirectUriConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientRedirectUriConsts + { + public const int RedirectUriMaxLength = 2000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientScopeConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientScopeConsts.cs new file mode 100644 index 0000000000..6a5b62e692 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientScopeConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientScopeConsts + { + public const int ScopeMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientSecretConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientSecretConsts.cs new file mode 100644 index 0000000000..f3e47a5973 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Clients/ClientSecretConsts.cs @@ -0,0 +1,9 @@ +namespace Volo.Abp.IdentityServer.Clients +{ + public class ClientSecretConsts + { + public const int ValueMaxLength = 2000; + public const int TypeMaxLength = 250; + public const int DescriptionMaxLength = 2000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Grants/PersistedGrantConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Grants/PersistedGrantConsts.cs new file mode 100644 index 0000000000..75f67dcc3a --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/Grants/PersistedGrantConsts.cs @@ -0,0 +1,10 @@ +namespace Volo.Abp.IdentityServer.Grants +{ + public class PersistedGrantConsts + { + public const int KeyMaxLength = 200; + public const int TypeMaxLength = 50; + public const int SubjectIdMaxLength = 200; + public const int ClientIdMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityClaimConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityClaimConsts.cs new file mode 100644 index 0000000000..5a0ee5e177 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityClaimConsts.cs @@ -0,0 +1,7 @@ +namespace Volo.Abp.IdentityServer.IdentityResources +{ + public class IdentityClaimConsts + { + public const int TypeMaxLength = 200; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityResourceConsts.cs b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityResourceConsts.cs new file mode 100644 index 0000000000..c60cc34ad8 --- /dev/null +++ b/src/Volo.Abp.IdentityServer.Domain/Volo/Abp/IdentityServer/IdentityResources/IdentityResourceConsts.cs @@ -0,0 +1,9 @@ +namespace Volo.Abp.IdentityServer.IdentityResources +{ + public class IdentityResourceConsts + { + public const int NameMaxLength = 200; + public const int DisplayNameMaxLength = 200; + public const int DescriptionMaxLength = 1000; + } +} \ No newline at end of file diff --git a/src/Volo.Abp.IdentityServer.EntityFrameworkCore/Volo/Abp/IdentityServer/EntityFrameworkCore/IdentityServerDbContext.cs b/src/Volo.Abp.IdentityServer.EntityFrameworkCore/Volo/Abp/IdentityServer/EntityFrameworkCore/IdentityServerDbContext.cs index 84c7311b84..d56db360f0 100644 --- a/src/Volo.Abp.IdentityServer.EntityFrameworkCore/Volo/Abp/IdentityServer/EntityFrameworkCore/IdentityServerDbContext.cs +++ b/src/Volo.Abp.IdentityServer.EntityFrameworkCore/Volo/Abp/IdentityServer/EntityFrameworkCore/IdentityServerDbContext.cs @@ -63,16 +63,16 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore client.HasKey(x => x.Id); - client.Property(x => x.ClientId).HasMaxLength(200).IsRequired(); - client.Property(x => x.ProtocolType).HasMaxLength(200).IsRequired(); - client.Property(x => x.ClientName).HasMaxLength(200); - client.Property(x => x.ClientUri).HasMaxLength(2000); - client.Property(x => x.LogoUri).HasMaxLength(2000); - client.Property(x => x.Description).HasMaxLength(1000); - client.Property(x => x.FrontChannelLogoutUri).HasMaxLength(2000); - client.Property(x => x.BackChannelLogoutUri).HasMaxLength(2000); - client.Property(x => x.ClientClaimsPrefix).HasMaxLength(200); - client.Property(x => x.PairWiseSubjectSalt).HasMaxLength(200); + client.Property(x => x.ClientId).HasMaxLength(ClientConsts.ClientIdMaxLength).IsRequired(); + client.Property(x => x.ProtocolType).HasMaxLength(ClientConsts.ProtocolTypeMaxLength).IsRequired(); + client.Property(x => x.ClientName).HasMaxLength(ClientConsts.ClientNameMaxLength); + client.Property(x => x.ClientUri).HasMaxLength(ClientConsts.ClientUriMaxLength); + client.Property(x => x.LogoUri).HasMaxLength(ClientConsts.LogoUriMaxLength); + client.Property(x => x.Description).HasMaxLength(ClientConsts.DescriptionMaxLength); + client.Property(x => x.FrontChannelLogoutUri).HasMaxLength(ClientConsts.FrontChannelLogoutUriMaxLength); + client.Property(x => x.BackChannelLogoutUri).HasMaxLength(ClientConsts.BackChannelLogoutUriMaxLength); + client.Property(x => x.ClientClaimsPrefix).HasMaxLength(ClientConsts.ClientClaimsPrefixMaxLength); + client.Property(x => x.PairWiseSubjectSalt).HasMaxLength(ClientConsts.PairWiseSubjectSaltMaxLength); client.HasMany(x => x.AllowedGrantTypes).WithOne().HasForeignKey(x => x.ClientId).IsRequired(); client.HasMany(x => x.RedirectUris).WithOne().HasForeignKey(x => x.ClientId).IsRequired(); @@ -90,7 +90,7 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore builder.Entity(grantType => { grantType.ToTable(TablePrefix + "ClientGrantTypes"); - grantType.Property(x => x.GrantType).HasMaxLength(250).IsRequired(); + grantType.Property(x => x.GrantType).HasMaxLength(ClientGrantTypeConsts.GrantTypeMaxLength).IsRequired(); grantType.Property(x => x.ClientId).IsRequired(); }); @@ -98,63 +98,63 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore builder.Entity(redirectUri => { redirectUri.ToTable(TablePrefix + "ClientRedirectUris"); - redirectUri.Property(x => x.RedirectUri).HasMaxLength(2000).IsRequired(); + redirectUri.Property(x => x.RedirectUri).HasMaxLength(ClientRedirectUriConsts.RedirectUriMaxLength).IsRequired(); }); builder.Entity(postLogoutRedirectUri => { postLogoutRedirectUri.ToTable(TablePrefix + "ClientPostLogoutRedirectUris"); - postLogoutRedirectUri.Property(x => x.PostLogoutRedirectUri).HasMaxLength(2000).IsRequired(); + postLogoutRedirectUri.Property(x => x.PostLogoutRedirectUri).HasMaxLength(ClientPostLogoutRedirectUriConsts.PostLogoutRedirectUriMaxLength).IsRequired(); }); builder.Entity(scope => { scope.ToTable(TablePrefix + "ClientScopes"); - scope.Property(x => x.Scope).HasMaxLength(200).IsRequired(); + scope.Property(x => x.Scope).HasMaxLength(ClientScopeConsts.ScopeMaxLength).IsRequired(); }); builder.Entity(secret => { secret.ToTable(TablePrefix + "ClientSecrets"); - secret.Property(x => x.Value).HasMaxLength(2000).IsRequired(); - secret.Property(x => x.Type).HasMaxLength(250); - secret.Property(x => x.Description).HasMaxLength(2000); + secret.Property(x => x.Value).HasMaxLength(ClientSecretConsts.ValueMaxLength).IsRequired(); + secret.Property(x => x.Type).HasMaxLength(ClientSecretConsts.TypeMaxLength); + secret.Property(x => x.Description).HasMaxLength(ClientSecretConsts.DescriptionMaxLength); }); builder.Entity(claim => { claim.ToTable(TablePrefix + "ClientClaims"); - claim.Property(x => x.Type).HasMaxLength(250).IsRequired(); - claim.Property(x => x.Value).HasMaxLength(250).IsRequired(); + claim.Property(x => x.Type).HasMaxLength(ClientClaimConsts.TypeMaxLength).IsRequired(); + claim.Property(x => x.Value).HasMaxLength(ClientClaimConsts.ValueMaxLength).IsRequired(); }); builder.Entity(idPRestriction => { idPRestriction.ToTable(TablePrefix + "ClientIdPRestrictions"); - idPRestriction.Property(x => x.Provider).HasMaxLength(200).IsRequired(); + idPRestriction.Property(x => x.Provider).HasMaxLength(ClientIdPRestrictionConsts.ProviderMaxLength).IsRequired(); }); builder.Entity(corsOrigin => { corsOrigin.ToTable(TablePrefix + "ClientCorsOrigins"); - corsOrigin.Property(x => x.Origin).HasMaxLength(150).IsRequired(); + corsOrigin.Property(x => x.Origin).HasMaxLength(ClientCorsOriginConsts.OriginMaxLength).IsRequired(); }); builder.Entity(property => { property.ToTable(TablePrefix + "ClientProperties"); - property.Property(x => x.Key).HasMaxLength(250).IsRequired(); - property.Property(x => x.Value).HasMaxLength(2000).IsRequired(); + property.Property(x => x.Key).HasMaxLength(ClientPropertyConsts.KeyMaxLength).IsRequired(); + property.Property(x => x.Value).HasMaxLength(ClientPropertyConsts.ValueMaxLength).IsRequired(); }); builder.Entity(grant => { grant.ToTable(TablePrefix + "PersistedGrants"); - grant.Property(x => x.Key).HasMaxLength(200).ValueGeneratedNever(); - grant.Property(x => x.Type).HasMaxLength(50).IsRequired(); - grant.Property(x => x.SubjectId).HasMaxLength(200); - grant.Property(x => x.ClientId).HasMaxLength(200).IsRequired(); + grant.Property(x => x.Key).HasMaxLength(PersistedGrantConsts.KeyMaxLength).ValueGeneratedNever(); + grant.Property(x => x.Type).HasMaxLength(PersistedGrantConsts.TypeMaxLength).IsRequired(); + grant.Property(x => x.SubjectId).HasMaxLength(PersistedGrantConsts.SubjectIdMaxLength); + grant.Property(x => x.ClientId).HasMaxLength(PersistedGrantConsts.ClientIdMaxLength).IsRequired(); grant.Property(x => x.CreationTime).IsRequired(); grant.Property(x => x.Data).IsRequired(); @@ -167,9 +167,9 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore { identityResource.ToTable(TablePrefix + "IdentityResources").HasKey(x => x.Id); - identityResource.Property(x => x.Name).HasMaxLength(200).IsRequired(); - identityResource.Property(x => x.DisplayName).HasMaxLength(200); - identityResource.Property(x => x.Description).HasMaxLength(1000); + identityResource.Property(x => x.Name).HasMaxLength(IdentityResourceConsts.NameMaxLength).IsRequired(); + identityResource.Property(x => x.DisplayName).HasMaxLength(IdentityResourceConsts.DisplayNameMaxLength); + identityResource.Property(x => x.Description).HasMaxLength(IdentityResourceConsts.DescriptionMaxLength); identityResource.HasIndex(x => x.Name).IsUnique(); @@ -181,16 +181,16 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore { claim.ToTable(TablePrefix + "IdentityClaims").HasKey(x => x.Id); - claim.Property(x => x.Type).HasMaxLength(200).IsRequired(); + claim.Property(x => x.Type).HasMaxLength(IdentityClaimConsts.TypeMaxLength).IsRequired(); }); builder.Entity(apiResource => { apiResource.ToTable(TablePrefix + "ApiResources").HasKey(x => x.Id); - apiResource.Property(x => x.Name).HasMaxLength(200).IsRequired(); - apiResource.Property(x => x.DisplayName).HasMaxLength(200); - apiResource.Property(x => x.Description).HasMaxLength(1000); + apiResource.Property(x => x.Name).HasMaxLength(ApiResourceConsts.NameMaxLength).IsRequired(); + apiResource.Property(x => x.DisplayName).HasMaxLength(ApiResourceConsts.DisplayNameMaxLength); + apiResource.Property(x => x.Description).HasMaxLength(ApiResourceConsts.DescriptionMaxLength); apiResource.HasIndex(x => x.Name).IsUnique(); @@ -203,25 +203,25 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore { apiSecret.ToTable(TablePrefix + "ApiSecrets").HasKey(x => x.Id); - apiSecret.Property(x => x.Description).HasMaxLength(1000); - apiSecret.Property(x => x.Value).HasMaxLength(2000); - apiSecret.Property(x => x.Type).HasMaxLength(250); + apiSecret.Property(x => x.Description).HasMaxLength(ApiSecretConsts.DescriptionMaxLength); + apiSecret.Property(x => x.Value).HasMaxLength(ApiSecretConsts.ValueMaxLength); + apiSecret.Property(x => x.Type).HasMaxLength(ApiSecretConsts.TypeMaxLength); }); builder.Entity(apiClaim => { apiClaim.ToTable(TablePrefix + "ApiClaims").HasKey(x => x.Id); - apiClaim.Property(x => x.Type).HasMaxLength(200).IsRequired(); + apiClaim.Property(x => x.Type).HasMaxLength(ApiResourceClaimConsts.TypeMaxLength).IsRequired(); }); builder.Entity(apiScope => { apiScope.ToTable(TablePrefix + "ApiScopes").HasKey(x => x.Id); - apiScope.Property(x => x.Name).HasMaxLength(200).IsRequired(); - apiScope.Property(x => x.DisplayName).HasMaxLength(200); - apiScope.Property(x => x.Description).HasMaxLength(1000); + apiScope.Property(x => x.Name).HasMaxLength(ApiScopeConsts.NameMaxLength).IsRequired(); + apiScope.Property(x => x.DisplayName).HasMaxLength(ApiScopeConsts.DisplayNameMaxLength); + apiScope.Property(x => x.Description).HasMaxLength(ApiScopeConsts.DescriptionMaxLength); apiScope.HasIndex(x => x.Name).IsUnique(); @@ -232,7 +232,7 @@ namespace Volo.Abp.IdentityServer.EntityFrameworkCore { apiScopeClaim.ToTable(TablePrefix + "ApiScopeClaims").HasKey(x => x.Id); - apiScopeClaim.Property(x => x.Type).HasMaxLength(200).IsRequired(); + apiScopeClaim.Property(x => x.Type).HasMaxLength(ApiScopeClaimConsts.TypeMaxLength).IsRequired(); }); } }