Bulk upload: auto_renew - Bug about ghostwriter HOT 1 CLOSED

@lez0sec Thank you for reporting this! Sorry it has taken me so long to acknowledge this. The world had other plans for my priorities over these past couple of months, but a big update will be coming soon 🙂

Your observation is correct; the logic does not work as intended. The intention was for any entry for auto_renew to be interpreted as True. That's flawed on its own because a No/Disabled/False/etc entry would be treated as "True," but I had meant to revisit that later. The logic was incorrect because it checked if auto_renew was in the row, which it always would be if you had the column in your csv file. It was not checking if it was blank or not after that.

I also did some additional testing with various inputs that showed untrimmed spaces caused errors as well. The csv entries are now stripped before any value checks.

I am using your suggested solution. I slightly expanded it to include a pair of other potential options. This will be in the next update I am bringing in as a PR shortly:

            # Accept a variety of "True" values to mean True
            # Thanks to @lez0sec for fixing this logic:
            #   https://github.com/GhostManager/Ghostwriter/issues/73
            if 'auto_renew' in entry:
                if any(yes_option in entry['auto_renew'].lower().strip().strip() for yes_option in ['yes', 'enabled', 'true', 'x', 'enable']):
                    entry['auto_renew'] = True
                else:
                    entry['auto_renew'] = False

from ghostwriter.