artin/OBP-API
1
0
Fork 0
mirror of https://github.com/OpenBankProject/OBP-API.git synced 2026-02-06 15:27:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

feature/Add username constraints

Browse Source
This commit is contained in:
Marko Milić 2020-09-24 10:21:32 +02:00
parent b8d7237b8c
commit 56edf8b268
3 changed files with 37 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
obp-api/src/main/resources/i18n/lift-core.properties
View File

@ -366,4 +366,10 @@ Deleted = Deleted
#OBP specific fields
consumer.registration.nav.name=Get API Key
invalid.login.credentials=Invalid Login Credentials
invalid.login.credentials=Invalid Login Credentials
invalid.username=Invalid username. \
1) username is 8-100 characters long \
2) no _ or . at the beginning \
3) no __ or _. or ._ or .. inside \
4) allowed characters: a-z A-Z 0-9 . _ \
5) no _ or . at the end

27
obp-api/src/main/scala/code/model/dataAccess/AuthUser.scala
View File

@ -112,6 +112,24 @@ class AuthUser extends MegaProtoUser[AuthUser] with MdcLoggable {
override def validations = isEmpty(Helper.i18n("Please.enter.your.last.name")) _ :: super.validations
}
/**
* Regex to validate a username
*
* ^(?=.{8,100}$)(?![_.])(?!.*[_.]{2})[a-zA-Z0-9._]+(?<![_.])$
* └─────┬────┘└───┬──┘└─────┬─────┘└─────┬─────┘ └───┬───┘
* │ │ │ │ no _ or . at the end
* │ │ │ │
* │ │ │ allowed characters
* │ │ │
* │ │ no __ or _. or ._ or .. inside
* │ │
* │ no _ or . at the beginning
* │
* username is 8-100 characters long
*
*/
private val usernameRegex = """^(?=.{8,100}$)(?![_.])(?!.*[_.]{2})[a-zA-Z0-9._]+(?<![_.])$""".r
/**
* The username field for the User.
*/
@ -123,9 +141,16 @@ class AuthUser extends MegaProtoUser[AuthUser] with MdcLoggable {
case e if e.trim.isEmpty => List(FieldError(this, Text(msg))) // issue 179
case _ => Nil
}
def isUsernamelValid(msg: => String)(e: String) = e match {
case null => List(FieldError(this, Text(msg)))
case e if e.trim.isEmpty => List(FieldError(this, Text(msg)))
case e if usernameRegex.findFirstMatchIn(e).isDefined => Nil
case _ => List(FieldError(this, Text(msg)))
}
override def displayName = S.?("Username")
override def dbIndexed_? = true
override def validations = isEmpty(Helper.i18n("Please.enter.your.username")) _ ::
override def validations = isEmpty(Helper.i18n("Please.enter.your.username")) _ ::
isUsernamelValid(Helper.i18n("invalid.username")) _ ::
valUnique(Helper.i18n("unique.username")) _ ::
valUniqueExternally(Helper.i18n("unique.username")) _ ::
super.validations

8
obp-api/src/test/scala/code/sandbox/SandboxDataLoadingTest.scala
View File

@ -454,8 +454,8 @@ class SandboxDataLoadingTest extends FlatSpec with SendServerRequests with Match
val standardProducts = product1AtBank1 :: product2AtBank1 :: Nil
val user1 = SandboxUserImport(email = "user1@example.com", password = "TESOBE520berlin123!", user_name = "User 1")
val user2 = SandboxUserImport(email = "user2@example.com", password = "TESOBE520berlin123!", user_name = "User 2")
val user1 = SandboxUserImport(email = "user1@example.com", password = "TESOBE520berlin123!", user_name = "user.name_1")
val user2 = SandboxUserImport(email = "user2@example.com", password = "TESOBE520berlin123!", user_name = "user.name_2")
val standardUsers = user1 :: user2 :: Nil
@ -778,11 +778,11 @@ class SandboxDataLoadingTest extends FlatSpec with SendServerRequests with Match
}
//emails of the user we will eventually create to show multiple users with different ids are possible
val secondUserName = "user-two"
val secondUserName = "user_two"
val user1Json = Extraction.decompose(user1)
val differentUsername = "user-one"
val differentUsername = "user_one"
differentUsername should not equal(user1.user_name)
val userWithSameUsernameAsUser1 = user1Json