Skip to content
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.

Sign up for GitHub

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

Jump to bottom

106893 - fix to avoid using plain text in powershell script #3907

Merged
merged 1 commit into from
Oct 7, 2024
Merged

106893 - fix to avoid using plain text in powershell script #3907

merged 1 commit into from
Oct 7, 2024

Conversation

v-ajajvanu
Copy link
Contributor

@v-ajajvanu v-ajajvanu commented Jun 7, 2024 •
edited
Loading

Description

This PR fix the issue of getting psscriptanalyzer:Error: PSAvoidUsingConvertToSecureStringWithPlainText

Related issues

Addresses [issue AB#106893].

Testing

Describe how this change was tested.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@v-ajajvanu v-ajajvanu added Build Open source This change is only relevant to the OSS code or release. labels Jun 7, 2024
@v-ajajvanu v-ajajvanu added this to the S141 milestone Jun 7, 2024
@v-ajajvanu v-ajajvanu requested a review from a team as a code owner June 7, 2024 09:37
@v-ajajvanu v-ajajvanu changed the title User story 106893 - Removed -AsPlainText and made use of Secret manag… 106893 - fix to avoid using plain text in powershell script Jun 7, 2024
@v-ajajvanu v-ajajvanu requested a review from abiisnn June 13, 2024 06:05
@mikaelweave mikaelweave modified the milestones: S141, S142 Jun 18, 2024
@feordin
Copy link
Contributor

feordin commented Jul 26, 2024

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-aljagtap
Copy link
Contributor

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-aljagtap
Copy link
Contributor

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-aljagtap
Copy link
Contributor

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-aljagtap
Copy link
Contributor

/azp run

1 similar comment
@v-aljagtap
Copy link
Contributor

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-aljagtap
Copy link
Contributor

/azp run

1 similar comment
@v-aljagtap
Copy link
Contributor

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abiisnn
Copy link
Contributor

abiisnn commented Sep 25, 2024

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abiisnn
Copy link
Contributor

abiisnn commented Sep 25, 2024

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abiisnn abiisnn force-pushed the personal/v-ajaj/106893 branch from 94e7e08 to 3bddeb2 Compare October 7, 2024 21:40
abiisnn
abiisnn reviewed Oct 7, 2024
View reviewed changes
Copy link
Contributor

@abiisnn abiisnn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/azp rebase

@abiisnn abiisnn merged commit d59ab94 into main Oct 7, 2024
47 checks passed
@abiisnn abiisnn deleted the personal/v-ajaj/106893 branch October 7, 2024 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abiisnn abiisnn abiisnn approved these changes

@feordin feordin feordin approved these changes

Assignees
No one assigned
Labels
Build Open source This change is only relevant to the OSS code or release.
Projects
None yet
Milestone
S142
Development

Successfully merging this pull request may close these issues.
5 participants
@v-ajajvanu @feordin @v-aljagtap @abiisnn @mikaelweave