Commit Graph
8 Commits
  • fix(dns_infomaniak): correctly detect API errors (#7048)
    The add/rm success check never rejected anything: for any non-empty API
    response it always reported "Record added"/"Record deleted" and returned
    0, so the _err branch was dead code. A valid key looked fine only because
    the API call genuinely created the record; an invalid key returning
    {"result":"error"} produced the same "Record added" output even though
    nothing was created.
    
    Root cause, in:
      if [ -n "$response" ]; then
        if [ ! "$(echo "$response" | _contains '"result":"success"')" ]; then
    
      - _contains() ignores stdin (it reads only $1 and $2), so the piped
        "$response" was discarded.
      - The pattern '"result":"success"' was passed as $1 (the haystack),
        leaving $2 (the needle) empty, so it ran:
          echo '"result":"success"' | grep -- "" >/dev/null 2>&1
        grep with an empty pattern always matches.
      - That grep output is redirected to /dev/null, so the command
        substitution always captured "", making [ ! "" ] always true.
    
    Fix: call _contains "$response" '"result":"success"' directly and branch
    on its exit code, so error responses now correctly fail (return 1).
    
    Co-authored-by: neil <github@neilpang.com>
  • Rewrite token scope and URL to add one
    Updated comments for clarity and formatting.
  • Update dns_infomaniak.sh
    because infomaniak API v1 no longer works
  • DNS provider API: structured description
    Instead of using comments declare info in a special variable.
    Then the variable can be used to print the DNS API provider usage.
    The usage can be parsed on UI and show all needed inputs for options.
    
    The info is stored in plain string that it's both human-readable and easy to parse:
    
        dns_example_info='API name
         An extended description.
         Multiline.
        Domains: list of alternative domains to find
        Site: the dns provider website e.g. example.com
        Docs: Link to ACME.sh wiki for the provider
        Options:
         VARIABLE1 Title for the option1.
         VARIABLE2 Title for the option2. Default "default value".
         VARIABLE3 Title for the option3. Description to show on UI. Optional.
        Issues: Link to a support ticket on https://github.com/acmesh-official/acme.sh
        Author: First Lastname <authoremail@example.com>, Another Author <https://github.com/example>;
        '
    
    Here:
    VARIABLE1 will be required.
    VARIABLE2 will be required too but will be populated with a "default value".
    VARIABLE3 is optional and can be empty.
    
    A DNS provider may have alternative options like CloudFlare may use API KEY or API Token.
    You can use a second section OptionsAlt: section.
    
    Some providers may have alternative names or domains e.g. Aliyun and AlibabaCloud.
    Add them to Domains: section.
    
    Signed-off-by: Sergey Ponomarev <stokito@gmail.com>