AceCentre / AACSpeakHelper

Copies the pasteboard. Translates to defined lang, Reads aloud and replaces pasteboard with translated text
https://docs.acecentre.org.uk/products/v/aac-speak-helper-tool/
MIT License
0 stars 1 forks source link

Fixes for encryption #61

Closed willwade closed 1 month ago

willwade commented 1 month ago

I think we are missing configure_enc_utils in GUI dir as its referencing that - and he config.enc in that dir, But somethings not right with that either as if I copy them both into the widget dir i get

Traceback (most recent call last): File "C:\GitHub\TranslateAndTTS\GUI_TranslateAndTTS\widget.py", line 1751, in ms_token = config.get("azureTTS")["key"] ^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'get'

NB: - Make sure we reference config.enc to the same dir as settings.cfg - i say we just embed the key methods in widget.py

acerjay commented 1 month ago

I think we are missing configure_enc_utils in GUI dir as its referencing that - and he config.enc in that dir, But somethings not right with that either as if I copy them both into the widget dir i get

Traceback (most recent call last): File "C:\GitHub\TranslateAndTTS\GUI_TranslateAndTTS\widget.py", line 1751, in ms_token = config.get("azureTTS")["key"] ^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'get'

NB: - Make sure we reference config.enc to the same dir as settings.cfg - i say we just embed the key methods in widget.py load config added more lines of repeated code. i'm hoping this hack will save the day :D image Moreover, both of the GUI and Client is in the same folder so i think it is safe. Re: on Azure error, maybe setting.cfg is not included during your copy? I hope my added code fix the issue.

willwade commented 1 month ago

I have a feeling it will always fail that workflow as the env cant be applied to PR..

Running locally it makes it fine - but when I run widget.py I see

Traceback (most recent call last): File "C:\GitHub\TranslateAndTTS\GUI_TranslateAndTTS\widget.py", line 1751, in ms_token = config.get("azureTTS")["key"] ^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'get'

so config is being set to None. Interesting.

in the log

2024-09-28 06:36:04,943 � root � INFO � load_config:92 � Successfully decrypted configuration from config.enc. 2024-09-28 06:36:04,944 � root � INFO � load_config:121 � No settings.cfg file found. Skipping overrides. 2024-09-28 06:36:04,944 � root � INFO � load_config:123 � Final configuration loaded successfully. 2024-09-28 06:36:04,944 � root � INFO � :1754 � Configuration Loaded Successfully.

willwade commented 1 month ago

I think for me your sys.path append trick isnt working.. im going to merge and see what happens on GH and automated build.