Skip to content

Conversation

rishabh-f5
Copy link

@rishabh-f5 rishabh-f5 commented Aug 21, 2025

Proposed changes

Checklist

Before sharing this pull request, I completed the following checklist:

Screenshot 2025-08-25 at 9 37 15 PM

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@rishabh-f5 rishabh-f5 requested a review from a team as a code owner August 21, 2025 10:01
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginxaas Issues related to NGINX as a Service for Azure labels Aug 21, 2025
Copy link

github-actions bot commented Aug 21, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@rishabh-f5
Copy link
Author

I have hereby read the F5 CLA and agree to its terms

Y/8vCyctaONln4WT/hdeD9eB//xkQQgghhBBCCCGEEEIIIYQQQggh5FX6CfCArk8AKAAA'
--protected-files '[
{
"content":"c2VjcmV0X2NvbmZpZ3VyYXRpb25fZGF0YQ==",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need the content here, or are these files already in the package?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right! When using --package, the files are already included in the package data, so we don't need to duplicate the content in --protected-files.

InZyilhs/ayvJgGvnChTraHLDuyHD/eW0Zq1c3RYOuqFCUfuUz3Tg4rPznpH/wy/AchRx \
+mMGhxVo7LERZ2p1fmrWl4akxt29K17wRTQPtC3ccRR1D/ijpqmJe4fx698L8ZS3M91m \
Y/8vCyctaONln4WT/hdeD9eB//xkQQgghhBBCCCGEEEIIIYQQQggh5FX6CfCArk8AKAAA'
--protected-files '["/etc/nginx/ssl/certificates.conf", "/etc/nginx/ssl/private.key"]'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rishabh-f5 - did you try running this command and validate if it works? Based on an internal thread, I believe this won't. --protected-files input doesn't work with --package but we still have the ability to specify protected files as part of the json array with key protectedFiles. Please make sure that you test out the command and verify that the files listed as protected are indeed protected.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix package syntax and clarify protected files method and tested

Screenshot 2025-08-25 at 12 30 14 PM

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand; the screenshot seems to be an unrelated az nginx deployment example?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ryepup i have tested azure-cli cmd screen shot i have pasted here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation product/nginxaas Issues related to NGINX as a Service for Azure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants