Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix config for backward compatibility #11

Merged
merged 1 commit into from
May 17, 2017
Merged

Fix config for backward compatibility #11

merged 1 commit into from
May 17, 2017

Conversation

iwex
Copy link
Contributor

@iwex iwex commented May 17, 2017

No description provided.

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

@jhaoda forgot to add this for backward compat.

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

@iwex смысл?

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

@jhaoda кто-то обновит либу, а у него нет в конфиге 'client_public'. Или как вариант сменить версию до 4.0

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

@iwex в каком конфиге? Ты про services.php и фолбэк на .env? Про версию не понял.

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

@jhaoda да, я добавил новую строчку в services.php. Если кто-то сейчас обновит либу, и у них не будет этой строчки, то соцлогин не сработает

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

@iwex ты проверял, что не сработает? socialite не требует наличия параметров в services.php, он способен самостоятельно брать их из .env

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

@jhaoda так а откуда он узнает, что $this->getConfig('client_public') === env('ODNOKLASSNIKI_PUBLIC') ?

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

@iwex он будет искать env('ODNOKLASSNIKI_CLIENT_PUBLIC'), емнип.

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

И как он его там найдет? ODNOKLASSNIKI_CLIENT_PUBLIC != ODNOKLASSNIKI_PUBLIC + https://github.com/laravel/framework/blob/5.4/src/Illuminate/Foundation/Bootstrap/LoadEnvironmentVariables.php#L20

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

@iwex я спросил — ты проверял, что не сработает?

@iwex
Copy link
Contributor Author

iwex commented May 17, 2017

@jhaoda да

@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

А, это в доках надо указать, что бы ключ изменили, тогда найдёт — но это не вариант, требует повышения мажорной версии.

@jhaoda jhaoda merged commit a0db580 into jhaoda:master May 17, 2017
@jhaoda
Copy link
Owner

jhaoda commented May 17, 2017

Спасибо.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants