Skip to content

RMB-1036: Initialize loggers before setting log level (Ramsons)#1211

Merged
julianladisch merged 1 commit into
b35.3from
RMB-1036-ramsons
Dec 8, 2025
Merged

RMB-1036: Initialize loggers before setting log level (Ramsons)#1211
julianladisch merged 1 commit into
b35.3from
RMB-1036-ramsons

Conversation

@julianladisch

Copy link
Copy Markdown
Contributor

https://folio-org.atlassian.net/browse/RMB-1036

Before setting a log level for a class using PUT /admin/loglevel?level=WARN&java_package=org.folio.cql2pgjson.CQL2PgJSON we need to create the logger for that class. It may not have been used since startup.

See https://folio-org.atlassian.net/wiki/spaces/SYSOPS/pages/1407189020/RMB+log+level for details.

If multiple comma separated class names are passed each should be handled.

(cherry picked from commit 238cac8)

https://folio-org.atlassian.net/browse/RMB-1036

Before setting a log level for a class using PUT /admin/loglevel?level=WARN&java_package=org.folio.cql2pgjson.CQL2PgJSON we need to create the logger for that class. It may not have been used since startup.

See https://folio-org.atlassian.net/wiki/spaces/SYSOPS/pages/1407189020/RMB+log+level for details.

If multiple comma separated class names are passed each should be handled.

(cherry picked from commit 238cac8)
@julianladisch julianladisch changed the title RMB-1036: Initialize loggers before setting log level RMB-1036: Initialize loggers before setting log level (Ramsons) Dec 8, 2025
@sonarqubecloud

sonarqubecloud Bot commented Dec 8, 2025

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@julianladisch julianladisch merged commit f8c9537 into b35.3 Dec 8, 2025
7 of 8 checks passed
@julianladisch julianladisch deleted the RMB-1036-ramsons branch December 8, 2025 19:41
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

Successfully merging this pull request may close these issues.

2 participants