fix: potential race condition

[cb80]

Summary

Successfully fixed the concurrent map access race condition in the OIDC handler by adding proper synchronization with sync.RWMutex.

Changes Made

File: internal/handler/oidc.go

1. Added sync import (line 9)

2. Added mutex field to OIDC struct (line 32):

   providersMu sync.RWMutex // protects staticProviders

3. Protected write operations in RegisterStaticProvider (lines 133-134):

   handler.providersMu.Lock()
   defer handler.providersMu.Unlock()

4. Protected read operations in ProviderFor (lines 280, 287, 292, 295):

   • Added RLock() at the beginning of map access
   • Added RUnlock() before each return that reads from the map
   • Added final RUnlock() before falling through to session manager

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3085ca2-cc2f-411c-8b3a-b0e88af3301a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.}