Skip to content

Fix 'occured' -> 'occurred' typo in JGitEnvironmentRepository clone-error log#3217

Merged
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
SAY-5:fix-jgit-clone-occured-typo
Apr 16, 2026
Merged

Fix 'occured' -> 'occurred' typo in JGitEnvironmentRepository clone-error log#3217
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
SAY-5:fix-jgit-clone-occured-typo

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented Apr 15, 2026

logger.warn message in spring-cloud-config-server/src/main/java/org/springframework/cloud/config/server/environment/JGitEnvironmentRepository.java:679 emitted when the initial JGit clone to the base directory fails read Error occured cloning to base directory. The message reaches operators in spring-cloud-config-server logs. String-literal-only change.

@ryanjbaxter
Copy link
Copy Markdown
Contributor

Can you sign your commit so the DCO passes?

@github-project-automation github-project-automation bot moved this to Todo in 2025.1.2 Apr 15, 2026
@ryanjbaxter ryanjbaxter added this to the 5.0.3 milestone Apr 15, 2026
@SAY-5 SAY-5 force-pushed the fix-jgit-clone-occured-typo branch from ce6f84e to e0c6e45 Compare April 16, 2026 00:11
@SAY-5
Copy link
Copy Markdown
Contributor Author

SAY-5 commented Apr 16, 2026

@ryanjbaxter thanks for the review. Force-pushed the same commit with a Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com> DCO trailer so the DCO check passes.

@ryanjbaxter
Copy link
Copy Markdown
Contributor

Its still not signed correctly, see the error message at the bottom of this page
https://github.com/spring-cloud/spring-cloud-config/pull/3217/checks?check_run_id=71557566731

…rror log

The error message in JGitEnvironmentRepository.java read 'An error occured trying to clone'. Fixed to 'occurred'. Log-message-only change.

Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@SAY-5 SAY-5 force-pushed the fix-jgit-clone-occured-typo branch from e0c6e45 to 7648235 Compare April 16, 2026 01:31
@SAY-5
Copy link
Copy Markdown
Contributor Author

SAY-5 commented Apr 16, 2026

@ryanjbaxter Force-pushed with matching author and Signed-off-by identity (Sai Asish Y <say.apm35@gmail.com>) so the DCO check should pass now. Sorry for the back and forth!

@ryanjbaxter ryanjbaxter merged commit 807d5fa into spring-cloud:main Apr 16, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in 2025.1.2 Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants