<div dir="ltr">Hi Alessio,<div><br></div><div>sorry for the delayed response. I finally found some time to have a deeper look at this. I would like to describe my idea, so I can get feedback before starting the implementation.</div><div><br></div><div><br></div><div>To recap what I'm trying to achieve: Basically the idea is to create a SPI which allows to alter the metadata of resource classes. I came up with the following interface for such a SPI:</div><div><br></div><div><font face="monospace, monospace"> public interface ResourceClassProcessor {</font></div><div><font face="monospace, monospace"> ResourceClass process( ResourceClass clazz );</font></div><div><font face="monospace, monospace"> }<br></font></div><div><br></div><div>IMO this interface provides most flexibility. The idea is similar to what CDI provides with <a href="https://docs.oracle.com/javase/8/docs/api/java/lang/reflect/AnnotatedType.html">AnnotatedType</a>. Implementations of the new SPI can either return the provided class (if they don't want to change it) or they can return a new instance, typically something that wraps the original class and changes only a few methods.</div><div><br></div><div>Integrating this SPI requires two refactorings of the existing code base, but I really think that this will improve the overall structure of the metadata handling:</div><div><ul><li>Currently <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/metadata/ResourceClass.java">ResourceClass</a>, <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/metadata/ResourceMethod.java">ResourceMethod</a>, etc. are classes with protected fields which are directly accessed by the <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/metadata/ResourceBuilder.java">ResourceBuilder</a>. The first required refactoring would be to extract interfaces from these classes so that we get an interface ResourceClass with a default implementation DefaultResourceClass. Same for ResourceMethod, etc. This will allow other implementations of these interfaces later on.</li><li>Currently <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/metadata/ResourceBuilder.java">ResourceBuilder</a> has only static methods to build ResourceClasse instances. This makes it difficult to "configure" this builder with such processors. Therefore, I would like to refactor the builder from a "utility class" with static methods to something that can be instantiated and therefore also be configured. I think <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/ResteasyProviderFactory.java">ResteasyProviderFactory</a> would be a good place to create such a builder _once_ and then provide it to all components that want to use it.</li></ul>With these refactorings integrating the new SPI should be easy. <a href="https://github.com/resteasy/Resteasy/blob/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/ResteasyProviderFactory.java">ResteasyProviderFactory</a> could lookup the SPI implementations using the default provider mechanism. The processors will then be feed into the ResourceBuilder when creating it. The ResourceBuilder can then build the metadata for classes from annotations (as it is done today) and after that apply all ResourceClassProcessors before returning the ResourceClass to the other components (like the registry). This will ensure that the processors are applied consistently.</div><div><br></div><div>I think the two refactorings mentioned above a worth the effort because they improve the overall structure of the <a href="https://github.com/resteasy/Resteasy/tree/master/resteasy-jaxrs/src/main/java/org/jboss/resteasy/spi/metadata">metadata package</a> and provides much more flexibility in the future. And they also allow integrate the new SPI very easily.</div><div><br></div><div><br></div><div>What do you think? If you agree that the refactorings are a good thing, I will start to work on them and provide a pull request.</div><div><br></div><div>Christian</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-09-27 22:25 GMT+02:00 Alessio Soldano <span dir="ltr"><<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Forwarding to the list :-)<div><div class="h5"><br><br><div><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Alessio Soldano</b> <span dir="ltr"><<a href="mailto:asoldano@redhat.com" target="_blank">asoldano@redhat.com</a>></span><br>Date: Wed, Sep 27, 2017 at 10:24 PM<br>Subject: Re: [resteasy-dev] SPI to add JAX-RS annotations dynamically<br>To: Christian Kaltepoth <<a href="mailto:christian@kaltepoth.de" target="_blank">christian@kaltepoth.de</a>><br><br><br><div dir="ltr">Hi Christian,<br><div><div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, Sep 27, 2017 at 9:19 PM, Christian Kaltepoth <span dir="ltr"><<a href="mailto:christian@kaltepoth.de" target="_blank">christian@kaltepoth.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div>sorry for the delayed response. I agree that if some SPI would allow to modify the builders, it should be done before the resource is registered in the resource factory. Do you think such a SPI would be useful for others?</div></div></blockquote><div><br></div></span><div>Probably yes, but your scenario might even be enough to justify it.<br></div><span><div> </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div> I would be happy to contribute some code if it improves portability between RESTEasy and the MVC RI.</div></div></blockquote><div><br></div></span><div>Sure, feel free to work on that and send a PR.<br></div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Thanks for pointing me to the resteasy-cdi integration. I'll have a deeper look at it in the next days.</div></div></blockquote><div><br></div></span><div>OK, good, I hope that can offer some idea for integrating.</div><div><br></div><div>Cheers</div><span class="m_4720272789294461365HOEnZb"><font color="#888888"><div>Alessio</div></font></span></div><br></div></div></div>
</div><br></div></div></div></div>
<br>______________________________<wbr>_________________<br>
resteasy-dev mailing list<br>
<a href="mailto:resteasy-dev@lists.jboss.org">resteasy-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/resteasy-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/resteasy-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div>Christian Kaltepoth</div><div>Blog: <a href="http://blog.kaltepoth.de/" target="_blank">http://blog.kaltepoth.de/</a></div><div>Twitter: <a href="http://twitter.com/chkal" target="_blank">http://twitter.com/chkal</a></div><div>GitHub: <a href="https://github.com/chkal" target="_blank">https://github.com/chkal</a></div><div><br></div></div>
</div>