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

add a logger config for org.geonetwork.security.external.integration #407

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

landryb
Copy link
Member

@landryb landryb commented Jun 27, 2024

this way we can see errors and warnings when sync fails... otherwise the errors are just silently dropped.

witnessed on demo.geor where we didnt understand why sync wasnt working, those messages were not logged....

2024-06-27T09:15:52,763 WARN  [org.geonetwork.security.external.integration] - Synchronization failed, retrying in 10 SECONDS.
org.springframework.dao.InvalidDataAccessApiUsageException: The given id must not be null!; nested exception is java.lang.IllegalArgumentException: The given id must not be null!

needs backport to 24.0 branch

….integration

this way we cant see errors and warnings when sync fails...
@landryb landryb merged commit 02f9d0f into master Jun 27, 2024
github-actions bot pushed a commit that referenced this pull request Jun 27, 2024
….integration (#407)

this way we cant see errors and warnings when sync fails...

(cherry picked from commit 02f9d0f)
github-actions bot added a commit that referenced this pull request Jun 27, 2024
[24.0] [geonetwork] add a logger config for org.geonetwork.security.external.integration (#407)
github-actions bot pushed a commit that referenced this pull request Jun 27, 2024
….integration (#407)

this way we cant see errors and warnings when sync fails...

(cherry picked from commit 02f9d0f)
Copy link

💚 All backports created successfully

Status Branch Result
24.0
docker-24.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

github-actions bot added a commit that referenced this pull request Jun 27, 2024
[docker-24.0] [geonetwork] add a logger config for org.geonetwork.security.external.integration (#407)
@f-necas f-necas deleted the gn-log-sync branch June 27, 2024 10:27
f-necas pushed a commit that referenced this pull request Sep 4, 2024
….integration (#407)

this way we cant see errors and warnings when sync fails...

(cherry picked from commit 02f9d0f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants