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

Cleanup code in cache for logs #569

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Cleanup code in cache for logs #569

merged 1 commit into from
Nov 17, 2024

Conversation

javuto
Copy link
Collaborator

@javuto javuto commented Nov 17, 2024

Removing all references of logging status/result/query logs in cache/Redis, since it won't be enabled again as it was causing issues and it was just legacy code. Also removed the configuration parameters from JSON and osctrl-admin and osctrl-tls parameters for the same logging flags.

@javuto javuto added osctrl-tls osctrl-tls related changes osctrl-admin osctrl-admin related changes cache Cache related issues 🧹 cleanup Cleanup related issues labels Nov 17, 2024
@javuto javuto merged commit ac7197a into main Nov 17, 2024
52 checks passed
@javuto javuto deleted the cleanup-expiring-logs-redis branch November 17, 2024 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cache Cache related issues 🧹 cleanup Cleanup related issues osctrl-admin osctrl-admin related changes osctrl-tls osctrl-tls related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant