Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Revert changes done for introducing a gateway proxy to infosec endpoints #25

Merged
merged 11 commits into from
Oct 28, 2024
Merged

Revert changes done for introducing a gateway proxy to infosec endpoints #25

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f36a63f
Revert changes done to introduce gateway proxy to infosec endpoints
imesh94 Oct 27, 2024
a3730a7
Add correct well known url
imesh94 Oct 27, 2024
9b7c851
Add newline
imesh94 Oct 27, 2024
a2dd7e2
Add correct par endpoint
imesh94 Oct 27, 2024
36d2d7d
Remove InfoSecDataPublishingHandler
imesh94 Oct 27, 2024
8038786
Add toml changes
imesh94 Oct 27, 2024
415bcb8
Fix not detenting infosec requests as high priority
imesh94 Oct 27, 2024
74f54a2
Update CDSCurrentPeakTPSApp.siddhi
imesh94 Oct 27, 2024
ca36673
Fix invalid priority for cdr arrangement revocation endpoint
imesh94 Oct 28, 2024
f5a8866
Merge remote-tracking branch 'origin/main' into fix/revert-proxy-change
imesh94 Oct 28, 2024
c20855a
Check HttpServletRequest before casting
imesh94 Oct 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
188 changes: 0 additions & 188 deletions ...src/main/java/org/wso2/openbanking/cds/gateway/handlers/InfoSecDataPublishingHandler.java
View file
Open in desktop

This file was deleted.

18 changes: 9 additions & 9 deletions ...ng.cds.gateway/src/main/java/org/wso2/openbanking/cds/gateway/utils/GatewayConstants.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,17 +91,17 @@ private GatewayConstants() {
public static final String WELL_KNOWN_API = "WellKnownAPI";
public static final String PAR_API = "PARAPI";

public static final String TOKEN_ENDPOINT = "/token";
public static final String AUTHORIZE_ENDPOINT = "/authorize";
public static final String JWKS_ENDPOINT = "/jwks";
public static final String USERINFO_ENDPOINT = "/userinfo";
public static final String REVOKE_ENDPOINT = "/revoke";
public static final String INTROSPECTION_ENDPOINT = "/token/introspect";
public static final String PAR_ENDPOINT = "/par";
public static final String WELL_KNOWN_ENDPOINT = "/.well-known/openid-configuration";
public static final String TOKEN_ENDPOINT = "/oauth2/token";
public static final String AUTHORIZE_ENDPOINT = "/oauth2/authorize";
public static final String JWKS_ENDPOINT = "/oauth2/jwks";
public static final String USERINFO_ENDPOINT = "/oauth2/userinfo";
public static final String REVOKE_ENDPOINT = "/oauth2/revoke";
public static final String INTROSPECTION_ENDPOINT = "/oauth2/introspect";
public static final String PAR_ENDPOINT = "/api/openbanking/push-authorization/par";
public static final String WELL_KNOWN_ENDPOINT = "/oauth2/token/.well-known/openid-configuration";
public static final String REGISTER_ENDPOINT = "/register";
public static final String REGISTER_CLIENT_ID_ENDPOINT = "/register/{ClientId}";
public static final String CDR_ARRANGEMENT_ENDPOINT = "/{cdrArrangementId}";
public static final String CDR_ARRANGEMENT_ENDPOINT = "/arrangements/1.0.0";
public static final String DISCOVERY_OUTAGES_ENDPOINT = "/discovery/outages";
public static final String DISCOVERY_STATUS_ENDPOINT = "/discovery/status";
public static final String PRODUCTS_ENDPOINT = "/banking/products";
Expand Down
138 changes: 0 additions & 138 deletions ...test/java/org/wso2/openbanking/cds/gateway/handlers/InfoSecDataPublishingHandlerTest.java
View file
Open in desktop

This file was deleted.

1 change: 0 additions & 1 deletion components/org.wso2.openbanking.cds.gateway/src/test/resources/testng.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
<class name="org.wso2.openbanking.cds.gateway.executors.core.CDSAPIRequestRouterTest"/>
<class name="org.wso2.openbanking.cds.gateway.throttling.CDSThrottleDataPublisherImplTest"/>
<class name="org.wso2.openbanking.cds.gateway.mediators.GatewayErrorMediatorTest"/>
<class name="org.wso2.openbanking.cds.gateway.handlers.InfoSecDataPublishingHandlerTest"/>
<class name="org.wso2.openbanking.cds.gateway.executors.reporting.CDSCommonDataReportingExecutorTest"/>
<class name="org.wso2.openbanking.cds.gateway.executors.header.validation.CDSHeaderValidationExecutorTest"/>
<class name="org.wso2.openbanking.cds.gateway.executors.jwt.authentication.JWTAuthenticationExecutorTest"/>
Expand Down
7 changes: 5 additions & 2 deletions ...y/src/main/java/org/wso2/openbanking/cds/identity/filter/InfoSecDataPublishingFilter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,10 @@ public void destroy() {
public boolean shouldPublishCurrentRequestData(ServletRequest request) {

// If the request is internal traffic, no need to publish data
return expectedExternalTrafficHeaderValue.equalsIgnoreCase(
((HttpServletRequest) request).getHeader(externalTrafficHeaderName));
if (request instanceof HttpServletRequest) {
return expectedExternalTrafficHeaderValue.equalsIgnoreCase(
((HttpServletRequest) request).getHeader(externalTrafficHeaderName));
}
return false;
}
}
Loading
Loading