Skip to content

Add Kommo Socialite Provider#1378

Merged
atymic merged 3 commits intoSocialiteProviders:masterfrom
rafgirao:master
Aug 29, 2025
Merged

Add Kommo Socialite Provider#1378
atymic merged 3 commits intoSocialiteProviders:masterfrom
rafgirao:master

Conversation

@rafgirao
Copy link
Copy Markdown
Contributor

No description provided.

}

// Fall back to config
return $this->getConfig('subdomain', 'yayforms');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dont hardcode this value

protected function getSubdomain()
{
// Try to get subdomain from request parameters first
$subdomain = $this->request->get('subdomain');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this secure?

@rafgirao
Copy link
Copy Markdown
Contributor Author

Thanks, @atymic — I’ve pushed updates addressing your comments.

@atymic atymic merged commit 4789031 into SocialiteProviders:master Aug 29, 2025
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.

2 participants