[GitHub] [airflow] kaxil commented on issue #6901: [AIRFLOW-6345] Ensure arguments to ProxyFix are integers

[GitBox <gi...@apache.org>]

kaxil commented on issue #6901: [AIRFLOW-6345] Ensure arguments to ProxyFix are integers
URL: https://github.com/apache/airflow/pull/6901#issuecomment-568929470
 
 
   CI Failed
   
   ```
   FAILED tests/www/test_app.py::TestApp::test_constructor_proxyfix_args - TypeE...
   
   ____________________ TestApp.test_constructor_proxyfix_args ____________________
   /usr/local/lib/python3.6/contextlib.py:51: in inner
       with self._recreate_cm():
   /usr/local/lib/python3.6/contextlib.py:81: in __enter__
       return next(self.gen)
   tests/test_utils/config.py:42: in conf_vars
       conf.set(section, key, value)
   /usr/local/lib/python3.6/configparser.py:1192: in set
       self._validate_value_types(option=option, value=value)
   _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
   self = <airflow.configuration.AirflowConfigParser object at 0x7fed7fd8e630>
       def _validate_value_types(self, *, section="", option="", value=""):
           """Raises a TypeError for non-string values.
       
           The only legal non-string value if we allow valueless
           options is None, so we need to check if the value is a
           string if:
           - we do not allow valueless options, or
           - we allow valueless options but the value is not None
       
           For compatibility reasons this method is not used in classic set()
           for RawConfigParsers. It is invoked in every case for mapping protocol
           access and in ConfigParser.set().
           """
           if not isinstance(section, str):
               raise TypeError("section names must be strings")
           if not isinstance(option, str):
               raise TypeError("option keys must be strings")
           if not self._allow_no_value or value:
               if not isinstance(value, str):
   >               raise TypeError("option values must be strings")
   E               TypeError: option values must be strings
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services