fix bug: DotEnv._get_stream raise IOError when .env is a directory rather than a file (theskumar#120)

ausaki · theskumar · commit b7f5a6749859 · 2018-08-09T17:10:39.000+05:30
# How this bug happened ?

The function `find_dotenv` try to find a file named `.env`, if there is a directory named `.env` by coincidence, then `DotEnv._get_stream` will  raise IOError.

# Fix this bug

use `os.path.isfile` instead of `os.path.exists`.
diff --git a/dotenv/main.py b/dotenv/main.py
@@ -57,7 +57,7 @@ def _get_stream(self):
         if isinstance(self.dotenv_path, StringIO):
             return self.dotenv_path
 
-        if os.path.exists(self.dotenv_path):
+        if os.path.isfile(self.dotenv_path):
             self._is_file = True
             return io.open(self.dotenv_path)
 
@@ -241,7 +241,7 @@ def find_dotenv(filename='.env', raise_error_if_not_found=False, usecwd=False):
 
     for dirname in _walk_to_root(path):
         check_path = os.path.join(dirname, filename)
-        if os.path.exists(check_path):
+        if os.path.isfile(check_path):
             return check_path
 
     if raise_error_if_not_found:
