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

Secure BWA with Entra #33805

Merged
merged 8 commits into from
Oct 9, 2024
Merged

Secure BWA with Entra #33805

merged 8 commits into from
Oct 9, 2024

Conversation

Updates

Updates
@guardrex guardrex self-assigned this Oct 7, 2024
@guardrex
Copy link
Collaborator Author

guardrex commented Oct 8, 2024

UPDATED ...

The latest version of the remarks on security ...

Don't store app secrets, connection strings, credentials, passwords, personal identification numbers (PINs), private C#/.NET code, or private keys/tokens in client-side code, which is ***always insecure***. In test/staging and production environments, server-side Blazor code and web APIs should use secure authentication flows that avoid maintaining credentials within project code or configuration files. Outside of local development testing, we recommend avoiding the use of environment variables to store sensitive data, as environment variables aren't the most secure approach. For local development testing, the [Secret Manager tool](xref:security/app-secrets) is recommended for securing sensitive data.

I mirrored these updates on the Blazor ROPC PR, and I'm 👂 for further feedback on improving this guidance.

@guardrex
Copy link
Collaborator Author

guardrex commented Oct 9, 2024

@halter73 ... I think we're good to go here.

I'll review these articles again before GA.

I'm going get this in NOW 🏃‍♂️ ... you know ... before I die in a hurricane! 💀😨😆

@guardrex guardrex merged commit 7bd23df into main Oct 9, 2024
3 checks passed
@guardrex guardrex deleted the guardrex/blazor-bwa-with-entra branch October 9, 2024 16:18
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.

Add documentation for integrating Blazor Web Apps with the Microsoft Identity Platform
4 participants