Skip to content

fix: 8 - change scope of --debug logging to local logger instead of globally - #781

Merged
daniel-rdt merged 5 commits into
masterfrom
fix/security-fix-11
Jul 8, 2026
Merged

fix: 8 - change scope of --debug logging to local logger instead of globally#781
daniel-rdt merged 5 commits into
masterfrom
fix/security-fix-11

Conversation

@daniel-rdt

@daniel-rdt daniel-rdt commented Jul 6, 2026

Copy link
Copy Markdown
Member

Changes proposed in this Pull Request

This PR changes the default scope of the root level logging defined in create_zenodo_deposition_cli to info and only changes the local logger instance to debug if required.

Tasks

Workflow

Open issues

  • Possibly upstream this change as the related code is not open-tyndp specific

Notes

Checklist

Required:

  • Security scans show no high-severity bugs, critical vulnerabilities, or exposed secrets.
  • Changes are tested locally and behave as expected.
  • Code and workflow changes are documented.
  • A release note entry is added to doc/release_notes.md.

If applicable:

  • Changes in configuration options are reflected in scripts/lib/validation.
  • Changes in configuration options are added to config/test/*.yaml.
  • Multiple climate years test passes locally (pixi run -e open-tyndp tyndp-cyears-test).
  • For new data sources or versions, these instructions have been followed.
  • Open-TYNDP SPDX license header is added to all touched files.
  • Module docstrings are added to new Python scripts.
  • New rules are documented in the appropriate doc/*.md files.
  • Major features are documented in doc/index.md.

@daniel-rdt daniel-rdt self-assigned this Jul 6, 2026
@daniel-rdt daniel-rdt added SB Scenario Building CBA Cost Benefit Analysis security labels Jul 6, 2026
@daniel-rdt
daniel-rdt requested a review from tgilon July 6, 2026 13:43
@daniel-rdt daniel-rdt removed SB Scenario Building CBA Cost Benefit Analysis labels Jul 6, 2026
@daniel-rdt
daniel-rdt marked this pull request as ready for review July 6, 2026 14:14
# Conflicts:
#	doc/release_notes.md
@tgilon tgilon changed the title fix: 9 - change scope of --debug logging to local logger instead of globally fix: 8 - change scope of --debug logging to local logger instead of globally Jul 8, 2026

@tgilon tgilon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RTM, thank you @daniel-rdt ! I would encourage upstreaming this.

@daniel-rdt
daniel-rdt enabled auto-merge (squash) July 8, 2026 15:55
@daniel-rdt
daniel-rdt merged commit 5d135f2 into master Jul 8, 2026
6 of 7 checks passed
@daniel-rdt
daniel-rdt deleted the fix/security-fix-11 branch July 8, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants