From d2f9fd52fe8b0da6f23775f242660329fcef6421 Mon Sep 17 00:00:00 2001 From: Christopher Speller Date: Fri, 6 May 2016 12:08:49 -0400 Subject: Fixing /logout command (#2908) --- api/command_logout.go | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) (limited to 'api/command_logout.go') diff --git a/api/command_logout.go b/api/command_logout.go index 912093162..9fff4e2d5 100644 --- a/api/command_logout.go +++ b/api/command_logout.go @@ -33,6 +33,16 @@ func (me *LogoutProvider) GetCommand(c *Context) *model.Command { } func (me *LogoutProvider) DoCommand(c *Context, channelId string, message string) *model.CommandResponse { - - return &model.CommandResponse{GotoLocation: c.GetTeamURL() + "/logout", ResponseType: model.COMMAND_RESPONSE_TYPE_EPHEMERAL, Text: c.T("api.command_logout.success_message")} + FAIL := &model.CommandResponse{ResponseType: model.COMMAND_RESPONSE_TYPE_EPHEMERAL, Text: c.T("api.command_logout.fail_message")} + SUCCESS := &model.CommandResponse{GotoLocation: "/", ResponseType: model.COMMAND_RESPONSE_TYPE_EPHEMERAL, Text: c.T("api.command_logout.success_message")} + + // We can't actually remove the user's cookie from here so we just dump their session and let the browser figure it out + if c.Session.Id != "" { + RevokeSessionById(c, c.Session.Id) + if c.Err != nil { + return FAIL + } + return SUCCESS + } + return FAIL } -- cgit v1.2.3-1-g7c22