fpapon commented on issue #58: [SHIRO-337] basic cdi support
URL: https://github.com/apache/shiro/pull/58#issuecomment-499796951
Hi @rmannibucau thanks for this PR! I will make some test case with the
current master (1.5.0).
fpapon commented on issue #85: [SHIRO-687] Adding Spring's Filters to
ShiroFilterFactorBean when using Java config
URL: https://github.com/apache/shiro/pull/85#issuecomment-499780938
@bdemers the license header is missing in
fpapon merged pull request #143: [SHIRO-696] Use root pom Jetty version in
Jaxrs sample
URL: https://github.com/apache/shiro/pull/143
This is an automated message from the Apache Git Service.
To respond to the message,
asfgit commented on issue #85: [SHIRO-687] Adding Spring's Filters to
ShiroFilterFactorBean when using Java config
URL: https://github.com/apache/shiro/pull/85#issuecomment-499654080
Refer to this link for build results (access rights to CI server needed):
asfgit commented on issue #135: [SHIRO-699] Fix maven warning for
exec-maven-plugin and upgrade to 1.6.0
URL: https://github.com/apache/shiro/pull/135#issuecomment-499650716
Refer to this link for build results (access rights to CI server needed):
asfgit commented on issue #137: [SHIRO-698] Improve build with maven profile
URL: https://github.com/apache/shiro/pull/137#issuecomment-499639460
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/106/
asfgit commented on issue #140: [SHIRO-697] Reduce shiro test logging level to
INFO
URL: https://github.com/apache/shiro/pull/140#issuecomment-499630688
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/105/
asfgit commented on issue #143: [SHIRO-696] Use root pom Jetty version in Jaxrs
sample
URL: https://github.com/apache/shiro/pull/143#issuecomment-499621809
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/104/
fpapon commented on issue #85: [SHIRO-687] Adding Spring's Filters to
ShiroFilterFactorBean when using Java config
URL: https://github.com/apache/shiro/pull/85#issuecomment-499607074
retest this please
This is an automated
fpapon opened a new pull request #143: [SHIRO-696] Use root pom Jetty version
in Jaxrs sample
URL: https://github.com/apache/shiro/pull/143
This is an automated message from the Apache Git Service.
To respond to the
fpapon merged pull request #140: Reduce shiro test logging level to INFO
URL: https://github.com/apache/shiro/pull/140
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
fpapon merged pull request #142: SHIRO-696 - Upgrade Jetty
URL: https://github.com/apache/shiro/pull/142
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
asfgit commented on issue #142: SHIRO-696 - Upgrade Jetty
URL: https://github.com/apache/shiro/pull/142#issuecomment-499478206
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/103/
coheigea opened a new pull request #142: SHIRO-696 - Upgrade Jetty
URL: https://github.com/apache/shiro/pull/142
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
fpapon merged pull request #141: SHIRO-695 - Update Hazelcast
URL: https://github.com/apache/shiro/pull/141
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
asfgit commented on issue #141: SHIRO-695 - Update Hazelcast
URL: https://github.com/apache/shiro/pull/141#issuecomment-498629013
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/102/
coheigea opened a new pull request #141: SHIRO-695 - Update Hazelcast
URL: https://github.com/apache/shiro/pull/141
Hazelcast should be updated as it carries a CVE:
hazelcast-3.7.2.jar (pkg:maven/com.hazelcast/hazelcast@3.7.2,
cpe:2.3:a:hazelcast:hazelcast:3.7.2:::*) :
asfgit commented on issue #59: [SHIRO-617] ShiroFilter initialize
WebEnvironment on startup
URL: https://github.com/apache/shiro/pull/59#issuecomment-497965629
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/101/Build
asfgit commented on issue #129: [SHIRO-694] Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-497965237
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/100/
fpapon commented on issue #59: [SHIRO-617] ShiroFilter initialize
WebEnvironment on startup
URL: https://github.com/apache/shiro/pull/59#issuecomment-497964395
@col-panic Hi, thanks for your PR, I'm actually working about OSGi in Shiro,
I will take a look.
fpapon commented on issue #140: Reduce shiro test logging level to INFO
URL: https://github.com/apache/shiro/pull/140#issuecomment-497919046
@bdemers I think we should also put the root logger to `INFO` in the
`samples/guice/src/main/resources/log4j.properties`
fpapon commented on issue #140: Reduce shiro test logging level to INFO
URL: https://github.com/apache/shiro/pull/140#issuecomment-497917934
@bdemers full agree, we only have to put in `TRACE` or `DEBUG` when we have
some errors and help to fix.
fpapon merged pull request #137: Build
URL: https://github.com/apache/shiro/pull/137
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
asfgit commented on issue #140: Reduce shiro test logging level to INFO
URL: https://github.com/apache/shiro/pull/140#issuecomment-497857314
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/99/
bdemers commented on issue #140: Reduce shiro test logging level to INFO
URL: https://github.com/apache/shiro/pull/140#issuecomment-497839283
@fpapon, related to your build cleanup PRs
I have mixed feelings about this, as generally I think extra logging is
useful for CI, but in general
asfgit commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-497836128
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/98/
bdemers commented on a change in pull request #137: Build
URL: https://github.com/apache/shiro/pull/137#discussion_r289524251
##
File path: pom.xml
##
@@ -1457,6 +1468,9 @@
ci
+
+false
+
Review
fpapon commented on a change in pull request #137: Build
URL: https://github.com/apache/shiro/pull/137#discussion_r289519208
##
File path: pom.xml
##
@@ -1354,6 +1355,16 @@
+
+fast
+
+true
+true
bdemers commented on a change in pull request #137: Build
URL: https://github.com/apache/shiro/pull/137#discussion_r289518821
##
File path: pom.xml
##
@@ -1457,6 +1468,9 @@
ci
+
+false
+
Review
bdemers commented on a change in pull request #137: Build
URL: https://github.com/apache/shiro/pull/137#discussion_r289518585
##
File path: pom.xml
##
@@ -1354,6 +1355,16 @@
+
+fast
+
+true
+true
fpapon commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-497822375
@bdemers are you ok with the lastest changes?
This is an automated message from the Apache Git Service.
To respond to
fpapon commented on a change in pull request #66: [SHIRO 625] Update 10 minute
Tutorial
URL: https://github.com/apache/shiro/pull/66#discussion_r289263089
##
File path: core/src/main/java/org/apache/shiro/env/BasicIniEnvironment.java
##
@@ -0,0 +1,22 @@
+package
asfgit commented on issue #66: [SHIRO 625] Update 10 minute Tutorial
URL: https://github.com/apache/shiro/pull/66#issuecomment-497573238
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/97/Build result:
fpapon commented on issue #66: [SHIRO 625] Update 10 minute Tutorial
URL: https://github.com/apache/shiro/pull/66#issuecomment-497565847
retest this please
This is an automated message from the Apache Git Service.
To respond
fpapon commented on issue #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-497424418
Thanks @bdemers !
This is an automated message from the Apache Git Service.
To respond to the
bdemers merged pull request #78: Adding another WildcardPermission test case
URL: https://github.com/apache/shiro/pull/78
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
fpapon commented on issue #78: Adding another WildcardPermission test case
URL: https://github.com/apache/shiro/pull/78#issuecomment-497423173
@bdemers can we merge this?
This is an automated message from the Apache Git
bdemers commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497347975
@steinarb yup, you should be all set!
Thanks again!!
This is an automated
steinarb commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497344471
> @steinarb Looks good to me! can you sign an Apache CLA?
https://www.apache.org/licenses/#contributor-license-agreements
@bdemers I have
asfgit commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497334871
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/96/
bdemers commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497328346
@steinarb Looks good to me! can you sign an Apache CLA?
https://www.apache.org/licenses/#contributor-license-agreements
bdemers commented on a change in pull request #138: Support base64 encoded salt
in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#discussion_r288985987
##
File path: core/src/main/java/org/apache/shiro/realm/jdbc/JdbcRealm.java
##
@@ -247,7 +261,11 @@ protected
bdemers commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497325189
Thanks @steinarb! I just retargeted the PR against `master` (it was 1.4
previously)
bdemers commented on issue #139: SHIRO-693 - Update plugins
URL: https://github.com/apache/shiro/pull/139#issuecomment-497323818
Thanks @coheigea !!
This is an automated message from the Apache Git Service.
To respond to the
bdemers merged pull request #139: SHIRO-693 - Update plugins
URL: https://github.com/apache/shiro/pull/139
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
asfgit commented on issue #139: SHIRO-693 - Update plugins
URL: https://github.com/apache/shiro/pull/139#issuecomment-497293824
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/95/
coheigea opened a new pull request #139: SHIRO-693 - Update plugins
URL: https://github.com/apache/shiro/pull/139
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
asfgit commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497153211
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/94/
asfgit commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497142120
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/93/
asfgit commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497134073
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/92/
asfgit commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497120236
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/91/
bdemers commented on issue #138: Support base64 encoded salt in JdbcRealm
URL: https://github.com/apache/shiro/pull/138#issuecomment-497101348
@steinarb can you rebase your branch against master?
This is an automated message
steinarb opened a new pull request #138: Support base64 encoded salt in
JdbcRealm
URL: https://github.com/apache/shiro/pull/138
This fixes [SPAP-552](https://issues.apache.org/jira/browse/SHIRO-552)
The change has been tested in https://github.com/steinarb/authservice/ and
worked
bdemers merged pull request #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
asfgit commented on issue #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-497078820
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/90/
bdemers commented on issue #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-497058433
retest this please
This is an automated message from the Apache Git Service.
To respond to the
fpapon commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496823457
I made changes:
- deactivate `jacoco` by default
- activate `jacoco` with the `ci` profile
- add a `fast` profile with unit tests and ITtests deactivated
asfgit commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496822686
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/87/
fpapon commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496602102
> Thanks for taking this on!
>
> I think we should try to simplify this a little more to make this easier
for contributors.
> Maybe ending with something like `mvn clean
fpapon commented on a change in pull request #137: Build
URL: https://github.com/apache/shiro/pull/137#discussion_r288206310
##
File path: pom.xml
##
@@ -1474,5 +1478,54 @@
+
+samples
+
+
bdemers commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496511288
Thanks for taking this on!
I think we should try to simplify this a little more to make this easier for
contributors.
Maybe ending with something like `mvn clean install
asfgit commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496322335
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/83/
asfgit commented on issue #137: Build
URL: https://github.com/apache/shiro/pull/137#issuecomment-496321836
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/82/
fpapon opened a new pull request #137: Build
URL: https://github.com/apache/shiro/pull/137
Hi @bdemers, I made some changes on the maven build:
- move `integration-tests` and `test-coverage` to the `ci` profile
- add a `samples` profile and move the module `sample`into
This can
fpapon merged pull request #135: Fix maven warning for exec-maven-plugin and
upgrade to 1.6.0
URL: https://github.com/apache/shiro/pull/135
This is an automated message from the Apache Git Service.
To respond to the
asfgit commented on issue #135: Fix maven warning for exec-maven-plugin and
upgrade to 1.6.0
URL: https://github.com/apache/shiro/pull/135#issuecomment-495938395
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/80/
fpapon opened a new pull request #135: Fix maven warning for exec-maven-plugin
and upgrade to 1.6.0
URL: https://github.com/apache/shiro/pull/135
This is an automated message from the Apache Git Service.
To respond to the
fpapon merged pull request #134: [SHIRO-691] Upgrade to maven-jar-plugin 3.1.1
URL: https://github.com/apache/shiro/pull/134
This is an automated message from the Apache Git Service.
To respond to the message, please log on
asfgit commented on issue #134: [SHIRO-691] Upgrade to maven-jar-plugin 3.1.1
URL: https://github.com/apache/shiro/pull/134#issuecomment-495933628
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/79/
fpapon opened a new pull request #134: [SHIRO-691] Upgrade to maven-jar-plugin
3.1.1
URL: https://github.com/apache/shiro/pull/134
This is an automated message from the Apache Git Service.
To respond to the message, please
fpapon merged pull request #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90
This is an automated message from the Apache Git Service.
To respond to the
fpapon commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-493051625
I think we could add `&& hasPrincipals()` here:
fpapon commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-492897172
Yes I think so :)
This is an automated message from the
bdemers commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-492819583
AHH!! maybe adding the null check to `isAuthenticated()` is the way to go
then?
fpapon commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-492774048
@bdemers ok, if I understand the use case, this is about the serialization
of the `Principal`. Sometimes it cannot be
bdemers commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-492765167
@fpapon i think we would want to figure out how/why a subject is
authenticated without a principal.
Possibly making
fpapon commented on issue #90: [SHIRO-661] Add check for the principal of
subject whether is null
URL: https://github.com/apache/shiro/pull/90#issuecomment-492758890
@bdemers as we have no update about the user, can I merge this and made the
update in the `Subject` `isAuthenticated`
fpapon merged pull request #133: [SHIRO-689] Upgrade to commons-codec 1.12
URL: https://github.com/apache/shiro/pull/133
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
asfgit commented on issue #133: [SHIRO-689] Upgrade to commons-codec 1.12
URL: https://github.com/apache/shiro/pull/133#issuecomment-491529508
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/77/
fpapon opened a new pull request #133: [SHIRO-689] Upgrade to commons-codec 1.12
URL: https://github.com/apache/shiro/pull/133
This is an automated message from the Apache Git Service.
To respond to the message, please log
fpapon merged pull request #132: [SHIRO-685] Potential NullPointerException if
PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132
This is an automated message from the Apache Git Service.
To respond to
asfgit commented on issue #132: [SHIRO-685] Potential NullPointerException if
PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132#issuecomment-491001863
Refer to this link for build results (access rights to CI server needed):
fpapon commented on a change in pull request #132: [SHIRO-685] Potential
NullPointerException if PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132#discussion_r282588719
##
File path: core/src/test/java/org/apache/shiro/realm/AuthorizingRealmTest.java
asfgit commented on issue #132: [SHIRO-685] Potential NullPointerException if
PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132#issuecomment-490680337
Refer to this link for build results (access rights to CI server needed):
bdemers commented on a change in pull request #132: [SHIRO-685] Potential
NullPointerException if PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132#discussion_r282270904
##
File path: core/src/test/java/org/apache/shiro/realm/AuthorizingRealmTest.java
fpapon opened a new pull request #132: [SHIRO-685] Potential
NullPointerException if PermissionResolver retu…
URL: https://github.com/apache/shiro/pull/132
…rn null/empty string.
Made changes according to the JIRA and I added a test case.
@bdemers can you review please?
fpapon merged pull request #131: [SHIRO-688] Upgrade to commons-cli 1.4
URL: https://github.com/apache/shiro/pull/131
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
asfgit commented on issue #131: [SHIRO-688] Upgrade to commons-cli 1.4
URL: https://github.com/apache/shiro/pull/131#issuecomment-490586968
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/74/
fpapon opened a new pull request #131: [SHIRO-688] Upgrade to commons-cli 1.4
URL: https://github.com/apache/shiro/pull/131
This is an automated message from the Apache Git Service.
To respond to the message, please log on
fpapon merged pull request #130: [SHIRO-681] Upgrade to compiler Java 8
URL: https://github.com/apache/shiro/pull/130
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
fpapon opened a new pull request #130: [SHIRO-681] Upgrade to compiler Java 8
URL: https://github.com/apache/shiro/pull/130
This is an automated message from the Apache Git Service.
To respond to the message, please log on
asfgit commented on issue #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-489774012
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/72/
asfgit commented on issue #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129#issuecomment-489759648
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/71/Build result:
FAILURE[...truncated
bdemers opened a new pull request #129: Adds BearerToken support
URL: https://github.com/apache/shiro/pull/129
Abstracts the HTTP header processing out of BasicHttpAuthenticationFilter
into HttpAuthenticationFilter (to support both BASIC and Bearer tokens)
Adds `authcBearer` to
asfgit commented on issue #85: Adding Spring's Filters to ShiroFilterFactorBean
when using Java config
URL: https://github.com/apache/shiro/pull/85#issuecomment-489734717
Refer to this link for build results (access rights to CI server needed):
asfgit commented on issue #85: Adding Spring's Filters to ShiroFilterFactorBean
when using Java config
URL: https://github.com/apache/shiro/pull/85#issuecomment-489734369
Refer to this link for build results (access rights to CI server needed):
fpapon merged pull request #44: Release Shiro 1.4.1
URL: https://github.com/apache/shiro-site/pull/44
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
fpapon opened a new pull request #44: Release Shiro 1.4.1
URL: https://github.com/apache/shiro-site/pull/44
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
fpapon merged pull request #43: Update java-authentication-guide.md.vtl
URL: https://github.com/apache/shiro-site/pull/43
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
fpapon commented on issue #43: Update java-authentication-guide.md.vtl
URL: https://github.com/apache/shiro-site/pull/43#issuecomment-487899404
Good catch ;)
LGTM!
Thanks
This is an automated message from the Apache Git
801 - 900 of 1017 matches
Mail list logo