Fixed issues in scope validation feature

revert-70aa11f8
Milan Perera 8 years ago
parent de47812d6d
commit 41353f8014

@ -87,27 +87,16 @@
org.apache.commons.logging, org.apache.commons.logging,
org.osgi.service.component, org.osgi.service.component,
org.wso2.carbon.device.mgt.oauth.extensions.*, org.wso2.carbon.device.mgt.oauth.extensions.*,
org.wso2.carbon.identity.application.common.model, org.wso2.carbon.identity.*;version="${carbon.identity.framework.version.range}",
org.wso2.carbon.identity.oauth.callback,
org.wso2.carbon.identity.oauth2,
org.wso2.carbon.identity.oauth2.model,
org.wso2.carbon.identity.oauth2.validators,
org.wso2.carbon.user.api, org.wso2.carbon.user.api,
org.wso2.carbon.user.core.service, org.wso2.carbon.user.core.service,
org.wso2.carbon.user.core.tenant, org.wso2.carbon.user.core.tenant,
org.json.simple, org.json.simple,
javax.cache, javax.cache,
org.wso2.carbon.identity.core.util,
org.wso2.carbon.identity.oauth2.dto,
org.wso2.carbon.identity.oauth2.token,
org.wso2.carbon.utils, org.wso2.carbon.utils,
org.wso2.carbon.context, org.wso2.carbon.context,
org.wso2.carbon.identity.oauth.cache,
org.wso2.carbon.identity.oauth.config,
org.wso2.carbon.identity.oauth2.dao,
org.wso2.carbon.utils.multitenancy, org.wso2.carbon.utils.multitenancy,
org.apache.commons.codec.binary;version="${commons-codec.wso2.osgi.version.range}", org.apache.commons.codec.binary;version="${commons-codec.wso2.osgi.version.range}",
org.wso2.carbon.identity.application.authentication.framework.model,
org.wso2.carbon.base, org.wso2.carbon.base,
org.apache.commons.collections, org.apache.commons.collections,
org.apache.commons.lang, org.apache.commons.lang,
@ -116,15 +105,10 @@
org.opensaml.security, org.opensaml.security,
org.opensaml.xml.*, org.opensaml.xml.*,
org.w3c.dom, org.w3c.dom,
org.wso2.carbon.identity.application.common.util, org.wso2.carbon.idp.mgt;version="${carbon.identity.framework.version.range}",
org.wso2.carbon.identity.base,
org.wso2.carbon.identity.oauth2.token.handlers.grant.*,
org.wso2.carbon.identity.oauth2.util,
org.wso2.carbon.idp.mgt,
org.opensaml.common.xml, org.opensaml.common.xml,
org.wso2.carbon.identity.oauth.common,
org.opensaml, org.opensaml,
org.wso2.carbon.apimgt.keymgt.issuers org.wso2.carbon.apimgt.keymgt;version="${carbon.api.mgt.version.range}"
</Import-Package> </Import-Package>
</instructions> </instructions>
</configuration> </configuration>

@ -88,7 +88,7 @@ public class ScopeValidationHandler extends OAuth2ScopeValidator {
} }
// validate scope via relevant scope validator that matches with the prefix // validate scope via relevant scope validator that matches with the prefix
return scopeValidator.validateScope(accessTokenDO, resourceScope); return scopeValidator.validateScope(accessTokenDO, resource);
} }
private String getResourceScope(String resource) { private String getResourceScope(String resource) {

@ -38,7 +38,7 @@ import org.opensaml.xml.security.x509.X509Credential;
import org.opensaml.xml.signature.SignatureValidator; import org.opensaml.xml.signature.SignatureValidator;
import org.opensaml.xml.validation.ValidationException; import org.opensaml.xml.validation.ValidationException;
import org.w3c.dom.NodeList; import org.w3c.dom.NodeList;
import org.wso2.carbon.apimgt.keymgt.issuers.ScopesIssuingHandler; import org.wso2.carbon.apimgt.keymgt.ScopesIssuer;
import org.wso2.carbon.base.MultitenantConstants; import org.wso2.carbon.base.MultitenantConstants;
import org.wso2.carbon.identity.application.authentication.framework.model.AuthenticatedUser; import org.wso2.carbon.identity.application.authentication.framework.model.AuthenticatedUser;
import org.wso2.carbon.identity.application.common.model.FederatedAuthenticatorConfig; import org.wso2.carbon.identity.application.common.model.FederatedAuthenticatorConfig;
@ -101,7 +101,7 @@ public class ExtendedSAML2BearerGrantHandler extends AbstractAuthorizationGrantH
@Override @Override
public boolean validateScope(OAuthTokenReqMessageContext tokReqMsgCtx) { public boolean validateScope(OAuthTokenReqMessageContext tokReqMsgCtx) {
return ScopesIssuingHandler.getInstance().setScopes(tokReqMsgCtx); return ScopesIssuer.getInstance().setScopes(tokReqMsgCtx);
} }
/** /**

@ -1766,6 +1766,7 @@
<!-- Carbon Identity --> <!-- Carbon Identity -->
<carbon.identity.framework.version>5.7.0</carbon.identity.framework.version> <carbon.identity.framework.version>5.7.0</carbon.identity.framework.version>
<carbon.identity.framework.version.range>[5.0.0, 6.0.0)</carbon.identity.framework.version.range>
<identity.inbound.auth.oauth.version>5.3.1</identity.inbound.auth.oauth.version> <identity.inbound.auth.oauth.version>5.3.1</identity.inbound.auth.oauth.version>
<identity.inbound.auth.saml.version>5.3.0</identity.inbound.auth.saml.version> <identity.inbound.auth.saml.version>5.3.0</identity.inbound.auth.saml.version>
@ -1797,7 +1798,7 @@
<commons-lang.wso2.osgi.version.range>[2.6.0,3.0.0)</commons-lang.wso2.osgi.version.range> <commons-lang.wso2.osgi.version.range>[2.6.0,3.0.0)</commons-lang.wso2.osgi.version.range>
<!-- Carbon API Management --> <!-- Carbon API Management -->
<carbon.api.mgt.version>6.1.35</carbon.api.mgt.version> <carbon.api.mgt.version>6.1.64</carbon.api.mgt.version>
<carbon.api.mgt.version.range>(6.0.0,7.0.0]</carbon.api.mgt.version.range> <carbon.api.mgt.version.range>(6.0.0,7.0.0]</carbon.api.mgt.version.range>
<!-- Carbon Analytics Commons --> <!-- Carbon Analytics Commons -->

Loading…
Cancel
Save