What is better, coming back here where onsuccess is already defined, or passing onsuccess into complete_sign_in?