Skip to content

Potential fix for 1 code quality finding - #731

Closed
kojiwakayama wants to merge 1 commit into
mainfrom
ai-findings-autofix/src-server-handlers-request-api-project-discovery.ts
Closed

kojiwakayama wants to merge 1 commit into
mainfrom
ai-findings-autofix/src-server-handlers-request-api-project-discovery.ts

Conversation

@kojiwakayama

Copy link
Copy Markdown
Contributor

This PR applies 1/2 suggestions from code quality AI findings. 1 suggestion was skipped to avoid creating conflicts.

…ery.ts from Copilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Thanks for your contribution, @web-flow!

Before we can merge this PR, we need you to accept our Contributor License Agreement (CLA).

Missing CLA entries for: @web-flow.

How to sign: Add contributor entry lines in CONTRIBUTORS.md in this PR:

Your Name (@web-flow)

By adding your name, you agree to the CLA. This is a one-time step.

If you are contributing on behalf of a company, please see our Corporate CLA.

if (!cacheCompletedDiscovery) {
discoveredProjects.delete(key);
const current = discoveredProjects.get(key);
if (current === promise) {
kojiwakayama added a commit that referenced this pull request Mar 25, 2026
- Fix JWKS cache race condition in proxy handler
- Add open redirect validation for auth return path
- Guard discovery cache deletion against concurrent overwrites
- Use shorthand property for projectUpdated param
@kojiwakayama

Copy link
Copy Markdown
Contributor Author

Consolidated in #740.

kojiwakayama added a commit that referenced this pull request Mar 25, 2026
- Fix JWKS cache race condition in proxy handler
- Add open redirect validation for auth return path
- Guard discovery cache deletion against concurrent overwrites
- Use shorthand property for projectUpdated param
@ariskemper
ariskemper deleted the ai-findings-autofix/src-server-handlers-request-api-project-discovery.ts branch May 13, 2026 10:04
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.

1 participant