Conversation
Hi @ranbreuer, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
TTYL, MSBOT; |
@@ -87,3 +87,45 @@ The following decoded JSON web token | |||
"nbf": 1360043456 | |||
} | |||
``` | |||
|
|||
## Adding Permission Scopes to Embed Tokens | |||
When using Embed tokens, one might want to restrict usage of the resources he gives access to. For this reason, you can generate a token with scoped permissions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is public documentation - do we need to review it with technical writers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do, will do this as part of merge to master
|
||
if (datasetId) { | ||
token.addClaim('did', datasetId); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What if you have both rid and did?
@@ -34,6 +44,14 @@ export class PowerBIToken { | |||
} | |||
} | |||
|
|||
if (scopes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it valid not passing any scopes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reverting to defaults
No description provided.