Skip to content

Conversation

Copy link

Copilot AI commented Oct 30, 2025

Users cannot customize the OIDC provider icon in Sentry's authentication selection UI because the provider name was dynamically set to the ISSUER value, making the CSS class unpredictable.

Changes

Configuration

  • Added OIDC_PROVIDER_NAME setting (defaults to "OIDC")
  • Decoupled provider display name from ISSUER (now used only for token validation)

Provider

  • Changed OIDCProvider.name from ISSUER to PROVIDER_NAME constant
  • Generates predictable CSS class: OIDC_PROVIDER_NAME = "My Company SSO".provider-logo.my-company-sso

Documentation & Example

  • Added icon usage guide with three implementation options
  • Included example SVG icon with user + lock symbol
  • Documented CSS class generation rules

Usage

Configure the provider name in sentry.conf.py:

OIDC_PROVIDER_NAME = "Acme Corp SSO"

Add custom CSS to Sentry:

.provider-logo.acme-corp-sso {
    background-image: url('/static/acme-icon.svg');
}

Fixes #19

Original prompt

This section details on the original issue you should resolve

<issue_title>custom icon </issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Add custom icon feature to application Add configurable provider name for custom icon support Oct 30, 2025
Copilot AI requested a review from max-wittig October 30, 2025 18:55
Copilot finished work on behalf of max-wittig October 30, 2025 18:55
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.

custom icon

3 participants