libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

KEYCLOAK-15167 Retrieve email from Twitter IdP

Browse source
This commit is contained in:
Nathan Strobbe 2020-10-28 20:56:09 +01:00 committed by Marek Posolda
parent 0210acadad
commit 64717f650b
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
services/src/main/java/org/keycloak/social/twitter/TwitterIdentityProvider.java
View file

@ -47,6 +47,7 @@ import twitter4j.Twitter;
import twitter4j.TwitterFactory;
import twitter4j.auth.AccessToken;
import twitter4j.auth.RequestToken;
import twitter4j.conf.ConfigurationBuilder;
import javax.ws.rs.GET;
import javax.ws.rs.QueryParam;
@ -201,8 +202,7 @@ public class TwitterIdentityProvider extends AbstractIdentityProvider<OAuth2Iden
}
try (VaultStringSecret vaultStringSecret = session.vault().getStringSecret(getConfig().getClientSecret())) {
Twitter twitter = new TwitterFactory().getInstance();
Twitter twitter = new TwitterFactory(new ConfigurationBuilder().setIncludeEmailEnabled(true).build()).getInstance();
twitter.setOAuthConsumer(getConfig().getClientId(), vaultStringSecret.get().orElse(getConfig().getClientSecret()));
String twitterToken = authSession.getAuthNote(TWITTER_TOKEN);
@ -217,6 +217,7 @@ public class TwitterIdentityProvider extends AbstractIdentityProvider<OAuth2Iden
identity.setIdp(TwitterIdentityProvider.this);
identity.setUsername(twitterUser.getScreenName());
identity.setEmail(twitterUser.getEmail());
identity.setName(twitterUser.getName());

1
testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/broker/SocialLoginTest.java
View file

@ -369,7 +369,6 @@ public class SocialLoginTest extends AbstractKeycloakTest {
public void twitterLogin() {
setTestProvider(TWITTER);
performLogin();
assertUpdateProfile(false, false, true);
assertAccount();
}