Issue with configuration

[simmia24]

Encountering configuration issues while attempting to use environment variables for configuration. Authentication is not functioning as expected with this approach. Although using admin YAML resolves the issue, but unable to start a DAG through the UI.

[yottahmd]

Hi @simmia24 👋 Is it possible to provide the admin.yaml and the DAG file to reproduce the problem on my local?

[simmia24]

configs.zip

Hi @yohamta

I have attached admin.yaml with dag file.

[yottahmd]

@simmia24 Thank you for attaching the file! I noticed something in your admin.yaml file that might be causing an issue. Could you try commenting out the line below and see if that solves the problem?

- command: /Users/simranarora/dagu/bin/dagu                 # default: dagu
+ # command: /Users/simranarora/dagu/bin/dagu                 # default: dagu

[simmia24]

Hi @yohamta
Issue is resolved with your solution. Thank You for your help. Can you provide information on how the "command" key was causing an issue? It would be much appreciated if you could elaborate on this.

[yottahmd]

@simmia24 I'm glad to hear that the issue has been resolved with my solution. Regarding the "command" key in the config, we plan to remove it as it is currently unnecessary. If the "command" key is present and causing issues, it may be due to differences in the dagu path or insufficient permissions. We understand that errors can be hard to understand, so we're working to improve this in the future. If you have any further issues or questions, please don't hesitate to create another issue. Thanks!

[dat-adi]

Hi @yohamta, while that solution may have resolved the issue for simran. I did notice that there was a difference in the expected environment variables in the source code as compared to the documentation.

For reference, The expected environment variables from the source code are the following,

	viper.BindEnv("isBasicAuth", "DAGU_IS_BASICAUTH")
	viper.BindEnv("basicAuthUsername", "DAGU_BASICAUTH_USERNAME")
	viper.BindEnv("basicAuthPassword", "DAGU_BASICAUTH_PASSWORD")

However, the mentioned environment variables in the documentation are as given, src: https://dagu.readthedocs.io/en/latest/auth.html

Shall I drop a PR to fix the docs?

[yottahmd]

Hey @dat-adi 👋 Thank you for spotting the issue! I'd really appreciate it if you could submit a pull request to fix the document 🙏 You can find the file here: https://github.com/dagu-dev/dagu/blob/main/docs/source/config.rst

[dat-adi]

Hi @yohamta, I've raised the PR for the same. Hope it helps! :D