search

Eigenbahn / ai-dungeon-cli

:european_castle: A cli client to play.aidungeon.io
MIT License
149 stars 33 forks source link

Debug is true in production #28

Closed Oshgnacknak closed 4 years ago

Oshgnacknak commented 4 years ago

I'm getting debug output after installing from the AUR:


 ▄▄▄       ██▓   ▓█████▄  █    ██  ███▄    █   ▄████ ▓█████  ▒█████   ███▄    █
▒████▄    ▓██▒   ▒██▀ ██▌ ██  ▓██▒ ██ ▀█   █  ██▒ ▀█▒▓█   ▀ ▒██▒  ██▒ ██ ▀█   █
▒██  ▀█▄  ▒██▒   ░██   █▌▓██  ▒██░▓██  ▀█ ██▒▒██░▄▄▄░▒███   ▒██░  ██▒▓██  ▀█ ██▒
░██▄▄▄▄██ ░██░   ░▓█▄   ▌▓▓█  ░██░▓██▒  ▐▌██▒░▓█  ██▓▒▓█  ▄ ▒██   ██░▓██▒  ▐▌██▒
 ▓█   ▓██▒░██░   ░▒████▓ ▒▒█████▓ ▒██░   ▓██░░▒▓███▀▒░▒████▒░ ████▓▒░▒██░   ▓██░
 ▒▒   ▓▒█░░▓      ▒▒▓  ▒ ░▒▓▒ ▒ ▒ ░ ▒░   ▒ ▒  ░▒   ▒ ░░ ▒░ ░░ ▒░▒░▒░ ░ ▒░   ▒ ▒
  ▒   ▒▒ ░ ▒ ░    ░ ▒  ▒ ░░▒░ ░ ░ ░ ░░   ░ ▒░  ░   ░  ░ ░  ░  ░ ▒ ▒░ ░ ░░   ░ ▒░
  ░   ▒    ▒ ░    ░ ░  ░  ░░░ ░ ░    ░   ░ ░ ░ ░   ░    ░   ░ ░ ░ ▒     ░   ░ ░
      ░  ░ ░        ░       ░              ░       ░    ░  ░    ░ ░           ░
                  ░

query options (variant #1)
{'user': {'id': '1361639', 'username': 'user_1361639', '__typename': 'User'}, 'content': {'id': 'scenario:458612', 'userId': '2510019', 'contentType': 'scenario', 'contentId
ptions': [{'id': 'scenario:458617', 'title': 'fantasy (recommended)', '__typename': 'Content'}, {'id': 'scenario:458620', 'title': 'mystery', '__typename': 'Content'}, {'id'
nt'}, {'id': 'scenario:458623', 'title': 'zombies', '__typename': 'Content'}, {'id': 'scenario:458624', 'title': 'cyberpunk', '__typename': 'Content'}, {'id': 'scenario:4586
o:459772', 'title': 'archive', '__typename': 'Content'}], 'playPublicId': None, '__typename': 'Content'}}
Pick a setting...

It seam like debug is hard coded to be enabled : https://github.com/Eigenbahn/ai-dungeon-cli/blob/master/ai_dungeon_cli/__init__.py#L33

otreblan commented 4 years ago

@Oshgnacknak This commit should fix it. https://aur.archlinux.org/cgit/aur.git/commit/?h=ai-dungeon-cli&id=c65570ecd5b00e7e2d6fde0e84a1f0387dd0b846

p3r7 commented 4 years ago

Woopsie.

Just released that fixes the issue 0.4.1.

Ideally this flag should only be set via a CLI arg.