Skip to content

Commit aa96588

Browse files
committed
Remove superfluous call to #create_new_auth_token.
* There was prior discussion around removing this line of code, inside lynndylanhurley#990. See: lynndylanhurley#990 (comment) * While line in question _is_ superfluous, removing it was blocked by a bad test. This test was corrected in the previous commit (9ebc5bd).
1 parent 9ebc5bd commit aa96588

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

app/controllers/devise_token_auth/concerns/set_user_by_token.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,9 @@ def set_user_by_token(mapping=nil)
6060
if devise_warden_user && devise_warden_user.tokens[@client_id].nil?
6161
@used_auth_by_token = false
6262
@resource = devise_warden_user
63-
@resource.create_new_auth_token
63+
# REVIEW: The following line _should_ be safe to remove;
64+
# the generated token does not get used anywhere.
65+
# @resource.create_new_auth_token
6466
end
6567
end
6668

0 commit comments

Comments
 (0)