Skip to content

Redis key vault auth and 3 bug fixes#767

Draft
Xeelee33 wants to merge 4 commits intomicrosoft:Developmentfrom
Xeelee33:feature/redis_keyvault_auth
Draft

Redis key vault auth and 3 bug fixes#767
Xeelee33 wants to merge 4 commits intomicrosoft:Developmentfrom
Xeelee33:feature/redis_keyvault_auth

Conversation

@Xeelee33
Copy link
Contributor

@Xeelee33 Xeelee33 commented Mar 4, 2026

This pull requests adds 1 new feature and 3 bug fixes.

Feature - adds key vault secret authentication option for Redis
Commit: 3368d6e
Since authenticating to Redis via managed identity is not supported in Azure Gov (Confirmed in issue #696), added a Key Vault authentication option so Gov customers can utilize Redis session caching without storing plain text access keys in Cosmos. It builds off the existing Key Vault settings in the Security tab in Admin Settings. Unlike for agents, the secret name for Redis doesn't have any specific format restrictions.

image

Bug fix 1 - Resolved visible text being cut off and no horizontal scroll bars in blocks in message responses
Commit: 9823912
Resolved issue described in issue #719. Confirmed that all drop downs and expansion windows like message metadata details still function properly.
image

Bug fix 2 - Resolved invisible icons in New Prompt editor
Commit: e4157b6
Icons were missing in Create New Prompt editor in both light and dark mode due to the SimpleMDE toolbar in the editor using Front Awesome icons that don't exist in the project. Repointed the icons to the equivalent Bootstrap icons and made necessary adjustments for dark theme.

Before
image
image

After
image
image

Bugfix 3 - Resolved conversation details not loading for legacy conversations
Commit: 5c48426
The chat_type property being called in the conversation metadata query doesn't exist in legacy conversations, so clicking on the conversation details button on legacy conversations throws an error:
image

This commit just adds a default value of 'personal' to chat_type when it's called but returns no value. This same logic was previously added to the conversation search function (code snippet below is from current main branch):

for c in conversations:
# Default to 'personal' if chat_type is not defined (legacy conversations)
chat_type = c.get('chat_type', 'personal')
if chat_type in chat_types:
filtered_in.append(c)
else:
filtered_out.append(c)

After the fix, the conversation details window populates properly for legacy conversations:
image

…sation details load properly for legacy conversations
…s in overflowing message responses to function properly
…conflict, resolved for both light and dark mode themes.
@Xeelee33 Xeelee33 changed the title Feature/redis keyvault auth Redis key vault auth and 3 bug fixes Mar 4, 2026
@paullizer
Copy link
Contributor

Great additions and bug fixes!

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.

2 participants