Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Handle Soft-Deleted Users in OAuth Login to Prevent Duplicate Entry Errors #173

Open
kanglalaw opened this issue Sep 27, 2024 · 0 comments · May be fixed by #175
Open

Bug: Handle Soft-Deleted Users in OAuth Login to Prevent Duplicate Entry Errors #173

kanglalaw opened this issue Sep 27, 2024 · 0 comments · May be fixed by #175
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@kanglalaw
Copy link

kanglalaw commented Sep 27, 2024

PHP Version

8.2.22

CodeIgniter4 Version

4.5.4

Shield Version

1.1.0

Shield OAuth Version?

dev-develop

Which operating systems have you tested for this bug?

Windows

Which server did you use?

apache

Database

MySQL 8.0.30

Did you add customize OAuth?

YES, For Facebook Login, following documentation for Login with Yahoo

What happened?

Issue Description

I am encountering a problem when a user who has been soft-deleted attempts to log in using OAuth (Google). I receive the following error:
Screenshot_112

Steps to Reproduce

  1. Soft delete a user.
  2. The user attempts to log in again using OAuth (Google) with the same email.

Expected Output

I would expect the system to handle this situation gracefully, either by permanently deleting the entry for the soft-deleted user or by reactivating the user when they attempt to log in again, or by displaying a notification that the user has previously registered and been deleted by an admin, prompting them to contact the admin for further assistance.

Anything else?

log-2024-09-27 copy.log

@kanglalaw kanglalaw added the bug Something isn't working label Sep 27, 2024
@datamweb datamweb added the good first issue Good for newcomers label Sep 27, 2024
@warcooft warcooft linked a pull request Oct 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants