flathunters / flathunter

A bot to help people with their rental real-estate search. 🏠🤖
GNU Affero General Public License v3.0
853 stars 183 forks source link

`Config file loaction must be specified` after having pulled commits made after 29.04.24 #588

Closed robertszooba closed 6 months ago

robertszooba commented 6 months ago

Despite unstable operation and frequent crashes due to various errors (IS24 bot detection, undetected_chromedriver) I had managed to run the program until now. Unfortunately, after executing the pull command, which updated my version with commits made after 29/04/24, I am unable to start the program at all due to the following error: flathunter.exceptions.ConfigException: Config file loaction must be specified, or FLATHUNTER_TARGET_URLS must be set. Has anything changed in the way the configuration file (config.yaml) is processed?

codders commented 6 months ago

Hey hey,

Yeah - I changed the config parsing yesterday. The tests passed, but I didn't try running the app manually. Can you roll back to the previous version and check that that fixes your issue?

Thanks

robertszooba @.***> schrieb am Sa., 11. Mai 2024, 11:41:

Despite unstable operation and frequent crashes due to various errors (IS24 bot detection, undetected_chromedriver) I had managed to run the program until now. Unfortunately, after executing the pull command, which updated my version with commits made after 29/04/24, I am unable to start the program at all due to the following error: flathunter.exceptions.ConfigException: Config file loaction must be specified, or FLATHUNTER_TARGET_URLS must be set. Has anything changed in the way the configuration file (config.yaml) is processed?

— Reply to this email directly, view it on GitHub https://github.com/flathunters/flathunter/issues/588, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAEK5S44CSZUCHQEPWV7ITZBXRUZAVCNFSM6AAAAABHRZH5RSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4TAOBQHE2TKMI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

robertszooba commented 6 months ago

Reverting to the newest commit (as of 11/5) to the one with the message Fix quoting issue in config dis (9/5) solves the issue.

codders commented 6 months ago

I made a change in #591 that should resolve the issue you saw