Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-54511

[Examinador] Python pylint linter failing because HTTP requests are done without specifying timeouts

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • Major
    • None
    • None
    • tools
    • None

    Description

      What needs to be done?
      Examinador build is failing because the pylint missing-timeout linter is picking on some Python HTTP request functions not being supplied with the timeout argument:

      ************* Module setup_utils
      		 
      libraries/setup_utils.py:107:14: W3101: Missing timeout argument for method 'requests.get' can cause your program to hang indefinitely (missing-timeout)
      		 
      ************* Module utils
      		 
      libraries/utils.py:83:10: W3101: Missing timeout argument for method 'requests.delete' can cause your program to hang indefinitely (missing-timeout)

      We need to add reasonable timeouts for those.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            safian.ali Safian Ali
            maks.januska Maksimiljans Januska
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty