8000 Fix/decryption error invalid api key by Mpasha17 · Pull Request #684 · airweave-ai/airweave · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix/decryption error invalid api key #684

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Mpasha17
Copy link
@Mpasha17 Mpasha17 commented May 31, 2025

fx: Handle invalid API keys gracefully in read_api_keys endpoint

Previously, when encountering invalid/corrupted API keys, the endpoint would fail to return any keys. This fix improves error handling by:

  • Skipping invalid keys instead of failing
  • Returning an empty list when all keys are invalid
  • Adding better logging for debugging invalid keys
  • now if you reload the page there will be no error ,u can see all the keys

Before
Screenshot 2025-05-31 at 5 07 49 PM
After the fix
Screenshot 2025-05-31 at 5 08 42 PM

Fixes #682


Summary by cubic

Fixed the API keys endpoint to skip invalid or corrupted keys instead of failing, so users no longer see errors when loading their keys.

  • Bug Fixes
    • Skips invalid API keys and returns only valid ones.
    • Returns an empty list if all keys are invalid.
    • Adds logging for skipped keys to help with debugging.

Copy link
@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic reviewed 1 file and found no issues. Review PR in cubic.dev.

@ai-naymul
Copy link

@Mpasha17 Nice works. I already created a pr ref: #683

There are some of the field are not available in the api key data.

@orhanrauf

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.

Sometimes decryptions failed due to some of the api
2 participants
0