reta commented on a change in pull request #369: [CXF-7610] - Adding SPI to handle customizations to the server bean. URL: https://github.com/apache/cxf/pull/369#discussion_r162792677
########## File path: rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ResourceUtils.java ########## @@ -902,9 +904,18 @@ public static JAXRSServerFactoryBean createApplication(Application app, bean.getProperties(true).putAll(appProps); } bean.setApplication(app); - + customize(bean); return bean; } + + private static void customize(JAXRSServerFactoryBean bean) { + ServiceLoader<JAXRSServerFactoryCustomizationExtension> extensions Review comment: @johnament That's the thing, right, if you leave it here it should become a public instance method of the `JAXRSServerFactoryBean`. Than the question would be, it is there but never being called inside `JAXRSServerFactoryBean`, what the purpose of this method? I think we could always move it back if the use case pops up. Make sense? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services