Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aries_cloudagent/config/logging.py:fileConfig assumes a config file with a handler_timed_file_handler section #3186

Closed
rngadam opened this issue Aug 20, 2024 · 0 comments

Comments

@rngadam
Copy link
Contributor

rngadam commented Aug 20, 2024

1.0.0rc6-42-g0c2fb7b49

simple log file config:

 scripts/run_docker start --no-ledger --no-transport --log-level info --log-file /dev/stdout

results in a reference to a non-existing configuration option:

NoSectionError       (note: full exception trace is shown but execution is paused at: <module>)
No section: 'handler_timed_file_handler'
KeyError: 'handler_timed_file_handler'

During handling of the above exception, another exception occurred:

  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/config/logging.py", line 122, in fileConfig
    cp.set(
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/config/logging.py", line 287, in _setup_log_config_file
    fileConfig(
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/config/logging.py", line 206, in _configure_logging
    cls._setup_log_config_file(log_config_path, log_file)
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/config/logging.py", line 197, in configure
    cls._configure_logging(
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/config/util.py", line 19, in common_config
    LoggingConfigurator.configure(
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/commands/start.py", line 50, in execute
    common_config(settings)
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/commands/__init__.py", line 38, in run_command
    module.execute(argv)
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/__main__.py", line 64, in run
    run_command(command, args)
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/__main__.py", line 75, in main
    run(args)
  File "/home/user/repos/aries-cloudagent-python/aries_cloudagent/__main__.py", line 69, in script_main
    main(sys.argv)
  File "<string>", line 1, in <module> (Current frame)
configparser.NoSectionError: No section: 'handler_timed_file_handler'

code seems to be assuming the existence of a config file with a section handler_timed_file_handler

https://github.com/rngadam/aries-cloudagent-python/blob/0c2fb7b495caf0311adfdbe1d056f426177d3d94/aries_cloudagent/config/logging.py#L121-L133

rngadam added a commit to rngadam/aries-cloudagent-python that referenced this issue Aug 20, 2024
…d_file_handler exists before trying to modify it

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
rngadam added a commit to rngadam/aries-cloudagent-python that referenced this issue Aug 20, 2024
…d_file_handler exists before trying to modify it

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
rngadam added a commit to rngadam/aries-cloudagent-python that referenced this issue Aug 21, 2024
…d_file_handler exists before trying to modify it

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
jamshale pushed a commit to jamshale/acapy that referenced this issue Aug 22, 2024
…d_file_handler exists before trying to modify it (openwallet-foundation#3187)

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
Co-authored-by: Stephen Curran <swcurran@gmail.com>
Signed-off-by: jamshale <jamiehalebc@gmail.com>
rngadam added a commit to rngadam/aries-cloudagent-python that referenced this issue Aug 28, 2024
…d_file_handler exists before trying to modify it

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
darshilnb pushed a commit to Northern-Block/aries-cloudagent-python that referenced this issue Sep 5, 2024
…d_file_handler exists before trying to modify it (openwallet-foundation#3187)

Signed-off-by: Ricky Ng-Adam <ricky.ng-adam-ext@mcn.gouv.qc.ca>
Co-authored-by: Stephen Curran <swcurran@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant