[GitHub] [tomee] asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl

2019-11-22 Thread GitBox
asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl URL: https://github.com/apache/tomee/pull/613#issuecomment-557716427 Can one of the admins verify this patch? This is an autom

[GitHub] [tomee] asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl

2019-11-22 Thread GitBox
asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl URL: https://github.com/apache/tomee/pull/613#issuecomment-557716428 Can one of the admins verify this patch? This is an autom

[GitHub] [tomee] cocorossello opened a new pull request #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl

2019-11-22 Thread GitBox
cocorossello opened a new pull request #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl URL: https://github.com/apache/tomee/pull/613 Some tests in openejb-core fail, but it doen't seem related at all

[GitHub] [tomee] asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl

2019-11-22 Thread GitBox
asf-ci commented on issue #613: TOMEE-2744: JD14 compatibility: remove usage of javax.security.acl URL: https://github.com/apache/tomee/pull/613#issuecomment-557716426 Can one of the admins verify this patch? This is an autom

Removal of Xalan

2019-11-22 Thread Jonathan Gallimore
Just wanted to highlight this change that I have committed: https://issues.apache.org/jira/browse/TOMEE-2743 I ran into an issue where an application was using an XML Transformer via TransformerFactory.newInstance(), and running into this issue: https://issues.apache.org/jira/browse/XALANJ-2617 wh

Re: Docker Images on Dockerhub

2019-11-22 Thread Carl Mosca
Jon, I am not familiar with the flip to the useradd but one of the key points is to ensure that the runtime user id may be arbitrary. The idea is that (at least in some/our use cases), we don't know by design what the user id (UID) will be at runtime but we can be assured the group id will be roo

[GitHub] [tomee] jgallimore merged pull request #612: Fixed a minor typo

2019-11-22 Thread GitBox
jgallimore merged pull request #612: Fixed a minor typo URL: https://github.com/apache/tomee/pull/612 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [tomee] jgallimore commented on issue #612: Fixed a minor typo

2019-11-22 Thread GitBox
jgallimore commented on issue #612: Fixed a minor typo URL: https://github.com/apache/tomee/pull/612#issuecomment-557469000 Good catch, thank you! This is an automated message from the Apache Git Service. To respond to the mes

Re: Docker Images on Dockerhub

2019-11-22 Thread Jonathan Gallimore
There's other cleanup for old Dockerfiles, and making everything consistent throughout. Certainly some changes for new images had come through where they were running as root. There's a flip to using `useradd` to add the TomEE user as opposed to manually appending to /etc/passwd. I think the goal y

Re: Docker Images on Dockerhub

2019-11-22 Thread Carl Mosca
This has not been on my radar for some time but I thought the user was changed back in January/May or are these different efforts/projects? : https://github.com/tomitribe/docker-tomee/pull/33 On Fri, Nov 22, 2019 at 4:18 AM Jonathan Gallimore < jonathan.gallim...@gmail.com> wrote: > Sounds like

Re: Docker Images on Dockerhub

2019-11-22 Thread Jonathan Gallimore
Sounds like good feedback. I merged your changes in. You could well be a published author on the official docker images by the end of the day :) Jon On Fri, Nov 22, 2019 at 1:45 AM Jenkins, Rodney J (Rod) < jenki...@nationwide.com> wrote: > Update: > > They wanted a change to the Docker files.