playa-ru / keycloak-russian-providers

Popular russian mail (Yandex, Mail.ru) and social networks (VK, OK) identity providers for Keycloak Identity and Access Management server.
Apache License 2.0
142 stars 35 forks source link

vkid fix #46

Closed Spliterash closed 1 week ago

Spliterash commented 2 weeks ago

Попытка выполнить вход через vkid уводит keycloak в 502 при обратном редиректе при включённой обязательной почте. Метод который достаёт почту не учитывает то, что весь json лежит внутри user

Spliterash commented 2 weeks ago

Кстати, раз такой повод, хотел спросить, зачем выкидывать локализованное сообщение об необходимости почты, если единственное место где одно будет показано, это логи keycloak ? Может быть вы перепутали нужный эксепшн с illegal, тк я не очень глубоко разбираюсь в keycloak Я говорю вот про эту строчку

throw new IllegalArgumentException(Utils.toEmailErrorMessage("VK"));

onotoliy commented 1 week ago

Добрый день. Спасибо за ваш PR. Принимать его не стал, но правки ваши внес (сделал немного по другому). По поводу вопроса по обязательности почты, тоже поправил. Актуальная версия 25.0.2.rsp-2