litellm - ✅(Solved) Fix Typo in variable name: generic_role_mappoings_default_role [1 pull requests, 1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
BerriAI/litellm#25245Fetched 2026-04-08 03:02:31
View on GitHub
Comments
1
Participants
2
Timeline
3
Reactions
0
Author
Timeline (top)
cross-referenced ×2commented ×1

Fix Action

Fixed

PR fix notes

PR #25246: Fix typo: generic_role_mappoings_default_role → generic_role_mappings_default_role

Description (problem / solution / changelog)

Fixes #25245

Renames generic_role_mappoings_default_role to generic_role_mappings_default_role in _setup_role_mappings().

The env var GENERIC_ROLE_MAPPINGS_DEFAULT_ROLE was already correct, only the internal Python variable had the typo. No functional change.

Changed files

  • litellm/proxy/management_endpoints/ui_sso.py (modified, +2/-2)
RAW_BUFFERClick to expand / collapse

While configuring SSO role mappings with a generic OIDC provider, I noticed a typo in litellm/proxy/management_endpoints/ui_sso.py inside _setup_role_mappings().

The Python variable is named generic_role_mappoings_default_role (note: "mappoings") instead of generic_role_mappings_default_role.

The env var GENERIC_ROLE_MAPPINGS_DEFAULT_ROLE is spelled correctly so it doesn't break anything functionally, but the inconsistency could trip someone up reading the source.

Lines 715 and 734 in ui_sso.py.

extent analysis

TL;DR

The most likely fix is to correct the typo in the Python variable name generic_role_mappoings_default_role to generic_role_mappings_default_role in litellm/proxy/management_endpoints/ui_sso.py.

Guidance

  • Verify the typo by checking lines 715 and 734 in ui_sso.py to confirm the incorrect variable name.
  • Correct the typo by renaming the variable to generic_role_mappings_default_role to maintain consistency with the env var GENERIC_ROLE_MAPPINGS_DEFAULT_ROLE.
  • Review the surrounding code to ensure the corrected variable name is used consistently throughout the _setup_role_mappings() function.
  • Test the corrected code to ensure it functions as expected and does not introduce any new issues.

Example

# Before
generic_role_mappoings_default_role = ...

# After
generic_role_mappings_default_role = ...

Notes

This fix assumes that the typo is the only issue and that the corrected variable name is used correctly throughout the code. Additional testing may be necessary to ensure the fix does not introduce any new problems.

Recommendation

Apply workaround: Correct the typo in the variable name to maintain consistency and avoid potential issues. This fix is necessary to prevent confusion when reading the source code and to ensure the code functions as expected.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

litellm - ✅(Solved) Fix Typo in variable name: generic_role_mappoings_default_role [1 pull requests, 1 comments, 2 participants]