Nemcio/GitBucket
1
0
Fork 0
mirror of https://github.com/gitbucket/gitbucket.git synced 2025-11-14 17:35:50 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1878 from alejandroliu/OIC-fix

Browse Source 
Assign dummy password to newly created users
This commit is contained in:
Naoki Takezoe
2018-02-10 00:25:38 +09:00
committed by GitHub
parent aa7db68e68 283baaed57
commit 934d1df991
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/scala/gitbucket/core/service/AccountFederationService.scala
View File

@@ -34,7 +34,7 @@ trait AccountFederationService {
None
case None =>
findAvailableUserName(preferredUserName, mailAddress) flatMap { userName =>
createAccount(userName, "", fullName.getOrElse(userName), mailAddress, isAdmin = false, None, None)
createAccount(userName, "[DUMMY]", fullName.getOrElse(userName), mailAddress, isAdmin = false, None, None)
createAccountFederation(issuer, subject, userName)
getAccountByUserName(userName)
}

2
src/test/scala/gitbucket/core/service/AccountFederationServiceSpec.scala
View File

@@ -9,7 +9,7 @@ class AccountFederationServiceSpec extends FunSpec with ServiceSpecBase {
withTestDB { implicit session =>
val actual = AccountFederationService.getOrCreateFederatedUser("someIssuer", "someSubject", "dummy@example.com", Some("foo"), Some("Foo"))
assert(actual.get.userName == "foo")
assert(actual.get.password == "")
assert(actual.get.password == "[DUMMY]")
assert(actual.get.fullName == "Foo")
assert(actual.get.mailAddress == "dummy@example.com")
assert(!actual.get.isAdmin)