MicrosoftDocs / powerbi-docs

Creative Commons Attribution 4.0 International

444 stars 844 forks source link

Fix dataset stringification in RLS effective identity to produce a valid request #5096

Closed shoedler closed 1 month ago

shoedler commented 1 month ago

Calling .ToString() on the list of datasetIds will not return the expected individual GUIDs of the datasets. Instead, it returns a string representation of the list type.

This results in an invalid request, leading to a failure with a status code of 400 when attempting to generate an embed token using pbiClient.EmbedToken.GenerateToken(tokenRequest) a few lines later.

learn-build-service-prod[bot] commented 1 month ago

Learn Build status updates of commit 90f57f3:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
powerbi-docs/developer/embedded/cloud-rls.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

Court72 commented 1 month ago

@mberdugo

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team