Needs work
Project:
Drupal core
Version:
main
Component:
user.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 Oct 2018 at 17:51 UTC
Updated:
21 Aug 2023 at 13:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shawnmatthews commentedRe-roll after reading - https://www.drupal.org/node/1319154
Comment #4
katzillaHi @shawnmatthews. Thanks for your patch - exactly what I needed for my project :) The patch did not apply against 8.6.13 because of some missing Quotes in the .yml File. Also, this should be a GET and not a POST request, because we are not sending over any data. Attached a new patch.
Comment #6
katzillanew patch for d8.9
Comment #7
katzillawrong filename ;)
Comment #8
katzillaComment #10
inders commented+1 for this issue.
As of now, I am exposing this manually inside one of rest resource plugin. logout_token is needed for avoiding 403 errors in case of App. I was loosing my local session storage sometime but it was logged in at server. So was never able to login again inside App. I had to manually expose it from rest resource and use it inside Vue App. While doing error handling , it was easy to get this token and logging out from App.
Thank you!
Comment #12
mbovan commentedWould it make sense to use
session/token/logoutpath instead?Comment #13
mbovan commentedI addressed #12.
Comment #14
adityasingh commentedFixed coding standards issue.
Comment #15
adityasingh commentedAdding the interdiff.
Comment #17
sadikyalcin commentedI'm keep getting
access denied / This route can only be accessed by authenticated usersafter upgrading to 9.x. Any pointers on what the issue might be? I am definitely logged in.Comment #21
smustgrave commentedThis issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request as a guide.
public function logoutToken(Request $requestShould be typehinted
This will need test coverage.
Also will need a change record to announce the new route.
Comment #23
tcfunk commentedWould it make sense to just return the token value as text (similar to the response from /session/token) instead of as a json object?
Comment #24
maxilein commentedIn the meantime see: https://www.drupal.org/project/logout_token