Есть возражения против добавления параметра SSO IDP?

Я хотел бы добавить необязательный параметр в путь /session/sso, который контроллер будет добавлять к URL-адресу запроса к IDP. Цель — сообщить IDP, какой поток аутентификации инициировать, например, вход через социальные сети или локальный вход. Без подачи какого-либо сигнала IDP в рамках первоначального перенаправления Discourse SSO это сделать невозможно.

Я думаю о параметре с именем idp_param, например:
/session/sso?return_path=/&idp_param=fb

Есть ли какие-либо возражения против этого и против соответствующего pull request?

Причина в том, что я хочу создать большой баннер, который приглашает анонимных посетителей войти в систему с помощью одного из настроенных в нашем IDP внешних провайдеров социальных сетей, поскольку, на мой взгляд, единственная кнопка «Войти», предоставляемая Discourse, недостаточно привлекательна и информативна.

Текущий макет, где две кнопки социальных сетей добавляют idp_param в URL:

Другой вариант — сделать URL настройки SSO коллекцией «ключ-значение», чтобы можно было иметь разные URL провайдеров идентификации (IDP), на которые параметр мог бы указывать по ключу, например:

default: https://myidp.com/sso
fb: https://myidp.com/sso?provider=fb
google: https://myidp.com/sso?provider=google

/session/sso?return_path=/&sso_uri=fb

Однако это требует больше усилий для сценария с низким уровнем использования.