Project

General

Profile

Feature #27990

Add logout method to auth plugin

Added by Erin Clark about 1 year ago. Updated 10 months ago.

Status:
Done
Priority:
Blocks Until Resolved
Assignee:
Brandon Schneider
Category:
Middleware
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

Now that tokens can be used for authentication everywhere in the new ui we need a method to invalidate the token when the user logs out (i.e. auth.logout).

TokenExpired-2018-06-14.png (90.4 KB) TokenExpired-2018-06-14.png Michael Reynolds, 06/14/2018 12:21 PM
18559

Related issues

Related to FreeNAS - Bug #27817: Root login doesn't require password on new UIDone2018-01-16

Associated revisions

Revision 0fc76327 (diff)
Added by Brandon Schneider about 1 year ago

feat(middlewared): Add logout method to auth plugin

This allows removing a token's access to the system

Ticket: #27990

Revision 0310fdac (diff)
Added by Brandon Schneider about 1 year ago

feat(middlewared): Add logout method to auth plugin

This allows removing a token's access to the system

Ticket: #27990

Revision 232bee18 (diff)
Added by Brandon Schneider about 1 year ago

feat(middlewared): Add logout method to auth plugin (#849)

  • feat(middlewared): Add logout method to auth plugin

This allows removing a token's access to the system

Ticket: #27990

  • Fix bad call for pop_token, check if a token exists
  • Update docstring
  • Add empty accepts decoration

History

#1 Updated by Erin Clark about 1 year ago

  • Related to Bug #27817: Root login doesn't require password on new UI added

#2 Avatar?id=14398&size=24x24 Updated by Kris Moore about 1 year ago

  • Assignee changed from Vladimir Vinogradenko to Brandon Schneider
  • Priority changed from Expected to Blocks Until Resolved
  • Severity set to New

Brandon,

This seems a super good one to do some initial work on Middlewared. Please work with Erin when its in so she can use it on the UI side.

#3 Updated by Brandon Schneider about 1 year ago

  • Status changed from Not Started to Done

#4 Updated by William Grzybowski about 1 year ago

  • Status changed from Done to In Progress

Lets not set status to Done until the PR is merged please ; )

#5 Avatar?id=13649&size=24x24 Updated by Ben Gadd about 1 year ago

  • Due date set to 03/12/2018

#6 Updated by Brandon Schneider about 1 year ago

  • Status changed from In Progress to Done

#7 Updated by Dru Lavigne about 1 year ago

  • Subject changed from Need logout method to Add logout method to auth plugin
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#8 Updated by Dru Lavigne 12 months ago

  • Status changed from Done to Ready for Testing

#9 Updated by Michael Reynolds 10 months ago

18559

Spoke with Brandon and Erin who confirmed this is fixed.

see also the screenshot showing the auth token expired message

#10 Updated by Dru Lavigne 10 months ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF