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

Add security whitelist/blacklist for executed commands #18

Closed
wants to merge 2 commits into from
Closed

Add security whitelist/blacklist for executed commands #18

wants to merge 2 commits into from

Conversation

rayanbzd
Copy link

@rayanbzd rayanbzd commented Jan 8, 2022

2 new configurable options available on the config.json:

  • whitelistedCommands: list of whitelisted commands. Only the commands on this list can be executed on the console over the bridge.
  • useBlacklist: invert the whitelistedCommands to blacklist system. All commands can be executed over the bridge except the whitelistedCommands list.
  • if whitelistedCommands is set to null, this will disable the whitelist/blacklist system
    This system is disabled by default.

rayanbouzidi added 2 commits January 8, 2022 01:05
2 new configurable options available on the config.json:
- whitelistedCommands: list of whitelisted commands. Only the commands on this list can be executed on the console over the bridge.
- useBlacklist: invert the whitelistedCommands to blacklist system. All commands can be executed over the bridge except the whitelistedCommands list.
- if whitelistedCommands is set to null, this will disable the whitelist/blacklist system
This system is disabled by default.
@Eywek
Copy link
Member

Eywek commented Jan 9, 2022

@nivcoo t’as accès en write mtn

@nivcoo
Copy link
Member

nivcoo commented Jan 9, 2022

Il faut que tu upload la version sous forme de jar à la racine @Futurixel

@WarnDa
Copy link

WarnDa commented Jan 26, 2023

P'tite maj sur ça ?

@rayanbzd rayanbzd closed this by deleting the head repository Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants