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

[Doc]: Redis Cache documentation includes Storage Account Shared Access Signature that is being flagged by security tooling #26212

Open
jaredfholgate opened this issue Oct 2, 2024 · 1 comment
Assignees
Labels
Azure PS Team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@jaredfholgate
Copy link
Member

Type of issue

Other (describe below)

Feedback

The documentation for RedisCache includes examples showing SAS for storage accounts. This is getting flagged as an issue by security tooling. In my case it was because it got installed to a folder backed up to One Drive. But in any case, it should probably use a masked example to avoid being flagged.

Example here:

Export-AzRedisCache -ResourceGroupName "ResourceGroup13" -Name "RedisCache06" -Prefix "blobprefix" -Container "https://mystorageaccount.blob.core.windows.net/container18?sv=2015-04-05&sr=c&sig=HezZtBZ3DURmEGDduauE7pvETY4kqlPI8JCNa8ATmaw%3D&st=2016-05-27T00%3A00%3A00Z&se=2016-05-28T00%3A00%3A00Z&sp=rwdl"

This get distributed in the Microsoft.Azure.PowerShell.Cmdlets.RedisCache.dll-Help.xml file when Az PowerShell is installed.

Page URL

No response

Content source URL

Export-AzRedisCache -ResourceGroupName "ResourceGroup13" -Name "RedisCache06" -Prefix "blobprefix" -Container "https://mystorageaccount.blob.core.windows.net/container18?sv=2015-04-05&sr=c&sig=HezZtBZ3DURmEGDduauE7pvETY4kqlPI8JCNa8ATmaw%3D&st=2016-05-27T00%3A00%3A00Z&se=2016-05-28T00%3A00%3A00Z&sp=rwdl"

Author

No response

Document Id

No response

@jaredfholgate jaredfholgate added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Oct 2, 2024
@isra-fel
Copy link
Member

@Nickcandy please take a look
Also at xmls of Batch and Compute

@isra-fel isra-fel added Azure PS Team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that and removed needs-triage This is a new issue that needs to be triaged to the appropriate team. labels Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure PS Team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

3 participants