secret property now named uri to match otphp wording

This commit is contained in:
Bubka 2019-05-26 23:24:22 +02:00
parent bb4fbfd46e
commit e7695b64bc
5 changed files with 18 additions and 18 deletions

View File

@ -31,7 +31,7 @@ public function store(Request $request)
{ {
$twofaccount = TwoFAccount::create([ $twofaccount = TwoFAccount::create([
'name' => $request->name, 'name' => $request->name,
'secret' => $request->secret 'uri' => $request->uri
]); ]);
return response()->json($twofaccount, 201); return response()->json($twofaccount, 201);
@ -59,7 +59,7 @@ public function show(TwoFAccount $twofaccount)
public function generateTOTP(TwoFAccount $twofaccount) public function generateTOTP(TwoFAccount $twofaccount)
{ {
try { try {
$otp = Factory::loadFromProvisioningUri($twofaccount->secret); $otp = Factory::loadFromProvisioningUri($twofaccount->uri);
} catch (InvalidArgumentException $exception) { } catch (InvalidArgumentException $exception) {
return response()->json([ return response()->json([
'message' => 'Error generating TOTP', 'message' => 'Error generating TOTP',

View File

@ -9,7 +9,7 @@ class TwoFAccount extends Model
{ {
use SoftDeletes; use SoftDeletes;
protected $fillable = ['name', 'secret', 'icon']; protected $fillable = ['name', 'uri', 'icon'];
/** /**

View File

@ -16,7 +16,7 @@ public function up()
Schema::create('twofaccounts', function (Blueprint $table) { Schema::create('twofaccounts', function (Blueprint $table) {
$table->increments('id'); $table->increments('id');
$table->string('name')->unique(); $table->string('name')->unique();
$table->string('secret'); $table->string('uri');
$table->string('icon')->nullable(); $table->string('icon')->nullable();
$table->timestamps(); $table->timestamps();
$table->softDeletes(); $table->softDeletes();

View File

@ -17,26 +17,26 @@ public function run()
TwoFAccount::create([ TwoFAccount::create([
'name' => $faker->unique()->domainName, 'name' => $faker->unique()->domainName,
'secret' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test', 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test',
]); ]);
$deletedResource = TwoFAccount::create([ $deletedResource = TwoFAccount::create([
'name' => $faker->unique()->domainName, 'name' => $faker->unique()->domainName,
'secret' => $faker->password, 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test',
]); ]);
$deletedResource->delete(); $deletedResource->delete();
TwoFAccount::create([ TwoFAccount::create([
'name' => $faker->unique()->domainName, 'name' => $faker->unique()->domainName,
'secret' => $faker->password, 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test',
]); ]);
TwoFAccount::create([ TwoFAccount::create([
'name' => $faker->unique()->domainName, 'name' => $faker->unique()->domainName,
'secret' => $faker->password, 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test',
]); ]);
TwoFAccount::create([ TwoFAccount::create([
'name' => $faker->unique()->domainName, 'name' => $faker->unique()->domainName,
'secret' => $faker->password, 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test',
]); ]);
} }
} }

View File

@ -41,12 +41,12 @@ public function testTwoFAccountCreation()
$response = $this->actingAs($user, 'api') $response = $this->actingAs($user, 'api')
->json('POST', '/api/twofaccounts', [ ->json('POST', '/api/twofaccounts', [
'name' => 'testCreation', 'name' => 'testCreation',
'secret' => 'test', 'uri' => 'test',
]) ])
->assertStatus(201) ->assertStatus(201)
->assertJson([ ->assertJson([
'name' => 'testCreation', 'name' => 'testCreation',
'secret' => 'test', 'uri' => 'test',
]); ]);
} }
@ -62,11 +62,11 @@ public function testTOTPgeneration()
$twofaccount = TwoFAccount::create([ $twofaccount = TwoFAccount::create([
'name' => 'testTOTP', 'name' => 'testTOTP',
'secret' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test' 'uri' => 'otpauth://totp/test@test.com?secret=A4GRFHVVRBGY7UIW&issuer=test'
]); ]);
$response = $this->actingAs($user, 'api') $response = $this->actingAs($user, 'api')
->json('POST', '/api/twofaccounts/' . $twofaccount->id . '/totp') ->json('GET', '/api/twofaccounts/' . $twofaccount->id . '/totp')
->assertStatus(200) ->assertStatus(200)
->assertJsonStructure([ ->assertJsonStructure([
'totp', 'totp',
@ -86,13 +86,13 @@ public function testTwoFAccountUpdate()
$response = $this->actingAs($user, 'api') $response = $this->actingAs($user, 'api')
->json('PUT', '/api/twofaccounts/1', [ ->json('PUT', '/api/twofaccounts/1', [
'name' => 'testUpdate', 'name' => 'testUpdate',
'secret' => 'testUpdate', 'uri' => 'testUpdate',
]) ])
->assertStatus(200) ->assertStatus(200)
->assertJson([ ->assertJson([
'id' => 1, 'id' => 1,
'name' => 'testUpdate', 'name' => 'testUpdate',
'secret' => 'testUpdate', 'uri' => 'testUpdate',
'icon' => null, 'icon' => null,
]); ]);
} }
@ -114,7 +114,7 @@ public function testTwoFAccountIndexListing()
'*' => [ '*' => [
'id', 'id',
'name', 'name',
'secret', 'uri',
'icon', 'icon',
'created_at', 'created_at',
'updated_at', 'updated_at',
@ -135,7 +135,7 @@ public function testTwoFAccountDeletion()
$twofaccount = TwoFAccount::create([ $twofaccount = TwoFAccount::create([
'name' => 'testDelete', 'name' => 'testDelete',
'secret' => 'test' 'uri' => 'test'
]); ]);
$response = $this->actingAs($user, 'api') $response = $this->actingAs($user, 'api')
@ -154,7 +154,7 @@ public function testTwoFAccountPermanentDeletion()
$twofaccount = TwoFAccount::create([ $twofaccount = TwoFAccount::create([
'name' => 'testHardDelete', 'name' => 'testHardDelete',
'secret' => 'test' 'uri' => 'test'
]); ]);
$twofaccount->delete(); $twofaccount->delete();