wearefrank / frank-runner

Run any Frank with Tomcat
Apache License 2.0
16 stars 12 forks source link

Add ladybug-frontend-*.jar to jarscanning #100

Closed mhdirkse closed 3 months ago

philipsens commented 3 months ago

A title like: Add ladybug-frontend-*.jar to jarscanning would be more usefull

nielsm5 commented 2 months ago

De Frank!Runner heeft een behoorlijke lijst, is dit echt nodig? In ons docker image staat alleen het FF, en niets van de ladybug.

jacodg commented 2 months ago

Ik vraag me af of die van de Docker image wel werkt als ik kijk naar:

https://github.com/frankframework/frankframework/blob/da1d65d519906eb25136995e9f47fee55425da5f/docker/Tomcat/webapp/Dockerfile#L25

De catalinaAdditional.properties wordt ge-append aan catalina.properties waar tomcat.util.scan.StandardJarScanFilter.jarsToSkip al in staat. Dus met deze append komt tomcat.util.scan.StandardJarScanFilter.jarsToSkip er twee keer in te staan en is het de vraag welke van de twee Tomcat gebruikt. Ik vermoed dat die van Tomcat zelf wordt gebruikt want anders zou ik niet direct weten hoe de Ladybug resources gevonden kunnen worden.

De lijst in Ladybug is langer dan in de Docker image nodig zal zijn omdat de Frank!Runner voor verschillende Frank!Framework versies dezelfde Tomcat installatie gebruikt.