Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix s3cfg_path handling when HOME does not exists #172

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix s3cfg_path handling when HOME does not exists #172

wants to merge 1 commit into from

Commits on Feb 6, 2020

  1. fix s3cfg_path handling when HOME does not exists 

    following exception occurs when enviromental variable HOME is not set in s3_keys_from_s3cfg method

Exception UnboundLocalError: local variable 's3cfg_path' referenced before assignment

exception is triggered by accessing the dictionary key HOME which does not exist, resulting in s3cfg_path is not set. but then, the variable s3cfg_path is sadly used in the Exception handling.

this patch fixes this by checking the presence of HOME environmental variable. if not found, method returns None
    @honza801
    honza801 authored Feb 6, 2020
    Configuration menu
    Copy the full SHA
    a29d249 View commit details
    Browse the repository at this point in the history