[jira] [Commented] (LOG4J2-1852) Plugins lookup by package is not compatible with Spring Boot jar packaging

2017-07-17 Thread Tanner Altares (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4J2-1852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16089889#comment-16089889
 ] 

Tanner Altares commented on LOG4J2-1852:


PR https://github.com/apache/logging-log4j2/pull/93 has been submitted as a 
possible option to address this ticket.

> Plugins lookup by package is not compatible with Spring Boot jar packaging
> --
>
> Key: LOG4J2-1852
> URL: https://issues.apache.org/jira/browse/LOG4J2-1852
> Project: Log4j 2
>  Issue Type: Bug
>  Components: Plugins
>Affects Versions: 2.7
> Environment: Spring Boot 1.5.x, java 8
>Reporter: Maksym Novoseltsev
>Priority: Minor
>
> Here is you can find a description of the issue and pretty usefull answer 
> from Spring Boot team regarding the very same issue:
> https://github.com/spring-projects/spring-boot/issues/8637
> Also you will find a sample project there that will help to reproduce issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (LOG4J2-1852) Plugins lookup by package is not compatible with Spring Boot jar packaging

2017-07-17 Thread Tanner Altares (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4J2-1852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16089643#comment-16089643
 ] 

Tanner Altares commented on LOG4J2-1852:


I can work on submitting a patch this week.  [~garydgregory] the 
Log4j2Plugins.dat file was included in the executable Jar and was discovered by 
ResolverUtil, unfortunately when discovery happens the URI is rewritten.  On 
initial discovery it is found under !/BOOT-INF/lib/, then modified to .  To by pass 
this you can generate a flattened uber jar with the shade plugin, but tolerance 
to handle Spring Boot's packaging seems like an easier approach will utilizing 
the framework.  

> Plugins lookup by package is not compatible with Spring Boot jar packaging
> --
>
> Key: LOG4J2-1852
> URL: https://issues.apache.org/jira/browse/LOG4J2-1852
> Project: Log4j 2
>  Issue Type: Bug
>  Components: Plugins
>Affects Versions: 2.7
> Environment: Spring Boot 1.5.x, java 8
>Reporter: Maksym Novoseltsev
>Priority: Minor
>
> Here is you can find a description of the issue and pretty usefull answer 
> from Spring Boot team regarding the very same issue:
> https://github.com/spring-projects/spring-boot/issues/8637
> Also you will find a sample project there that will help to reproduce issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (LOG4J2-1852) Plugins lookup by package is not compatible with Spring Boot jar packaging

2017-07-17 Thread Maksym Novoseltsev (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4J2-1852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16089434#comment-16089434
 ] 

Maksym Novoseltsev commented on LOG4J2-1852:


Hi [~garydgregory], unfortunately, I am not in a position right now to prepare 
a patch. I will keep in mind this issue and try to review it once more. Would 
you be able to share a link with FAQ regarding contribution rules/advices?

> Plugins lookup by package is not compatible with Spring Boot jar packaging
> --
>
> Key: LOG4J2-1852
> URL: https://issues.apache.org/jira/browse/LOG4J2-1852
> Project: Log4j 2
>  Issue Type: Bug
>  Components: Plugins
>Affects Versions: 2.7
> Environment: Spring Boot 1.5.x, java 8
>Reporter: Maksym Novoseltsev
>Priority: Minor
>
> Here is you can find a description of the issue and pretty usefull answer 
> from Spring Boot team regarding the very same issue:
> https://github.com/spring-projects/spring-boot/issues/8637
> Also you will find a sample project there that will help to reproduce issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (LOG4J2-1852) Plugins lookup by package is not compatible with Spring Boot jar packaging

2017-07-16 Thread Gary Gregory (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4J2-1852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16089115#comment-16089115
 ] 

Gary Gregory commented on LOG4J2-1852:
--

Are you able to provide a patch?

Gary

> Plugins lookup by package is not compatible with Spring Boot jar packaging
> --
>
> Key: LOG4J2-1852
> URL: https://issues.apache.org/jira/browse/LOG4J2-1852
> Project: Log4j 2
>  Issue Type: Bug
>  Components: Plugins
>Affects Versions: 2.7
> Environment: Spring Boot 1.5.x, java 8
>Reporter: Maksym Novoseltsev
>Priority: Minor
>
> Here is you can find a description of the issue and pretty usefull answer 
> from Spring Boot team regarding the very same issue:
> https://github.com/spring-projects/spring-boot/issues/8637
> Also you will find a sample project there that will help to reproduce issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (LOG4J2-1852) Plugins lookup by package is not compatible with Spring Boot jar packaging

2017-07-11 Thread Tanner Altares (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4J2-1852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16082052#comment-16082052
 ] 

Tanner Altares commented on LOG4J2-1852:


Is there any update or on this issue?  As stated by the Spring Boot team using 
the JARURLConnection route could eliminate the need to modify the URL done by 
the ResolverUtil.  A possible approach with limited modifications could be to 
overload the loadImplementationsInJar method to allow for a JarFile and do the 
needed matching tests.  This should handle common packaging structures of 
jars/wars along with the nested approach for Spring Boot's packaging.


{code:java}
private void loadImplementationsInJar(ResolverUtil.Test test, String parent, 
JarFile jarFile) {

final Enumeration entries = jarFile.entries();

while (entries.hasMoreElements()) {
JarEntry entry = entries.nextElement();
String name = entry.getName();

if (!entry.isDirectory() && name.startsWith(parent) && 
this.isTestApplicable(test, name)) {
this.addIfMatching(test, name);
}
}
 }
{code}


> Plugins lookup by package is not compatible with Spring Boot jar packaging
> --
>
> Key: LOG4J2-1852
> URL: https://issues.apache.org/jira/browse/LOG4J2-1852
> Project: Log4j 2
>  Issue Type: Bug
>  Components: Plugins
>Affects Versions: 2.7
> Environment: Spring Boot 1.5.x, java 8
>Reporter: Maksym Novoseltsev
>Priority: Minor
>
> Here is you can find a description of the issue and pretty usefull answer 
> from Spring Boot team regarding the very same issue:
> https://github.com/spring-projects/spring-boot/issues/8637
> Also you will find a sample project there that will help to reproduce issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)