lmccay merged pull request #459:
URL: https://github.com/apache/knox/pull/459
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
xuzikun2003 opened a new pull request #459:
URL: https://github.com/apache/knox/pull/459
## What changes were proposed in this pull request?
Currently, Knox only supports the Content-Encoding "gzip" and the
Content-Encoding "deflate" is not supported. When the upstream server (such
lmccay commented on pull request #459:
URL: https://github.com/apache/knox/pull/459#issuecomment-864198747
@xuzikun2003 - it seems you have some checkstyle issues that need to be
resolved.
Please see https://travis-ci.com/github/apache/knox/jobs/515699869 for the
issues towards the
xuzikun2003 commented on pull request #459:
URL: https://github.com/apache/knox/pull/459#issuecomment-864207198
@lmccay , the style issue is already fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
smolnar82 merged pull request #458:
URL: https://github.com/apache/knox/pull/458
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #458:
URL: https://github.com/apache/knox/pull/458
## What changes were proposed in this pull request?
In some cases, it's better to return `401/UNAUTHORIZED` than
`400/BAD_REQUEST`.
## How was this patch tested?
Ran JUnit tests and
pzampino merged pull request #456:
URL: https://github.com/apache/knox/pull/456
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 merged pull request #457:
URL: https://github.com/apache/knox/pull/457
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #457:
URL: https://github.com/apache/knox/pull/457
## What changes were proposed in this pull request?
Populating and returning the appropriate signature algorithm in the response
of Knox's JWKS endpoint.
## How was this patch tested?
pzampino commented on a change in pull request #456:
URL: https://github.com/apache/knox/pull/456#discussion_r648643626
##
File path:
gateway-provider-ha/src/main/java/org/apache/knox/gateway/ha/dispatch/ConfigurableHADispatch.java
##
@@ -224,7 +224,7 @@ private void
moresandeep commented on a change in pull request #456:
URL: https://github.com/apache/knox/pull/456#discussion_r648517128
##
File path:
gateway-provider-ha/src/main/java/org/apache/knox/gateway/ha/dispatch/ConfigurableHADispatch.java
##
@@ -224,7 +224,7 @@ private void
lmccay commented on a change in pull request #456:
URL: https://github.com/apache/knox/pull/456#discussion_r648502609
##
File path:
gateway-provider-ha/src/main/java/org/apache/knox/gateway/ha/dispatch/ConfigurableHADispatch.java
##
@@ -224,7 +224,7 @@ private void
pzampino opened a new pull request #456:
URL: https://github.com/apache/knox/pull/456
…ig until session is established
## What changes were proposed in this pull request?
Modified ConfigurableHaDispatch behavior wrt to failover when sticky
sessions are enabled, but fallback
smolnar82 merged pull request #455:
URL: https://github.com/apache/knox/pull/455
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #455:
URL: https://github.com/apache/knox/pull/455
## What changes were proposed in this pull request?
Instead of logging the number of expired tokens removed from the DB at
`DEBUG` level, Knox logs the
## How was this patch tested?
smolnar82 merged pull request #454:
URL: https://github.com/apache/knox/pull/454
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #454:
URL: https://github.com/apache/knox/pull/454
## What changes were proposed in this pull request?
Improvements on Token Generation UI:
- more refined DIV elements
- copy-to-clipboard function is invoked when end-users click on `JWT
smolnar82 merged pull request #452:
URL: https://github.com/apache/knox/pull/452
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 removed a comment on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853625828
Thanks, @stoty for your comment. I did a smoke test in a local cluster with
all the service UIs available and found the following services need the
trailing slash: Oozie,
smolnar82 merged pull request #453:
URL: https://github.com/apache/knox/pull/453
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853625828
Thanks, @stoty for your comment. I did a smoke test in a local cluster with
all the service UIs available and found the following services need the
trailing slash: Oozie, Ranger,
smolnar82 merged pull request #452:
URL: https://github.com/apache/knox/pull/452
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 merged pull request #453:
URL: https://github.com/apache/knox/pull/453
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #453:
URL: https://github.com/apache/knox/pull/453
## What changes were proposed in this pull request?
Limiting the comment field length to 255 characters. I also added validation
on that field in case the limit does not take place for any reason.
smolnar82 removed a comment on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853625828
Thanks, @stoty for your comment. I did a smoke test in a local cluster with
all the service UIs available and found the following services need the
trailing slash: Oozie,
smolnar82 commented on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853625828
Thanks, @stoty for your comment. I did a smoke test in a local cluster with
all the service UIs available and found the following services need the
trailing slash: Oozie, Ranger,
stoty edited a comment on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853019011
When I tested, the UI pages for some services, like the Oozie UI, didn't
load without the trailing slash.
As it is kind of hopeless to test every single service, the most
stoty edited a comment on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853019011
When I tested, the UI pages for some services, like the Oozie UI, didn't
load without the trailing slash.
As it is kind of hopeless to test every single service, the most
stoty commented on pull request #452:
URL: https://github.com/apache/knox/pull/452#issuecomment-853019011
When I tested, the UI pages for some services, look the Oozie UI, didn't
load without the trailing slash.
As it is kind of hopeless to test every single service, the most
smolnar82 opened a new pull request #452:
URL: https://github.com/apache/knox/pull/452
## What changes were proposed in this pull request?
Extra trailing slash in the generated service context caused problems.
## How was this patch tested?
Updated JUnit tests and tested
moresandeep merged pull request #451:
URL: https://github.com/apache/knox/pull/451
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
moresandeep opened a new pull request #451:
URL: https://github.com/apache/knox/pull/451
## What changes were proposed in this pull request?
Adding 'kid' and 'jku' claims as part of JOSE Headers for tokens issued by
Knox
## How was this patch tested?
This patch was tested
smolnar82 merged pull request #450:
URL: https://github.com/apache/knox/pull/450
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #450:
URL: https://github.com/apache/knox/pull/450
## What changes were proposed in this pull request?
In order to avoid misunderstandings and customer escalations, the token
generation UI now indicates what is the configured maximum lifetime
smolnar82 merged pull request #449:
URL: https://github.com/apache/knox/pull/449
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 opened a new pull request #449:
URL: https://github.com/apache/knox/pull/449
## What changes were proposed in this pull request?
Added more refined validation on each token lifespan field to make sure we
do not allow:
- negative values
- floating numbers
-
smolnar82 merged pull request #448:
URL: https://github.com/apache/knox/pull/448
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633569862
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@
pzampino commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633550033
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -30,8
smolnar82 merged pull request #448:
URL: https://github.com/apache/knox/pull/448
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633646832
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@
pzampino commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633584944
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -30,8
pzampino commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633584139
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -30,8
smolnar82 commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633569862
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@
pzampino commented on a change in pull request #448:
URL: https://github.com/apache/knox/pull/448#discussion_r633550033
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -30,8
smolnar82 opened a new pull request #448:
URL: https://github.com/apache/knox/pull/448
## What changes were proposed in this pull request?
In order to avoid expensive MAC calculation at request verification time,
previous successful passcode token verifications are cached and
moresandeep merged pull request #423:
URL: https://github.com/apache/knox/pull/423
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 merged pull request #437:
URL: https://github.com/apache/knox/pull/437
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
pzampino commented on a change in pull request #437:
URL: https://github.com/apache/knox/pull/437#discussion_r630379211
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -141,9
smolnar82 merged pull request #447:
URL: https://github.com/apache/knox/pull/447
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on a change in pull request #447:
URL: https://github.com/apache/knox/pull/447#discussion_r626278725
##
File path:
gateway-server/src/main/java/org/apache/knox/gateway/services/token/impl/JDBCTokenStateService.java
##
@@ -193,8 +194,17 @@ protected void
moresandeep merged pull request #445:
URL: https://github.com/apache/knox/pull/445
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
moresandeep commented on a change in pull request #447:
URL: https://github.com/apache/knox/pull/447#discussion_r626109820
##
File path:
gateway-server/src/main/java/org/apache/knox/gateway/services/token/impl/JDBCTokenStateService.java
##
@@ -193,8 +194,17 @@ protected void
smolnar82 opened a new pull request #447:
URL: https://github.com/apache/knox/pull/447
## What changes were proposed in this pull request?
There is a new `KNOXTOKEN` service API to enable/disable a Knox-generated
token.
Instead of adding another column in KNOX_TABLES, a brand new
moresandeep merged pull request #446:
URL: https://github.com/apache/knox/pull/446
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r626000702
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -31,15
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625904268
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SSOCookieFederationFilter.java
##
@@
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625902703
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SSOCookieFederationFilter.java
##
@@
smolnar82 merged pull request #443:
URL: https://github.com/apache/knox/pull/443
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
vedantlodha opened a new pull request #446:
URL: https://github.com/apache/knox/pull/446
## What changes were proposed in this pull request?
Zeppelin uses JSESSIONID to identify a session. Zeppelin UI service
definition uses DefaultDispatch which prevents "SET-COOKIE" response
pzampino commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625827897
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -31,15
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625822042
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -186,4
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625821690
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -90,6
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625819010
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/JWTFederationFilter.java
##
@@ -31,15
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625815547
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/JWTMessages.java
##
@@ -59,8 +59,11 @@
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625814216
##
File path:
gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java
##
@@ -201,6 +246,86 @@
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625812360
##
File path:
gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java
##
@@ -201,6 +246,86 @@
moresandeep commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625805715
##
File path:
gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java
##
@@ -129,10 +152,27 @@
pzampino commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625797509
##
File path:
gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java
##
@@ -201,6 +246,86 @@
pzampino commented on a change in pull request #445:
URL: https://github.com/apache/knox/pull/445#discussion_r625756920
##
File path:
gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java
##
@@ -129,10 +152,27 @@
mudit-97 commented on a change in pull request #423:
URL: https://github.com/apache/knox/pull/423#discussion_r625633724
##
File path:
gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/ConfigurableDispatch.java
##
@@ -41,13 +47,26 @@
public class ConfigurableDispatch
moresandeep opened a new pull request #445:
URL: https://github.com/apache/knox/pull/445
## What changes were proposed in this pull request?
This change adds unauthenticated path list support for the following two
authentication providers (along with SSOCookieProvider)
-
smolnar82 commented on a change in pull request #443:
URL: https://github.com/apache/knox/pull/443#discussion_r625323200
##
File path:
gateway-applications/src/main/resources/applications/tokengen/app/js/tokengen.js
##
@@ -53,20 +53,23 @@ function b64DecodeUnicode(str) {
}
smolnar82 merged pull request #444:
URL: https://github.com/apache/knox/pull/444
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 merged pull request #441:
URL: https://github.com/apache/knox/pull/441
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
pzampino merged pull request #440:
URL: https://github.com/apache/knox/pull/440
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on pull request #444:
URL: https://github.com/apache/knox/pull/444#issuecomment-831204421
> It would be nice if we can validate JDBC url syntax and let users know
that the configured URL was wrong in the logs.
Thanks, @moresandeep for your review.
This is
smolnar82 opened a new pull request #444:
URL: https://github.com/apache/knox/pull/444
## What changes were proposed in this pull request?
Instead of declaring connection properties one-by-one in `gateway-site.xml`
a simple `connection URL` is enough to create connections to
smolnar82 commented on a change in pull request #443:
URL: https://github.com/apache/knox/pull/443#discussion_r624738363
##
File path:
gateway-applications/src/main/resources/applications/tokengen/app/js/tokengen.js
##
@@ -53,20 +53,23 @@ function b64DecodeUnicode(str) {
}
smolnar82 merged pull request #442:
URL: https://github.com/apache/knox/pull/442
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
smolnar82 commented on a change in pull request #442:
URL: https://github.com/apache/knox/pull/442#discussion_r624737344
##
File path:
gateway-server/src/main/java/org/apache/knox/gateway/config/impl/GatewayConfigImpl.java
##
@@ -1258,4 +1261,20 @@ public int
smolnar82 commented on a change in pull request #441:
URL: https://github.com/apache/knox/pull/441#discussion_r624737130
##
File path:
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java
##
@@ -207,6 +211,42 @@ public void
smolnar82 commented on a change in pull request #441:
URL: https://github.com/apache/knox/pull/441#discussion_r624735877
##
File path:
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java
##
@@ -207,6 +211,42 @@ public void
smolnar82 commented on a change in pull request #441:
URL: https://github.com/apache/knox/pull/441#discussion_r624735857
##
File path:
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java
##
@@ -207,6 +211,42 @@ public void
smolnar82 commented on a change in pull request #441:
URL: https://github.com/apache/knox/pull/441#discussion_r624735568
##
File path:
gateway-server/src/main/java/org/apache/knox/gateway/services/factory/TokenStateServiceFactory.java
##
@@ -51,8 +55,15 @@ protected Service
lmccay commented on a change in pull request #443:
URL: https://github.com/apache/knox/pull/443#discussion_r623873998
##
File path:
gateway-applications/src/main/resources/applications/tokengen/app/js/tokengen.js
##
@@ -53,20 +53,23 @@ function b64DecodeUnicode(str) {
}
lmccay commented on a change in pull request #441:
URL: https://github.com/apache/knox/pull/441#discussion_r623857140
##
File path:
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java
##
@@ -207,6 +211,42 @@ public void init()
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623523680
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -0,0
smolnar82 opened a new pull request #443:
URL: https://github.com/apache/knox/pull/443
## What changes were proposed in this pull request?
- let end-users declare a more refined lifespan
- let end-users add a comment to the generated
## How was this patch tested?
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623374691
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -0,0
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623373552
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -0,0
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623373356
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -0,0
smolnar82 opened a new pull request #442:
URL: https://github.com/apache/knox/pull/442
## What changes were proposed in this pull request?
Made it possible to connect to an SSL-enabled PostgreSQL server using an SSL
connection. To be able to achieve our goals here I needed to
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623364195
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/AbstractJWTFilter.java
##
@@ -444,41
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623362873
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/SignatureVerificationCache.java
##
@@ -0,0
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623361585
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/AbstractJWTFilter.java
##
@@ -444,41
pzampino commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623360938
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/AbstractJWTFilter.java
##
@@ -96,7 +92,7 @@
smolnar82 commented on a change in pull request #440:
URL: https://github.com/apache/knox/pull/440#discussion_r623334551
##
File path:
gateway-provider-security-jwt/src/main/java/org/apache/knox/gateway/provider/federation/jwt/filter/AbstractJWTFilter.java
##
@@ -444,41
smolnar82 opened a new pull request #441:
URL: https://github.com/apache/knox/pull/441
## What changes were proposed in this pull request?
The following features are implemented by this change:
- if there is a misconfiguration in JDBCTokenState service, the
smolnar82 merged pull request #439:
URL: https://github.com/apache/knox/pull/439
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
1201 - 1300 of 3972 matches
Mail list logo