swcs/OurUmbraco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Restrict number of characters in signup fields to prevent database errors ("string would be truncated")

Browse Source
This commit is contained in:
Sebastiaan Janssen
2016-12-07 18:02:52 +01:00
parent 9707c06d1b
commit d5761dbcd1
1 changed files with 11 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
OurUmbraco/Our/Controllers/RegisterController.cs
+11 -12
View File
@@ -64,19 +64,18 @@ namespace OurUmbraco.Our.Controllers
// these values are enforced in MemberDto which is internal ;-(
// we should really have ways to query for Core meta-data!
const int maxEmailLength = 900; // 1000*.9 for safety
const int maxLoginNameLength = 900; // 1000*.9 for safety
const int maxPasswordLength = 900; // 1000*.9 for safety
const int maxPropertyLength = 900; // keep it safe too
const int maxEmailLength = 400;
const int maxLoginNameLength = 200;
const int maxPasswordLength = 400;
const int maxPropertyLength = 400;
if (model.Email.Length > maxEmailLength
|| model.Name.Length > maxLoginNameLength
|| model.Password.Length > maxPasswordLength
|| model.Location.Length > maxPropertyLength
|| model.Longitude.Length > maxPropertyLength
|| model.Latitude.Length > maxPropertyLength
|| model.Company.Length > maxPropertyLength
|| model.TwitterAlias.Length > maxPropertyLength
if (model.Email != null && model.Email.Length > maxEmailLength
|| model.Name != null && model.Name.Length > maxLoginNameLength
|| model.Password != null && model.Password.Length > maxPasswordLength
|| model.Location != null && model.Location.Length > maxPropertyLength
|| model.Longitude != null && model.Longitude.Length > maxPropertyLength
|| model.Latitude != null && model.Latitude.Length > maxPropertyLength
|| model.TwitterAlias != null && model.TwitterAlias.Length > maxPropertyLength
)
{
// has to be a rogue registration