Skip to content

Possible Fix for #4541 - #4543

Closed
Samik Roy (samikroy) wants to merge 3 commits into
Azure:masterfrom
samikroy:patch-19
Closed

Possible Fix for #4541#4543
Samik Roy (samikroy) wants to merge 3 commits into
Azure:masterfrom
samikroy:patch-19

Conversation

@samikroy

@samikroy Samik Roy (samikroy) commented Mar 31, 2022

Copy link
Copy Markdown
Contributor

Possible Fix for #4541

Reset-AADPassword playbook not working with default OOB template

Required items, please complete

  • Update PS script
  • Updated the logic app / playbook definition.

@samikroy
Possible Fix for Azure#4541
33b9487 
Possible Fix for Azure#4541 

Reset-AADPassword playbook not working with default OOB template
@v-spadarthi v-spadarthi added the Playbook Playbook specialty review needed label Mar 31, 2022
@samikroy

Samik Roy (samikroy) commented Apr 4, 2022

Copy link
Copy Markdown
Contributor Author

Anki Narravula (@anki-narravula) & v-spadarthi - Please let me know for further changes to this.

@samikroy

Samik Roy (samikroy) commented Apr 12, 2022

Copy link
Copy Markdown
Contributor Author

Lior Tamir (@lior-tamir) & Benjamin Kovacevic (@BenjiSec) - Could you please help merging this !

@BenjiSec

Benjamin Kovacevic (BenjiSec) commented Apr 14, 2022

Copy link
Copy Markdown
Contributor

Samik Roy (@samikroy) - I'll have to change few things and will make new PR. Need to add control if user doesn't have a manager and will change connection for checking manager to HTTP. Also this is template so need to change template versioning as well as add release notes.

@Yaniv-Shasha

Yaniv Shasha (Yaniv-Shasha) commented Apr 15, 2022

Copy link
Copy Markdown
Contributor

template error

@samikroy

Samik Roy (samikroy) commented Apr 20, 2022

Copy link
Copy Markdown
Contributor Author

Samik Roy (@samikroy) - I'll have to change few things and will make new PR. Need to add control if user doesn't have a manager and will change connection for checking manager to HTTP. Also this is template so need to change template versioning as well as add release notes.

Benjamin Kovacevic (@BenjiSec) - Thank you for sharing this.

@jaocabete jao (jaocabete) mentioned this pull request Dec 6, 2023
Open
@muyenzo muyenzo mentioned this pull request Dec 6, 2023
Open
@udanashivm udanashivm mentioned this pull request Dec 6, 2023
Open
@Croat79 Croat79 mentioned this pull request Dec 7, 2023
Open
@riddopic Stefano Harding (riddopic) mentioned this pull request Dec 7, 2023
Open
@bkizer-tenable Blake Kizer (bkizer-tenable) mentioned this pull request Dec 7, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Yaniv-Shasha Yaniv Shasha (Yaniv-Shasha) Awaiting requested review from Yaniv-Shasha
@sarah-yo Sarah Young (sarah-yo) Awaiting requested review from sarah-yo
@sreedharande Sreedhar Ande (sreedharande) Awaiting requested review from sreedharande
@lior-tamir Lior Tamir (lior-tamir) Awaiting requested review from lior-tamir

Assignees

@anki-narravula Anki Narravula (anki-narravula)

Labels

Playbook Playbook specialty review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@samikroy @BenjiSec @Yaniv-Shasha @anki-narravula @v-spadarthi