I would think this either needs to set the current user or require the user to be returned and set as the current user. I realize it may work in this case, but if I look forward to using this in other locations that aspect could be easily lost and lead to confusion.