Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Leave" button in info panel does not work #9688

Closed
janrudolph opened this issue Feb 13, 2018 · 4 comments · Fixed by #9697
Closed

"Leave" button in info panel does not work #9688

janrudolph opened this issue Feb 13, 2018 · 4 comments · Fixed by #9697

Comments

@janrudolph
Copy link

Description:

When pressing the leave button in the right info panel, an user gets an 400 error message. The user is not able to leave the room.

Leaving by using the channel context menu in the left channel list works.

Server Setup Information:

  • Version of Rocket.Chat Server: 0.61.0

Steps to Reproduce:

  1. Create new channel and invite some other users
  2. Try to leave the channel by clicking on the (i) Icon in the top bar and the on the leave button in the right corner.
  3. You will get the error message.

Expected behavior:

User can leave the channel.

Actual behavior:

User cannot leave the channel by using the leave button in the right column.

leave-400

Leaving by using the button in the left column works.

leaving-left-column

@ThomasRoehl
Copy link
Contributor

On serverside this is the error:

Exception while invoking method 'leaveRoom' Error: Match error: Expected string, got null
I20180212-16:53:25.803(1)?     at exports.check (packages/check.js:55:15)
I20180212-16:53:25.803(1)?     at DDPCommon.MethodInvocation.leaveRoom (/Rocket.Chat/.meteor/local/build/programs/server/packages/rocketchat_lib.js:15033:3)
I20180212-16:53:25.803(1)?     at DDPCommon.MethodInvocation.methodsMap.(anonymous function) (/Rocket.Chat/.meteor/local/build/programs/server/packages/rocketchat_lib.js:2134:26)
I20180212-16:53:25.803(1)?     at DDPCommon.MethodInvocation.methodMap.(anonymous function) (packages/rocketchat_monitoring.js:2731:30)
I20180212-16:53:25.804(1)?     at maybeAuditArgumentChecks (/Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:1826:12)
I20180212-16:53:25.804(1)?     at DDP._CurrentMethodInvocation.withValue (/Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:892:126)
I20180212-16:53:25.804(1)?     at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1186:15)
I20180212-16:53:25.805(1)?     at DDPServer._CurrentWriteFence.withValue (/Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:892:98)
I20180212-16:53:25.805(1)?     at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1186:15)
I20180212-16:53:25.805(1)?     at Promise (/Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:892:46)
I20180212-16:53:25.805(1)?     at new Promise (<anonymous>:null:null)
I20180212-16:53:25.805(1)?     at Session.method (/Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:865:23)
I20180212-16:53:25.806(1)?     at /Rocket.Chat/.meteor/local/build/programs/server/packages/ddp-server.js:744:85
I20180212-16:53:25.806(1)?
I20180212-16:53:25.806(1)? Sanitized and reported to the client as: Match failed [400]
I20180212-16:53:25.807(1)? undefined undefined

@TwizzyDizzy
Copy link

TwizzyDizzy commented Feb 13, 2018

Can confirm this, my server-side error is identical, also here's the log of a JS error on the client side after confirming the leave-warning with "Yes, leave it":

my.host.name-1518535356407.log

Cheers
Thomas

@mrsimpson
Copy link
Collaborator

I noticed that the custom room type's warning text is not respected anymore.

@mrsimpson
Copy link
Collaborator

I am about to fix both issues

mrsimpson added a commit to assistify/Rocket.Chat that referenced this issue Feb 13, 2018
… the channel popover

Introduces a re-use-library for the channel related actions.
engelgabriel added a commit to assistify/Rocket.Chat that referenced this issue Feb 15, 2018
rodrigok added a commit to assistify/Rocket.Chat that referenced this issue Feb 17, 2018
rodrigok added a commit that referenced this issue Feb 19, 2018
[FIX] Harmonize channel-related actions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants