You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by radcortez <gi...@git.apache.org> on 2018/09/26 01:16:08 UTC

[GitHub] tomee pull request #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

GitHub user radcortez opened a pull request:

    https://github.com/apache/tomee/pull/173

    TOMEE-2247 - MicroProfile JWT Propagation 1.1

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/radcortez/tomee TOMEE-2247

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tomee/pull/173.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #173
    
----
commit 66fd53358b7035278bc6edf28a772885f5366d7c
Author: Roberto Cortez <ra...@...>
Date:   2018-09-11T15:45:36Z

    Update MP Geronimo Config to 1.2.

commit 5568d3d27095909e8da33130383608f22463cbea
Author: Roberto Cortez <ra...@...>
Date:   2018-09-12T11:29:45Z

    TOMEE-2247 - Update MP JWT API to 1.1.

commit bdcfcd24b55e267ff43495618b8a440f9a27efbb
Author: Roberto Cortez <ra...@...>
Date:   2018-09-12T11:30:48Z

    TOMEE-2247 - Added support for MP Configuration.

commit f06dde36142d630218c394d24e29ba82e6867090
Author: Roberto Cortez <ra...@...>
Date:   2018-09-12T14:53:35Z

    TOMEE-2247 - Load public key from classpath Location.

commit b1457d799fb811ea280a862c022bc85ade4d6524
Author: Roberto Cortez <ra...@...>
Date:   2018-09-12T14:57:19Z

    TOMEE-2247 - Fixed TCK IssValidationTest.

commit 66aab2fccba7bf374a02cc035ad92ed5182f9d5e
Author: Roberto Cortez <ra...@...>
Date:   2018-09-13T00:08:06Z

    TOMEE-2247 - Read key from http. Test not passing yet due to hardcoded urls.

commit 60a550e5d313d3dd112fce2175b4197c2480117f
Author: Roberto Cortez <ra...@...>
Date:   2018-09-13T16:23:53Z

    TOMEE-2247 - Refactor Arquillian packaging to try to fix issues with war name and context root.

commit 2c2b3de34eecde5f0f32de85f4dad1e5c0f58643
Author: Roberto Cortez <ra...@...>
Date:   2018-09-14T11:09:43Z

    TOMEE-2247 - Simplified Arquillian packaging.

commit f13c3c1522d3492747576ef87c8b314f14d1f6da
Author: Roberto Cortez <ra...@...>
Date:   2018-09-14T14:59:15Z

    TOMEE-2247 - Removed CDI RequestScope from ConfigurableJWTAuthContextInfo.

commit b0df69713557307b09213e0a116989e23ebafc47
Author: Roberto Cortez <ra...@...>
Date:   2018-09-24T11:52:26Z

    TOMEE-2247 - Changed ConfigurableJWTAuthContextInfo to be an ApplicationScoped bean that loads data on application initialization.

commit 758661c1c8d6f4c6dfa5f1b55dd8375c68e7a992
Author: Roberto Cortez <ra...@...>
Date:   2018-09-24T12:59:50Z

    TOMEE-2247 - Load public key from file Location.

commit 125f2f6e35dfcbf43c24b1d8b079289f72049ad9
Author: Roberto Cortez <ra...@...>
Date:   2018-09-24T22:05:58Z

    TOMEE-2247 - Leave Principal integration and injection for now.

commit 2b0ee0a12b26354932c6bde9c2b16268fa7250ba
Author: Roberto Cortez <ra...@...>
Date:   2018-09-24T22:26:02Z

    TOMEE-2247 - Simplified public key reading from input stream.

commit 777c627a8b720f9bc81f019fcffb02cb18682616
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T10:00:19Z

    TOMEE-2247 - Initial support for JWK.

commit 6ce478fc07c3cf24e6038436a784165470b132af
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T11:06:31Z

    TOMEE-2247 - Support to load keys from generic Urls.

commit 399fc4eba6c3f11124ad6961d5765b64338d3e9b
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T12:08:32Z

    TOMEE-2247 - Fixed test.

commit 8efb26fcfdb41f8a0bf7a934b8ded27255ca3e4e
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T12:14:02Z

    TOMEE-2247 - Ignore test temporarily.

commit fc2b3d991e17b51032a51b7cdb5686320bfbffdc
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T17:18:26Z

    TOMEE-2247 - Initial support for JWK Set.

commit ded275e9e01c66b6f4efb2caa70cdec1adbcb048
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T18:00:47Z

    TOMEE-2247 - Refactor to support multiple keys.

commit 14b73bcd9ce0189fd744a1e2ead05b790e16dc73
Author: Roberto Cortez <ra...@...>
Date:   2018-09-25T20:45:01Z

    TOMEE-2247 - Fixed JWK Set support.

----


---

[GitHub] tomee issue #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

Posted by radcortez <gi...@git.apache.org>.
Github user radcortez commented on the issue:

    https://github.com/apache/tomee/pull/173
  
    @jeanouii this work should now be complete. I've sent an email about this to the ML. I think we can merge it.


---

[GitHub] tomee issue #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

Posted by jeanouii <gi...@git.apache.org>.
Github user jeanouii commented on the issue:

    https://github.com/apache/tomee/pull/173
  
    So what do we do?
    Do we leave this open?


---

[GitHub] tomee issue #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

Posted by jeanouii <gi...@git.apache.org>.
Github user jeanouii commented on the issue:

    https://github.com/apache/tomee/pull/173
  
    @radcortez where are we at?
    There is a conflict and I'm not sure you have seen it


---

[GitHub] tomee pull request #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/tomee/pull/173


---

[GitHub] tomee issue #173: TOMEE-2247 - MicroProfile JWT Propagation 1.1

Posted by radcortez <gi...@git.apache.org>.
Github user radcortez commented on the issue:

    https://github.com/apache/tomee/pull/173
  
    Hey @jeanouii,
    
    This was on hold, because @jgallimore picked this up and continue the work on his own branch here:
    https://github.com/jgallimore/tomee/commits/jwt-1.1
    
    Since he is done, I'm rebasing this with his branch and push an updated PR.
    
    Due to this:
    https://github.com/eclipse/microprofile-jwt-auth/issues/118
    
    A few test cases in the TCK are failing. I believe this is an issue with the TCK itself. I think we can still test this if we implement our own tests with a separate app to expose the JWK endpoint. Me and @jgallimore discussed this on the ML and so I will be picking this up and hopefully complete the JWT 1.1 implementation work.


---