Deprecate javax.portlet:portlet-api and portlet support
#305
Comments
cyb3r4nt
added a commit
to cyb3r4nt/jsonrpc4j
that referenced
this issue
Jan 19, 2023
…avax.portlet.ResourceRequest, javax.portlet.ResourceResponse) as deprecated Added @deprecated marker and wrote a comment in the Javadoc Signed-off-by: cyb3r4nt <[email protected]>
cyb3r4nt
added a commit
to cyb3r4nt/jsonrpc4j
that referenced
this issue
Jan 19, 2023
…avax.portlet.ResourceRequest, javax.portlet.ResourceResponse) as deprecated Added @deprecated marker and wrote a comment in the Javadoc Signed-off-by: cyb3r4nt <[email protected]>
briandilley
added a commit
that referenced
this issue
May 24, 2023
#305: Deprecate portlet-api support in the JsonRpcServer
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
JsonRpcServerdepends on thejavax.portlet:portlet-apiand requires this to be in the classpath even if portlet support is not used.At the same time, portlet applications are not very popular nowadays,
and probably there are no many users who actually need this.
My personal opinion is that majority of users do not use that feature,
and are obligated to bring
javax.portlet:portlet-apionly becausejsonrpc4jneeds this.Please comment and vote if you still need that Java portlets support.
Portlet methods are actually quite short:
https://github.com/briandilley/jsonrpc4j/blob/1.6.0/src/main/java/com/googlecode/jsonrpc4j/JsonRpcServer.java#L76-L105
Active users may just inline this logic into their own projects,
and use other streaming method to handle those requests:
https://github.com/briandilley/jsonrpc4j/blob/1.6.0/src/main/java/com/googlecode/jsonrpc4j/JsonRpcBasicServer.java#L230-L260
The plan may be to mark it as
@Deprecated(forRemoval = true),and remove this in one of the future versions.
The text was updated successfully, but these errors were encountered: