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.

Sign up for GitHub

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

Jump to bottom

Integrate LuckPerms and MagiBridge plugins #276

Open
vorburger opened this issue Feb 25, 2022 · 7 comments
Open

Integrate LuckPerms and MagiBridge plugins #276

vorburger opened this issue Feb 25, 2022 · 7 comments
Assignees
@vorburger
Labels
High SECURITY Technical
Milestone
BYCTWD

Comments

@vorburger
Copy link
Member

Merge https://github.com/OASIS-learn-study/swissarmyknife-minecraft-server into this project.

@edewit
@vorburger
Copy link
Member Author

The priority is LuckPerms and MagiBridge.

@vorburger vorburger added this to the BYCTWD milestone Apr 1, 2022
@vorburger vorburger self-assigned this Apr 1, 2022
@vorburger
Copy link
Member Author

As part of working on this sissue I'll make the container default to creative mode (and perhaps some other default settings).

@vorburger vorburger changed the title Integrate Plugins like LuckPerms, WorldEdit etc. Integrate LuckPerms and MagiBridge plugins Apr 1, 2022
@vorburger vorburger mentioned this issue Apr 1, 2022
Merged
@vorburger
Copy link
Member Author

https://github.com/OASIS-learn-study/swissarmyknife-minecraft-server/issues had a bunch of old additional idea (si jamais).

@vorburger
Copy link
Member Author

As part of working on this sissue I'll make the container default to creative mode

That turned out to be surprisingly painful, because of itzg/docker-minecraft-server#1452.

I'll just add and use a "full" server.properties instead of using docker-minecraft-server respective environment variables.

PR coming up.

@vorburger vorburger mentioned this issue Apr 2, 2022
Merged
This was referenced Apr 11, 2022
Merged
Merged
vorburger added a commit to OASIS-learn-study/swissarmyknife-minecraft-server-binaries that referenced this issue Apr 13, 2022
@vorburger
Replace LuckPerms 4.2.25 with 5.3.98
d543406 
For OASIS-learn-study/minecraft-storeys-maker#276.

According to https://ore.spongepowered.org/Luck/LuckPerms/versions/5.3.98,
this v5.3.98 is (quote) "The final release of LuckPerms for Sponge API 7."

https://luckperms.net/download today has a 5.4.18 which says Spong API 8,
which https://github.com/OASIS-learn-study/minecraft-storeys-maker/
does not yet support.
vorburger added a commit to vorburger/minecraft-storeys-maker that referenced this issue Apr 13, 2022
@vorburger
Add LuckPerms mod to container (re. OASIS-learn-study#276)
f30332f
vorburger added a commit that referenced this issue Apr 13, 2022
@vorburger
Add LuckPerms mod to container (re. #276)
aeb29e4
@vorburger
Copy link
Member Author

develop...vorburger:luckperms2 has work from April.

develop...vorburger:luckperms3 has WIP from today (similar).

My old luckperms.cmds should be re-integrated.

/lp import (now, it changes) needs a JSON GZIP, so we need something else to load luckperms.cmds. Maybe /function?

@vorburger
Copy link
Member Author

#424 adds LuckPerms - but is not much tested, yet; remaining TODOs in future follow-up PRs include:

  1. Full test, with an additional account
  2. The GuardGameModeJoinListener shuold set a connecting Player's gamemode, depending on permissions - I'm not sure if this really still work.
  3. Activate permission(...) in LoginCommand - but how to get Mineflayer user to have that permission?
  4. MagiBridge (see MagiBridge.conf and this script)
  5. Nucleus (remember to add /nucleus setupperms ... to docs/ops.md.

@vorburger
Copy link
Member Author

  1. Activate permission(...) in LoginCommand - but how to get Mineflayer user to have that permission?

Maybe using launcher_accounts.json?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vorburger vorburger

Labels
High SECURITY Technical
Milestone
BYCTWD
Development

No branches or pull requests
1 participant
@vorburger