mirror of
https://github.com/librenms/librenms.git
synced 2024-10-07 16:52:45 +00:00
Correct config template for API transport (#8991)
Due to bug in the implementation of the alert transport mapping introduced in #8660 , the API transport is unable to be configured via the new interface. Only the 'get' method is displayed in the UI and the defined validation is expecting uppercase "GET" or "POST" as the method, making it impossible to configure the API transport. This change corrects the name to option value mapping to uppercase GET and adds back the POST method option. DO NOT DELETE THIS TEXT #### Please note > Please read this information carefully. You can run `./scripts/pre-commit.php` to check your code before submitting. - [X] Have you followed our [code guidelines?](http://docs.librenms.org/Developing/Code-Guidelines/) #### Testers If you would like to test this pull request then please run: `./scripts/github-apply <pr_id>`, i.e `./scripts/github-apply 5926`
This commit is contained in:
committed by
Neil Lathwood
parent
abd2c6363f
commit
f3c316f72d
@@ -87,8 +87,8 @@ class Api extends Transport
|
||||
'descr' => 'API Method: GET or POST',
|
||||
'type' => 'select',
|
||||
'options' => [
|
||||
'get' => 'get',
|
||||
'get' => 'get'
|
||||
'GET' => 'GET',
|
||||
'POST' => 'POST'
|
||||
]
|
||||
],
|
||||
[
|
||||
|
Reference in New Issue
Block a user