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

QuickMap RDP mapping not working for regular users #100

Closed
rodrigo-hissam opened this issue Sep 11, 2024 · 8 comments
Closed

QuickMap RDP mapping not working for regular users #100

rodrigo-hissam opened this issue Sep 11, 2024 · 8 comments
Assignees
Labels

Comments

@rodrigo-hissam
Copy link

Describe the bug
After a regular user logs into meshrouter and tries to do a quickmap RDP launch, no action is taken as the quickmap silently fails. Launching meshrouter in debug mode we can see this:

ExceptionSink: System.NullReferenceException: Object reference not set to an instance of an object.
at MeshCentralRouter.MainForm.QuickMap(Int32 protocol, Int32 port, Int32 appId, NodeClass node)
at MeshCentralRouter.MainForm.rdpToolStripMenuItem_Click(Object sender, EventArgs e)
at System.Windows.Forms.ToolStripItem.RaiseEvent(Object key, EventArgs e)
at System.Windows.Forms.ToolStripMenuItem.OnClick(EventArgs e)
at System.Windows.Forms.ToolStripItem.HandleClick(EventArgs e)
at System.Windows.Forms.ToolStripItem.HandleMouseUp(MouseEventArgs e)
at System.Windows.Forms.ToolStrip.OnMouseUp(MouseEventArgs mea)
at System.Windows.Forms.ToolStripDropDown.OnMouseUp(MouseEventArgs mea)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ToolStrip.WndProc(Message& m)
at System.Windows.Forms.ToolStripDropDown.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

To Reproduce
Steps to reproduce the behavior:

As a regular user, right click the device you want to RDP into and choose RDP
No RDP mapping is created, so nothing happens to the user
Expected behavior
An automatic mapping should be created and the RDP launch windows should appear.

Server Software (please complete the following information):

OS: Ubuntu 24.04.1
Version: MeshCentral v 1.29.1
Node: v19.8.1
Additional context
Admin users can still use meshrouter normally, only regular users seem to not be able to use the quickmap function.

@si458
Copy link
Collaborator

si458 commented Sep 12, 2024

right so indeed the issue is to do with the meshcentralrouter

a work around is to get a user to login to the web ui, then click the RDP option when they select a device,
then let the browser open up meshcentralrouter and it works

but will look into it asap

@si458 si458 self-assigned this Sep 12, 2024
@si458 si458 added the bug label Sep 12, 2024
@si458
Copy link
Collaborator

si458 commented Sep 12, 2024

@rodrigo-hissam @mcallisaya79 this should be fixed now.
please can you try this version? MeshCentralRouter.zip
if it asks to update say NO

@rodrigo-hissam
Copy link
Author

@si458 Just did a quick test and the quickmap function seems be working once again in this version you linked.

image

@si458
Copy link
Collaborator

si458 commented Sep 12, 2024

@rodrigo-hissam great thanks!
it was because it was checking if the device was in a group for relaying
but individual devices DONT have meshgroups as such
so hense it was erroring out!
also discovered an RDP bug where it was opening up multiple RDP ports over and over again,
rather than using the port that was already open, doh!

@si458 si458 closed this as completed Sep 12, 2024
@rodrigo-hissam
Copy link
Author

Thank you!! Will this end up being included in a new meshcentral release?

@si458
Copy link
Collaborator

si458 commented Sep 12, 2024

yes, ill will add it in later into the main repo so it will be available in the next release 👍
in the mean time, you can copy and paste the exe from above into node_modules/meshcentral/agents/
then restart your meshcentral, and everytime other people open up there version, they will get an update alert
and it will download the new version u patched from above 👍

@rodrigo-hissam
Copy link
Author

Thank you once again @si458

@si458
Copy link
Collaborator

si458 commented Sep 12, 2024

uploaded to main repo too ready for next release!
PS dont forget to donate if you can ❤️ https://www.si458.co.uk/2024/01/05/donation/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants