Feature Request #3133
Marking forced logins should (at least) be optional
| Status: | Closed | Start date: | 07/28/2010 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | % Done: | 0% |
||
| Category: | - | |||
| Target version: | Kohana v3.x - v3.0.10 | |||
| Resolution: | fixed | Points: |
Description
The ORM Auth driver always marks sessions when you call force_login:
// Mark the session as forced, to prevent users from changing account information
$this->_session->set('auth_forced', TRUE);
This behaviour should be optional. You don't always need this extra session variable. Actually, I've never needed it. Other ways exist to limit user permissions, like roles. Moreover, calling
force_login doesn't always imply you want to limit permissions in the first place. For example, I'm using it to create a regular auth session after somebody logged in via OAuth.
Ideally, I'd just get rid of auth_forced session setting. I guess I could live with an extra parameter too:
function force_login($user, $mark_session_as_forced = FALSE)
History
Updated by Ivan Brotkin over 2 years ago
Updated by Woody Gilk over 2 years ago
- Target version changed from v3.0.8 to v3.0.9
Updated by Jeremy Bush over 2 years ago
- Status changed from Feedback to Assigned
- Assignee changed from Geert De Deckere to Jeremy Bush
Updated by Jeremy Bush over 2 years ago
- Target version changed from v3.0.9 to v3.1.0
Updated by Isaiah DeRose-Wilson over 2 years ago
- Project changed from Kohana v3.x to Auth
- Category deleted (
Modules:Auth)
Updated by Jeremy Bush over 2 years ago
- Target version changed from v3.1.0 to v3.2.0
Updated by Ivan Brotkin over 2 years ago
- Assignee changed from Jeremy Bush to Woody Gilk
- Target version changed from v3.2.0 to v3.0.9
It was implemented 3 months ago, whats wrong?
Updated by Woody Gilk over 2 years ago
- Target version changed from v3.0.9 to v3.0.10
Updated by Woody Gilk over 2 years ago
- Assignee changed from Woody Gilk to Jeremy Bush
Updated by Isaiah DeRose-Wilson over 2 years ago
- Status changed from Assigned to Closed
- Assignee changed from Jeremy Bush to Isaiah DeRose-Wilson
- Resolution set to fixed
Looks like this was merged in a while ago but never got closed.