Skip to content

Commit 901a1be

Browse files
author
Jorge
committed
fixed: removed double encoding in OAuth2Provider
1 parent 088bea1 commit 901a1be

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

ChangeLog

-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
master
22
- Fixed example for view customization docs (thanks @yankov)
3-
- Fixed: scope values were not being url encoded in OAuth2Provider
43
- Enable retrieving email address from LinkedIn (thanks @targeter)
54
- Added strings to securesocial.conf in demos (thanks @alanrodas)
65
- Added idLengthInBytes property to customize the length of session ids (based on changes by @mkubala)

module-code/app/securesocial/core/OAuth2Provider.scala

+1-1
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ abstract class OAuth2Provider(application: Application, jsonResponse: Boolean =
126126
(OAuth2Constants.RedirectUri, RoutesHelper.authenticate(id).absoluteURL(IdentityProvider.sslEnabled)),
127127
(OAuth2Constants.ResponseType, OAuth2Constants.Code),
128128
(OAuth2Constants.State, state))
129-
settings.scope.foreach( s => { params = (OAuth2Constants.Scope, URLEncoder.encode(s, "UTF-8")) :: params })
129+
settings.scope.foreach( s => { params = (OAuth2Constants.Scope, s) :: params })
130130
val url = settings.authorizationUrl +
131131
params.map( p => p._1 + "=" + URLEncoder.encode(p._2, "UTF-8")).mkString("?", "&", "")
132132
if ( Logger.isDebugEnabled ) {

0 commit comments

Comments
 (0)