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.

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

Remove Basic Auth from Redeploy Ci Pipeline #3268

Merged
merged 18 commits into from
Apr 28, 2023

Conversation

PTaladay
Copy link
Collaborator

Description

Disabled Basic Auth when using the CI Deploy pipeline so we don't run a security risk.

Related issues

Addresses [issue #].

Testing

Redployed using script update and verified that Basic Auth was disabled.

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.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@PTaladay PTaladay added the Build label Apr 28, 2023
@PTaladay PTaladay added this to the S113 milestone Apr 28, 2023
@PTaladay PTaladay requested a review from a team as a code owner April 28, 2023 15:39
@PTaladay PTaladay enabled auto-merge (squash) April 28, 2023 16:03
@PTaladay PTaladay merged commit 1df358e into main Apr 28, 2023
@PTaladay PTaladay deleted the personal/patalada/FixRedeployPipeline branch April 28, 2023 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants