Feature Request #3133

Marking forced logins should (at least) be optional

Added by Geert De Deckere almost 4 years ago. Updated over 3 years ago.

Status:ClosedStart date:07/28/2010
Priority:NormalDue date:
Assignee:Isaiah DeRose-Wilson% 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

#2 Updated by Woody Gilk almost 4 years ago

  • Target version changed from v3.0.8 to v3.0.9

#3 Updated by Jeremy Bush over 3 years ago

  • Status changed from Feedback to Assigned
  • Assignee changed from Geert De Deckere to Jeremy Bush

#4 Updated by Jeremy Bush over 3 years ago

  • Target version changed from v3.0.9 to v3.1.0

#5 Updated by Isaiah DeRose-Wilson over 3 years ago

  • Project changed from Kohana v3.x to Auth
  • Category deleted (Modules:Auth)

#6 Updated by Jeremy Bush over 3 years ago

  • Target version changed from v3.1.0 to v3.2.0

#7 Updated by Ivan Brotkin over 3 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?

#8 Updated by Woody Gilk over 3 years ago

  • Target version changed from v3.0.9 to v3.0.10

#9 Updated by Woody Gilk over 3 years ago

  • Assignee changed from Woody Gilk to Jeremy Bush

#10 Updated by Isaiah DeRose-Wilson over 3 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.

Also available in: Atom PDF