-
Notifications
You must be signed in to change notification settings - Fork 1k
Task 1924572 Sample 4-1 basher update #602
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
Task 1924572 Sample 4-1 basher update #602
Conversation
to sign-in a user In reply to: 1156836710 In reply to: 1156836710 In reply to: 1156836710 In reply to: 1156836710 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:37 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
minimum is one, so change to one here, but the steps below will remain the same In reply to: 1156839720 In reply to: 1156839720 In reply to: 1156839720 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:137 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
find the package In reply to: 1156847059 In reply to: 1156847059 In reply to: 1156847059 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:281 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
one :) In reply to: 1156847267 In reply to: 1156847267 In reply to: 1156847267 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:285 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
link to "https://docs.microsoft.com/azure/active-directory/develop/access-tokens#validating-tokens" In reply to: 1156847970 In reply to: 1156847970 In reply to: 1156847970 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:297 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
In reply to: 1156850339 In reply to: 1156850339 In reply to: 1156850339 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:305 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
add check here that if there are only delegated permissions , this check is used and so on.. In reply to: 1156850993 In reply to: 1156850993 In reply to: 1156850993 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:309 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
Change the ToDoListController such that it works w/o any change for both delegated and app only scenarios. In reply to: 1156851866 In reply to: 1156851866 In reply to: 1156851866 Refers to: 4-WebApp-your-API/4-1-MyOrg/README.md:299 in 55e3a1c. [](commit_id = 55e3a1c, deletion_comment = False) |
4-WebApp-your-API/4-1-MyOrg/TodoListService/Controllers/TodoListController.cs
Outdated
Show resolved
Hide resolved
4-WebApp-your-API/4-1-MyOrg/TodoListService/Controllers/TodoListController.cs
Show resolved
Hide resolved
4-WebApp-your-API/4-1-MyOrg/TodoListService/Controllers/TodoListController.cs
Outdated
Show resolved
Hide resolved
4-WebApp-your-API/4-1-MyOrg/TodoListService/Controllers/TodoListController.cs
Outdated
Show resolved
Hide resolved
*/ | ||
|
||
// TodoListScope is the scope of the Web API you want to call. This is usually in the form of: "api://fc3ef71c-43ab-497d-89f0-332787e09c7c/ToDoList.Read", | ||
// - a list of scopes for a V2 application (for instance "api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.Read api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.Write") |
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.
// - a list of scopes for a V2 application (for instance "api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.Read api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.Write") | |
// - a list of scopes for a V2 application (for instance "api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.Read api://b3682cc7-8b30-4bd2-aaba-080c6bf0fd31/ToDoList.ReadWrite") |
…hub.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2 into aremo-ms/Task-1924572-4-1-basher-update
…hub.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2 into aremo-ms/Task-1924572-4-1-basher-update
Added App Permissions and enhanced new attribute that combines Roles and Scopes altogether.
ToDoList Service Api was refactored for best practices
Readme file was updated as well.