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

Split on cert filenames #10

Merged
merged 2 commits into from
Aug 7, 2023
Merged

Split on cert filenames #10

merged 2 commits into from
Aug 7, 2023

Conversation

Oliver-ctrlo
Copy link
Contributor

Renamed $filename to $full_filename

Added a split (on comma) to this this value to separate the different paths that might be listed within the certificate's filename.

Then added a loop to run through each path and treat it as a separate certificate filename.

This allows certificates to be saved into multiple location/files by using a comma separated list in the filename field.

Renamed $filename to $full_filename

Added a split on this this value to separate the different paths that might be listed within the certificate's filename (based on a comma).

Then added a loop to run through each path and treat it as a separate certificate filename. 

This allows certificates to be saved into multiple location/files by using a comma separated list in the filename field
@abeverley
Copy link
Contributor

Thanks Oliver, looks good. Can you just fix the indenting please for the new loop? Also, it would be worth doing foreach my $filename rather than reassigning to $filename afterwards.

@abeverley abeverley merged commit cffaea8 into ctrlo:master Aug 7, 2023
1 check passed
@abeverley
Copy link
Contributor

Thanks Oliver, merged and deployed

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.

2 participants