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.

Sign up for GitHub

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

Jump to bottom

Cleanup logs #1739

Merged
merged 6 commits into from
Aug 23, 2024
Merged

Cleanup logs #1739

merged 6 commits into from
Aug 23, 2024

Conversation

Frooodle
Copy link
Member

@Frooodle Frooodle commented Aug 22, 2024 •
edited
Loading

Description

cleanup of code
migrated depreciated code
changed logging to WARN for dependencies to reduce loud startup logs
removed authenticated bean that wasn't being used (we were using default it seems, still using BCrypt etc all good)

Checklist:

  • I have read the Contribution Guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@github-actions github-actions bot added Java Pull requests that update Java code Back End Issues related to back-end development Security Security-related issues or pull requests API API-related issues or pull requests labels Aug 22, 2024
@Frooodle Frooodle changed the title Test cleanups Cleanup logs Aug 23, 2024
@Frooodle Frooodle merged commit c7e5987 into main Aug 23, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
API API-related issues or pull requests Back End Issues related to back-end development Java Pull requests that update Java code Security Security-related issues or pull requests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Frooodle