Skip to content

Conversation

Roopan-Microsoft
Copy link
Collaborator

Purpose

  • This pull request introduces changes to the frontend server and client-side JavaScript to support an optional authentication feature. The key changes include adding a new environment variable for authentication, updating the configuration endpoint, and modifying client-side scripts to handle the new authentication setting.

Changes to support optional authentication:

  • src/frontend/frontend_server.py: Added the AUTH_ENABLED environment variable to the configuration endpoint, allowing the frontend to know if authentication is enabled.

Client-side handling of authentication:

  • src/frontend/wwwroot/app.js: Updated to store the AUTH_ENABLED value and use it to determine whether to use mock user data when authentication is disabled. [1] [2]
  • src/frontend/wwwroot/utils.js: Modified the GetAuthDetails function to check the authEnabled setting and return mock user headers if authentication is disabled. Added a helper function getMockUserHeaders to generate these headers. [1] [2] [3]

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Harsh-Microsoft and others added 2 commits March 25, 2025 18:27
@Roopan-Microsoft Roopan-Microsoft merged commit 1d056a6 into main Apr 3, 2025
12 checks passed
@github-actions
Copy link

github-actions bot commented May 2, 2025

🎉 This PR is included in version 1.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
feat: merging dev changes to main branch
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.

3 participants