Skip to content

Conversation

@Nayana-R-Gowda
Copy link
Collaborator

  • Including this password policy section under User Management
  • password policy
  • Adding this password policy under User Management

💡 Fix Description

How did you solve it? Key design points.
Adding this password policy under User Management

🧪 Verification

Check Command Status
Lint suite make lint pass
Unit tests make test pass

📐 MCP Compliance (if relevant)

  • Matches current MCP spec
  • No breaking change to MCP clients

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • No secrets/credentials committed

NAYANAR0502 and others added 5 commits December 12, 2025 22:39
Signed-off-by: NAYANAR <nayana.r5@ibm.com>
Signed-off-by: NAYANAR <nayana.r5@ibm.com>
Signed-off-by: NAYANAR <nayana.r5@ibm.com>
Signed-off-by: NAYANAR <nayana.r5@ibm.com>
- Add Font Awesome to download-cdn-assets.sh for airgap deployments
- Update admin.html to conditionally load Font Awesome based on ui_airgapped
- Update login.html to conditionally load Font Awesome based on ui_airgapped
- Update change-password-required.html:
  - Add airgap support for Tailwind CSS and Font Awesome
  - Use dynamic password policy settings from server config
  - Fix JavaScript validation to respect policy settings
  - Add missing ui_airgapped context variable to template

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
@crivetimihai crivetimihai force-pushed the user-mgmt-password-policy branch from ea7a5d5 to 3c1614c Compare December 12, 2025 23:04
Copy link
Member

@crivetimihai crivetimihai left a comment

Choose a reason for hiding this comment

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

Rebased, fix: airgap-safe Font Awesome and dynamic password policy, stylesheet

@crivetimihai crivetimihai merged commit b721ed7 into main Dec 12, 2025
52 checks passed
@crivetimihai crivetimihai deleted the user-mgmt-password-policy branch December 12, 2025 23:09
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.

4 participants