authById - Githubissues

mehrshaddarzi / wp-trait

Fast and standard development of WordPress plugins

MIT License

15 stars 5 forks source link

authById #40

Open 1mahdimf opened 2 years ago

1mahdimf commented 2 years ago

بعضی وقت ها نیازه از روش های غیرمعمول و خاص برای ورود کاربران (ورود با کد تایید) استفاده کنیم. با این فانکشن میشه بصورت شخصی سازی شده و با آیدی کاربر امکان ورود به حساب کاربری بدیم.

باید توجه کنیم قبل استفاده از این امکان حتما موارد امنیتی ورود رو بررسی کنیم و آیدی رو از دیتابیس بگیرم و به این فانکشن انتقال بدیم.

mehrshaddarzi commented 2 years ago

@1mahdimf

please check handle error set cookie when user run this method in wordPress RestAPI
Change method name to 'loginUsingId'
method must be Public

1mahdimf commented 2 years ago

در مورد اول یعنی در خواست restApi رو با خطا برگردونیم؟ متن خطا چی باشه؟
برای نام بهتر نیست از authById استفاده کنیم؟

iniznet commented 2 years ago

Also, we might need it to call do_action( 'wp_login', $user->user_login, $user ) to let other third party play.

The point here is to switch the current user to another user, in my case: authAs('user_login') and authAs(1), so it can be by both user_login and user_id.

1mahdimf commented 2 years ago

@mehrshaddarzi

به نظرم دست برنامه نویس رو باید ازاد گذاشت. خودش چک کنه. این فانکشن اخرین مرحله کاره. یعنی هرچیزی رو برنامه نویس در نظر داره اوکی بود این کد باید اجرا بشه. همچنین اینکه کجا باید اجرا بشه باید دست برنامه نویس باشه.