azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Update multiline-secrets.md

Open nohwnd opened this issue 2 years ago • 4 comments

The multiline secret is returned as My\nSecret, where the \n is NOT the newline character, but rather \ and n. This is unexpected, because the documentation says newline, and I as .NET developer where \n is commonly used to denote the invisible newline character expected that those are literal newlines.

nohwnd avatar Oct 21 '22 15:10 nohwnd

@nohwnd : Thanks for your contribution! The author(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Oct 21 '22 15:10 prmerger-automator[bot]

Probably not the right place to ask here, but is this even expected behavior? I would expect that random keys could contain \ followed by n, and then we have no way of knowing if that is unfortunate character combination or real newline.

nohwnd avatar Oct 21 '22 15:10 nohwnd

Learn Build status updates of commit 6618946:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/key-vault/secrets/multiline-secrets.md :white_check_mark:Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

opbld32 avatar Oct 21 '22 15:10 opbld32

@msmbaldwin

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Jak-MS avatar Oct 21 '22 15:10 Jak-MS

@nohwnd : Thanks for your contribution! The author(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Oct 24 '22 10:10 prmerger-automator[bot]

Learn Build status updates of commit 92458b8:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/key-vault/secrets/multiline-secrets.md :white_check_mark:Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

opbld32 avatar Oct 24 '22 10:10 opbld32

Thanks, @nohwnd. I agree with your reasoning and approve this PR. We appreciate your helping us clarify this confusing point. #sign-off

msmbaldwin avatar Oct 26 '22 02:10 msmbaldwin