Skip to content

Commit

Permalink
fix(accounts-controller): fix potential vulnerability with raw passwo…
Browse files Browse the repository at this point in the history
…rd saving/viewing.
  • Loading branch information
Awilum committed Jun 26, 2020
1 parent f48e51d commit aaaa806
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions app/Controllers/AccountsController.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ public function index(Request $request, Response $response, array $args) : Respo
$_path = explode('/', $account['path']);
$account_to_store['email'] = array_pop($_path);

Arr::delete($account, 'password');
Arr::delete($account, 'hashed_password');
Arr::delete($account, 'hashed_password_reset');

Expand Down Expand Up @@ -521,6 +522,7 @@ public function profile(Request $request, Response $response, array $args) : Res
$profile['email'] = $email;

Arr::delete($profile, 'uuid');
Arr::delete($profile, 'password');
Arr::delete($profile, 'hashed_password');
Arr::delete($profile, 'hashed_password_reset');
Arr::delete($profile, 'roles');
Expand Down

0 comments on commit aaaa806

Please sign in to comment.