fix: Stale api keys no longer display on the page once revoked

[Vchen7629]

Summary

Fixes #1877 , Updates the opensearch query to only fetch non revoked (revoked = False) api keys so the frontend recieves only non-revoked keys to display.

Demo (Before)

screen-capture.6.webm

Demo (After)

screen-capture.7.webm

Changes

• updates api_key_service.py list_keys opensearch query to also filter for keys that have revoked set to false
• Added a unit test to verify that query is sent correctly

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 49fb0ffd-ad51-49c4-b2c1-24be78f02ff5

📥 Commits

Reviewing files that changed from the base of the PR and between 89e0400 and 27dc424.

📒 Files selected for processing (2)

• src/services/api_key_service.py
• tests/unit/test_api_key_service.py

 _________________________________
< My bunny whiskers are tingling. >
 ---------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch stale-api-key-display-fix

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[richardz403]

LGTM.

Ran code locally, expired / revoked API keys no longer show up.

Ran tests locally, not issues found.

[coderabbitai]

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details

{}

[coderabbitai]

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details

{}