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

Conflict with SEServer Extender Essentials Plugin #139

Closed
togosh opened this issue Dec 22, 2015 · 10 comments
Closed

Conflict with SEServer Extender Essentials Plugin #139

togosh opened this issue Dec 22, 2015 · 10 comments

Comments

@togosh
Copy link

togosh commented Dec 22, 2015

As a server admin, I love the functionality of SEServerExtender with the Essentials Plugin and the Midspace Admin Script Mod, unfortunately it looks like Essentials and Midspace Admin do not play well together, how could we move forward?

References:

@midspace
Copy link
Owner

We have a permissions system, so it is possible to turn off a command.
https://github.com/midspace/Space-Engineers-Admin-script-mod/wiki/Permissions
/perm command setlevel motd 999

However, I think it still responds to messages with a security warning.
Is it possible for you to check this?

What we could do, it set it so it no longer blocks a message when no one has access.
That way other mods, or ServerExtender can then pick up the message.

@Spcemarine
Copy link
Collaborator

We could allow custom command names that override the old names. That way admins could give certain commands other names and thus the commands of other tools could be used.

midspace added a commit that referenced this issue Dec 24, 2015
Small enhancement to disable commands entirely with with `/perm command
setlevel <cmd> none`
Internally it sets the command level to uint.MaxValue, so it is still in
line with security settings, but it is an explicit value that will treat
the command as non-existent. It won't show up on help either.
@midspace
Copy link
Owner

Weird, I'm sure I posted a comment earlier.

Keen have already released this week's update.
I'll post an update when I get home.

@midspace
Copy link
Owner

I have published an update of the Admin mod with the enhancement.

Please test with SE Server Extender before I close this issue.
I have described in the change comments, use the following to disable the /motd in the Admin helper mod:
/perm command setlevel motd none

If you wish to re-enable it, use:
/perm command setlevel motd 100

@rexxar-tc
Copy link

You'll want to go grab the latest Essentials update if you want to test this mod. Old versions looked for it and disabled some commands.

Thanks, Midspace :)

@midspace
Copy link
Owner

What other commands does Server Extender and Essentials use that Admin helper mod conflict over?

@rexxar-tc
Copy link

/settings and /help. I had to disable them because of an issue I was having with the communication relays those commands can create. I fixed it some time ago, so it should be safe either way. (besides, I need to remove the relays, it's an outdated method)

@midspace
Copy link
Owner

Has this bug been fixed?
Is there any further issues with conflicts with SEServer Extender?
Can I close this issue? 😄

@rexxar-tc
Copy link

As far as I know there aren't any issues. I haven't received any reports, at least.

Actually, now I think about it, the new chat command system in SESE takes care of conflicts. I'll let you know if any do pop up ;)

@midspace
Copy link
Owner

Okay, I'll close this. It can always be reopened or a new bug logged.

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

No branches or pull requests

4 participants