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

occ command to set values #16

Closed
mmattel opened this issue Aug 21, 2018 · 4 comments
Closed

occ command to set values #16

mmattel opened this issue Aug 21, 2018 · 4 comments
Labels
enhancement New feature or request

Comments

@mmattel
Copy link
Contributor

mmattel commented Aug 21, 2018

@karakayasemi

It would be good to have the ability to configure the parameters also via occ command.
The key names could be like:

  • bfp:get
  • bfp:set

The arguments could be like:

  • --time_threshold
  • --ban_period
  • --fail_tolerance

The options for get could be like:

  • --all or an argument

Pls see also:
https://github.com/owncloud/documentation/issues/4379 (brute_force_protection: settings/admin)
owncloud-archive/documentation#4381 (Splitting occ_command.rst into two files for core and apps)

@mmattel mmattel added the enhancement New feature or request label Aug 21, 2018
@DeepDiver1975
Copy link
Member

App config values can already be Set via occ. Brute firce protection Just uses app config.
Documenting this properly shall be enough.

$this->config->getAppValue(
'brute_force_protection',
'brute_force_protection_fail_tolerance',
'3'
);

@karakayasemi
Copy link
Contributor

@mmattel , as @DeepDiver1975 said, all config values already can be configured via occ command. https://doc.owncloud.org/server/latest/admin_manual/configuration/server/occ_command.html#config-commands.
But, we should document this on documentation and this repository's README file. If everyone agrees, we can close this ticket and follow this topic on the other documentation related issues: #4. #5

@mmattel
Copy link
Contributor Author

mmattel commented Aug 21, 2018

Did not know.
Closing is fine.
I have already created an occ command file for apps (see above).
I will add a note there

@mmattel mmattel closed this as completed Aug 21, 2018
@mmattel
Copy link
Contributor Author

mmattel commented Aug 21, 2018

Added a description in owncloud-archive/documentation#4381

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

No branches or pull requests

3 participants