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

Disable local/basic accounts #4219

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tamirkamara
Copy link
Collaborator

@tamirkamara tamirkamara commented Dec 23, 2024

What is being addressed

Some policies keep changing less secure settings like local user access / deploy. TRE doesn't (shouldn't) use these options but it creates noise on Terraform runs.

How is this addressed

  • Disable storage account local user account
  • Disable basic auth on web apps

Base automatically changed from tamirkamara/4216-update-admin-vm to main December 25, 2024 09:50
@tamirkamara tamirkamara changed the title Disable local account and storage key access Disable local/basic accounts Dec 27, 2024
@tamirkamara tamirkamara force-pushed the tamirkamara/4218-disable-local-users branch from 4a0fa6c to 2437167 Compare December 27, 2024 09:32
Copy link

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 2437167.

@tamirkamara tamirkamara marked this pull request as ready for review December 27, 2024 12:45
@tamirkamara
Copy link
Collaborator Author

/test-extended

Copy link

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/12516228438 (with refid c818ff73)

(in response to this comment from @tamirkamara)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant