Planner app only access token check. #3261
-
|
Currently it's only possible to request Planner endpoints with a delegated token. App only tokens are not supported. I was wondering if it is useful to write a token check for these commands. When the user is logged in with an app only token, we could display an error message, something like I'm not 100% sure if this is a useful enhancement. What do you guys think? |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 3 replies
-
|
Great suggestion! We can't do it in the validation stage, but we could do it as the first thing inside the command action. Ideally, let's centralize the logic in a util function that we can reuse across all command's where it applies. |
Beta Was this translation helpful? Give feedback.
-
|
Indeed was thinking about something like |
Beta Was this translation helpful? Give feedback.
-
|
Ok do I create an issue for this one with a list of all commands to update? |
Beta Was this translation helpful? Give feedback.
-
|
Please do! |
Beta Was this translation helpful? Give feedback.
Great suggestion! We can't do it in the validation stage, but we could do it as the first thing inside the command action. Ideally, let's centralize the logic in a util function that we can reuse across all command's where it applies.