Skip to content
This repository has been archived by the owner on Aug 17, 2023. It is now read-only.

modify the default 'settings.yml' text for solving api_key get Nonetype error #8

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

Conversation

TaroballzChen
Copy link

@TaroballzChen TaroballzChen commented Jun 24, 2022

modify the default 'settings.yml' text for solving api_key get Nonetype error

@TaroballzChen TaroballzChen changed the title modify the default 'settings.yml' text for solving api_key get Nonety… modify the default 'settings.yml' text for solving api_key get Nonetype error Jun 24, 2022
@dfirsec
Copy link
Owner

dfirsec commented Jun 24, 2022

Issue resolved for TypeError: 'NoneType' object is not subscriptable concerning missing path for read/write of settings.yml file.

@TaroballzChen
Copy link
Author

TaroballzChen commented Jun 24, 2022

if firstly clone this project and executed without settings.yml, the default flow was created the new settings.yml file with TEXT variable include virustotal api key, but the yml format is NOT correct, so i just modify the TEXT variable content.

@dfirsec
Copy link
Owner

dfirsec commented Jun 24, 2022

Thank you. I recently edited the TEXT output format and it now reads/writes correctly. My Python black formatter was overwriting the original format. I'll apply a comment (locally) to exclude from formatting.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants