Skip to content

Fix Case Sensitivity Inconsistency in API Key References - #471

Open
Salman-Ahamed wants to merge 1 commit into
devfrom
fix/vault-builder-api-key-case
Open

Fix Case Sensitivity Inconsistency in API Key References#471
Salman-Ahamed wants to merge 1 commit into
devfrom
fix/vault-builder-api-key-case

Conversation

@Salman-Ahamed

@Salman-Ahamed Salman-Ahamed commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

🎯 What’s this PR about?

Resolved the case sensitivity inconsistency between Vault page and Builder components when handling API key references. Previously, Builder treated key references as case-sensitive while Vault allowed case variations, causing failed key lookups and confusing error messages when users referenced keys with different casing (e.g., 'Hello' vs 'hello').

###✅ Changes Included

  • Updated key name comparison logic to be case-insensitive by converting both key names and vault entry names to lowercase
  • Fixed key lookup failures for case mismatches between Vault and Builder
  • Eliminated false'missing key' notifications when keys exist with different casing
  • Prevented uniqueness errors when attempting to create keys that already exist with different case

📎 Related ClickUp Ticket

ClickUp: https://app.clickup.com/t/86euj8yx9


💻 Demo (optional)


✅ Checklist

  • Self-reviewed the code
  • Linked the correct ClickUp ticket
  • Tested locally (MANDATORY)
  • Marked as Draft if not ready for review

- Updated the key name comparison logic to be case-insensitive by converting both the key name and vault entry names to lowercase. This ensures more reliable matching when checking for existing keys.

@Salman-Ahamed Salman-Ahamed left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

reviewed

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