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

Feature Request: Choose Assistant version type for Assistant invites #3718

Closed
Brodur opened this issue Feb 28, 2022 · 18 comments · Fixed by #5672
Closed

Feature Request: Choose Assistant version type for Assistant invites #3718

Brodur opened this issue Feb 28, 2022 · 18 comments · Fixed by #5672
Assignees

Comments

@Brodur
Copy link

Brodur commented Feb 28, 2022

When using the add agent dialogue, you can specify which version to use. Ideally for invite links, emails, and codes you should also have a mechanism for this.
image

Failing that, I would like a way to specify the default version of assistant that gets shipped with an invite, as right now by default it is the system tray app which is not conducive to use as a one-off remote support tool as users may be confused or unclear on how to find the application running in the system tray after launching it. I'd prefer the application (always connected) be the default, as it does require user consent before a session can be started.

@Ylianst Ylianst self-assigned this Feb 28, 2022
@KBCACheia
Copy link

Good idea, any news about it?

@svewa
Copy link

svewa commented Sep 15, 2022

Came to make the same issue, good to see it's already open :-)
For an assistant situation I also can only see the application/always connected as desirable..

@si458 si458 assigned si458 and unassigned Ylianst Jan 6, 2024
@si458
Copy link
Collaborator

si458 commented Jan 6, 2024

just thinking outloud,
would an option under domains like assistantTypeInvite and setting it to say 0,1,2,3,4 be ok?
each number would link to the different types,
then when set to a certain number for that domain, all invite agent links will add the extra bit to download that type of assistant?

@Brodur
Copy link
Author

Brodur commented Jan 6, 2024

just thinking outloud, would an option under domains like assistantTypeInvite and setting it to say 0,1,2,3,4 be ok? each number would link to the different types, then when set to a certain number for that domain, all invite agent links will add the extra bit to download that type of assistant?

So long as the solution is documented, good enough for me!

@si458
Copy link
Collaborator

si458 commented Jan 6, 2024

PR done! only 2 years late, lets not rush this now...

assistantTypeAgentInvite must be integer, and default is 0 (left as 0 for backwards compatibility), can be set PER DOMAIN
0 = System Tray, Connect on user request
1 = System Tray, Always Connected
2 = Application, Connect on user request
3 = Application, Always connected
4 = System Tray, Monitor Only

@Brodur
Copy link
Author

Brodur commented Jan 14, 2024

PR done! only 2 years late, lets not rush this now...

assistantTypeAgentInvite must be integer, and default is 0 (left as 0 for backwards compatibility), can be set PER DOMAIN 0 = System Tray, Connect on user request 1 = System Tray, Always Connected 2 = Application, Connect on user request 3 = Application, Always connected 4 = System Tray, Monitor Only

Good day. I'm testing this out now and I'm not seeing the expected results. The agent seems to still default to the system tray application when I specify the following in the config:

  "domains": {
        "": {
                "agentInviteCodes": true,
                "assistantTypeAgentInvite": 3
        }
  },

Am I missing something in the formatting? I'd expect this config to yield the Application, Always Connected.

@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

have you updated to the latest release 1.1.18?

@Brodur
Copy link
Author

Brodur commented Jan 14, 2024

Yes, I verified server version 1.1.18 before testing. I used the inbuilt upgrade as I'm running on the typhonragewind/meshcentral docker image and that doesn't seem to be at release 1.1.18 yet.

Unsure on relevance, but I'll include this anyway - I did have an error occur during the upgrade, looks to be with node mailer.

-------- 1/14/2024, 3:58:45 PM ---- 1.1.18 --------

node:internal/modules/cjs/loader:1147
  throw err;
  ^

Error: Cannot find module 'nodemailer'
Require stack:
- /opt/meshcentral/node_modules/meshcentral/meshmail.js
- /opt/meshcentral/node_modules/meshcentral/meshcentral.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at module.exports.CreateMeshMail (/opt/meshcentral/node_modules/meshcentral/meshmail.js:53:32)
    at RandomBytesJob.<anonymous> (/opt/meshcentral/node_modules/meshcentral/meshcentral.js:1766:67)
    at RandomBytesJob.<anonymous> (node:internal/crypto/random:113:5)
    at RandomBytesJob.onJobDone (node:internal/crypto/random:308:3) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/opt/meshcentral/node_modules/meshcentral/meshmail.js',
    '/opt/meshcentral/node_modules/meshcentral/meshcentral.js'
  ]
}

Node.js v20.11.0

But the server started fine after that and appears to be functioning as intended for the most part.

@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

yes we are aware the built-in upgrade might break!
try the official docker image https://github.com/Ylianst/MeshCentral/pkgs/container/meshcentral
in theory its a like for like - ghcr.io/ylianst/meshcentral:1.1.18
ALSO NOTE: NEVER USE THE WEB-UPGRADE IN DOCKER AS IT WILL DEFO BREAK!
im going to do a PR to warn users if they are using docker to NOT use this option!

@Brodur
Copy link
Author

Brodur commented Jan 14, 2024

yes we are aware the built-in upgrade might break! try the official docker image https://github.com/Ylianst/MeshCentral/pkgs/container/meshcentral in theory its a like for like - ghcr.io/ylianst/meshcentral:1.1.18 ALSO NOTE: NEVER USE THE WEB-UPGRADE IN DOCKER AS IT WILL DEFO BREAK! im going to do a PR to warn users if they are using docker to NOT use this option!

I wasn't aware there was an official docker, nice! Updated my compose to use that - no change in behaviour observed though unfortunately. I also confirmed my config is loading - I see the line when I "Show server configuration."
Sidenote - I've had the upgrades work in the past on docker, they just never persist past a container restart as one would expect.

@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

if you check the url on the agentinvite page for the assistant link,
does it include &ac=NUMBER at the end of it or does the link not include it at all?
if is not including it at all, you must be on an old version still or maybe the page needs a CTRL+F5 or private mode
if it does but its the wrong number then its something else

@Brodur
Copy link
Author

Brodur commented Jan 14, 2024

The download link for the agent does appear to have the &ac=3 appended to the end. For a sanity test, I tried downloading the assistant agent from the add agent pane, and I think I may see an issue there - that download does not have the expected behavior either. When selecting application always connected there, I instead receive the system tray application.

@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

fuck! i know what ive done wrong
i implemented a fix for it always running in direct connect mode when you asked for monitor only #5693
i have a feeling this had fucked up the ac= option now, as its not right when u download it from the web ui now too rather than the agentinvite!

@si458 si458 reopened this Jan 14, 2024
@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

just pushed the fix for you! 0b0f299
i commited the wrong line in the original PR, it was a late night!
so you can either patch the webserver.js manually
OR
wait for a new release
OR
try the master docker image (wait 15mins for it to build first)
(this gets built every time we push a commit to the master branch so you can test new featres/bug fixes)

@Brodur
Copy link
Author

Brodur commented Jan 14, 2024

I'm fine with waiting, thanks for the fix! I'll give it another whirl when the next release hits.

@si458
Copy link
Collaborator

si458 commented Jan 14, 2024

the latest master image has been built if you wanted to try that,
it has the fix included AND get clipboard over the web-rdp haha

@si458 si458 closed this as completed Jan 14, 2024
@Brodur
Copy link
Author

Brodur commented Feb 9, 2024 via email

@silversword411
Copy link
Contributor

#5788

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

Successfully merging a pull request may close this issue.

6 participants