fix oauth collection bug

[emrgnt-cmplxty]

Important

Fixes OAuth user registration bug by ensuring consistent user creation logic in r2r_auth.py.

• Behavior:
  • Fixes bug in register() in r2r_auth.py by ensuring new_user is fully populated after creation.
  • Refactors oauth_callback_handler() in r2r_auth.py to use register() for user creation, ensuring consistent user creation logic.
• Misc:
  • Removes redundant user creation code in oauth_callback_handler() for Google and GitHub providers.

^{This description was created by for 4eb13f2. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 4eb13f2 in 13 seconds

More details

• Looked at 57 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. py/core/providers/auth/r2r_auth.py:611

• Draft comment:
  Using register instead of create_user ensures consistent user creation logic and handles email verification if required. This change is beneficial for maintaining uniformity in user registration processes.
• Reason this comment was not posted:
  Confidence changes required: 20%
  The PR changes the way new users are created during OAuth callback handling by using the register method instead of create_user. This ensures consistent user creation logic and handles email verification if required.

Workflow ID: wflow_wd1ciZai1FKYghX3

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.