gistfile1.py

def __exit__(self, exc_type, exc_value, traceback):
    try:
        if exc_value is None:
            self.session.commit()
    finally:
        self.session.close()
        
# Should it be this?

def __exit__(self, exc_type, exc_value, traceback):
    if exc_value is None:
        try:
            self.session.commit()
        finally:
            self.session.close()
    else:
        self.sesion.close()