-
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
Merged
kalyankrishna1
merged 69 commits into
master
from
aremo-ms/Task-1924572-4-1-basher-update
Aug 26, 2022
Merged
Changes from 9 commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
2599ef8
updated readme and ps1 with new wording
aremo-ms 7289a0d
Added BASHER logic to Service controller and made some refactoring
aremo-ms 3b794e6
update
aremo-ms fd8ba1a
changed to getting Owner as Object Id
aremo-ms 55e3a1c
fixed bug for readme and added app permissions
aremo-ms 6f2465f
some review comments were addressed
aremo-ms 6a8c470
updated README and configure.ps1
aremo-ms 06cce15
replaced Newtonsoft by JsonSerializer
aremo-ms 56ac49c
addressed most of comments. The last thing to address is token valida…
aremo-ms a57b73e
comments addressed
aremo-ms cd33fce
Added tenant validation on Service
aremo-ms a2bd757
Kalyan's edits to polish the code and readme
kalyankrishna1 dff53f9
merge fixed
kalyankrishna1 148968e
extended token validation code
kalyankrishna1 8a9f3bc
extended token validation code discussed
kalyankrishna1 516b438
secrets accidently added secrets removed
kalyankrishna1 bab24f2
removed a bunch of redundant files
kalyankrishna1 6f8dc6c
empty but shows changed !
kalyankrishna1 f866d5a
added optional claim
aremo-ms 727f74f
processing idtyp claim
aremo-ms 0242a69
updated readme with latest updates
aremo-ms 2e3d9e4
added an ignored folder
6f2ec1d
local branch merged
4180373
done and approved for BASHER
e6760af
enabled logging by default with disclaimers
7df60d9
added more missing material
1b56aa7
highlighted using certificates
5d4d4b6
update for certificate, updated link
aremo-ms 831f51f
merge from master
aremo-ms 5c5188c
Configuration scripts generated with updated sample.json and Code Gen…
aremo-ms 21a6f1c
updated appsettings.json with instruction for local and keyvault ceri…
aremo-ms a5a8507
updated configuration.ps1 and appsettings.json
aremo-ms ed95a14
updated appsettings.json
aremo-ms 32b9827
updated sample with latest Readme
aremo-ms a273a6b
Added instructions to use local certificate
aremo-ms ba2bfb4
reset appsettings.json
aremo-ms 40c86aa
nit
aremo-ms ab93e0f
nit
aremo-ms de8a786
added security warning to appsetting.json
aremo-ms 49a3a1c
reset appsettings.json
aremo-ms 180aad8
removed data
aremo-ms 3b6d228
removed UseNewSetup flag as redundant
aremo-ms 949d23b
updated ceritifcate redme name
aremo-ms fa8bbe0
updated comments on appsettings.json
aremo-ms fde05ea
Readme with updated warning about using client secrets
aremo-ms d8a4208
updated readme with correct certificate readme file path
aremo-ms cdb0b62
updated scopes comment
aremo-ms afbd2e8
removed redundant package
aremo-ms afc68bb
updated certificate part to be less confusiong
aremo-ms 4c7ad15
Compilation issues and some formatting addressed
cd752a1
More fixes
ab5a60c
updated the certs steps a bit
49abddc
updated the referenced code
164a302
reverting certificates CsharpConfigurations support, Automated Deply …
aremo-ms 9fa8f01
merge
aremo-ms 5a18734
typo fixed
2f8e0f4
minor working updates
aremo-ms 67ee023
minor fix
957dbf8
Merge branch 'aremo-ms/Task-1924572-4-1-basher-update' of https://git…
f1c4601
removed PasswordCredentials key
aremo-ms 68caf90
another round of minor edits
da47245
Updated readme file with deloyment steps
aremo-ms a74abf9
Merge branch 'aremo-ms/Task-1924572-4-1-basher-update' of https://git…
aremo-ms dd39fd5
Minor last minute edits
5f0986c
Minor edits and deployment related updates
49476ae
merge conflicts resolved
0f7e06b
renamed Delete to DeleteItem (action didn't work for Delete for some …
aremo-ms f9599f4
and this is done ..
e03d1a7
Merge branch 'master' into aremo-ms/Task-1924572-4-1-basher-update
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -94,8 +94,19 @@ | |
"SampleSubPath": "4-WebApp-Your-API\\4-1-MyOrg", | ||
"ProjectDirectory": "\\TodoListService" | ||
}, | ||
"AppScopes": [ "ToDoList.Read", "ToDoList.Write" ] | ||
|
||
"Scopes": [ "ToDoList.Read", "ToDoList.ReadWrite" ], | ||
"AppRoles": [ | ||
jmprieur marked this conversation as resolved.
Show resolved
Hide resolved
|
||
{ | ||
"Types": [ "Application" ], | ||
"Name": "ToDoList.Read.All", | ||
"Description": "Allow application to read all ToDo list items" | ||
}, | ||
{ | ||
"Types": [ "Application" ], | ||
"Name": "ToDoList.ReadWrite.All", | ||
"Description": "Allow application to read and write into ToDo list" | ||
} | ||
] | ||
}, | ||
{ | ||
"Id": "client", | ||
|
@@ -109,11 +120,13 @@ | |
"RequiredResourcesAccess": [ | ||
{ | ||
"Resource": "service", | ||
"DelegatedPermissions": [ "ToDoList.Read", "ToDoList.Write" ] | ||
"DelegatedPermissions": [ "ToDoList.Read", "ToDoList.ReadWrite" ] | ||
} | ||
], | ||
"Certificate": "the certificate will be named by application name", | ||
"ManualSteps": [], | ||
"EnableAccessTokenIssuance": "true", | ||
"EnableIdTokenIssuance": "false", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. remove these #Closed There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't see it in my sample.json |
||
"Sample": { | ||
"SampleSubPath": "4-WebApp-Your-API\\4-1-MyOrg", | ||
"ProjectDirectory": "\\Client" | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.