Skip to content

Conversation

tugytur
Copy link
Contributor

@tugytur tugytur commented Sep 10, 2025

Proposed changes

  • Change public cert permission
  • Fix variable name collision

Change the permission for the uploaded public certificate to 0644 because certain applications try to read the whole /etc/ssl/certs directory and throw warnings if some certs can't be read.

Resolve template error where loop variables log were shadowing the parent log dictionary, causing "object of type 'str' has no attribute 'error'" failures. Changed loop variable names to access_item and error_item to avoid naming conflicts. Don't enforce gzip variable to be a string.

Checklist

Before creating a PR, run through this checklist and mark each as complete:

@tugytur tugytur requested a review from a team as a code owner September 10, 2025 09:11
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.

1 participant