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

Modify settings file #24

Closed
Tracked by #5
n0nuser opened this issue Mar 22, 2022 · 0 comments · Fixed by #25 or #29
Closed
Tracked by #5

Modify settings file #24

n0nuser opened this issue Mar 22, 2022 · 0 comments · Fixed by #25 or #29
Assignees
Labels
enhancement New feature or request

Comments

@n0nuser
Copy link
Owner

n0nuser commented Mar 22, 2022

Split config file in sections.

{
    "uvicorn": {
        "host":"0.0.0.0",
        "port":8000,
        "workers":4,
        "reload":true,
        "debug":true,
        "log_level":"trace",
        "ssl_keyfile":null,
        "ssl_keyfile_password":null,
        "ssl_certfile":"None",
        "ssl_version":3,
        "ssl_cert_reqs":null,
        "ssl_ca_certs":null,
        "ssl_ciphers":"TLSv1",
        "limit_concurrency":null,
        "limit_max_requests":null,
        "backlog":2048,
        "timeout_keep_alive":5
    },
    "metrics": {
        "endpoint":true,
        "url":"http://httpbin.org/post",
        "post_interval":60,
        "enable_file":false,
        "metrics_file":"metrics.json"
    },
    "alerts":{
        "url":""
    }
}
@n0nuser n0nuser mentioned this issue Mar 22, 2022
12 tasks
@n0nuser n0nuser self-assigned this Mar 22, 2022
@n0nuser n0nuser added the enhancement New feature or request label Mar 22, 2022
@n0nuser n0nuser added this to To Do in TFG - Monitor App via automation Mar 22, 2022
n0nuser added a commit that referenced this issue Mar 22, 2022
@n0nuser n0nuser mentioned this issue Mar 26, 2022
TFG - Monitor App automation moved this from To Do to Done Mar 26, 2022
n0nuser added a commit that referenced this issue Mar 26, 2022
* Changed Pydantic for JSON config files.

* Fixed poetry entry-point.

It threw the error 'EntryPoint must be in 'name=module:attrs [extras]' format'. Fixed it changing the '-' in 'monitor-agent' for a '_'.

* Implemented receiving settings as a JSON ->FILE<- (not text).

Need to fix testing of data types when a JSON is received

* Reformatted code of 'settings' module

* Implemented alert manager.

Need to check if it's the best way to store the thresholds.

* Fix #24.

* SECURITY: Removed GET Settings.

* Remote Work

* Settings reformmated and FINALLY FIXED. Removed Thresholds POST

Thresholds will now be passed through the config file instead of saving it as a dictionary.

* Bump Dependency Versions
@n0nuser n0nuser linked a pull request Apr 9, 2022 that will close this issue
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
Development

Successfully merging a pull request may close this issue.

1 participant