Uploaded image for project: 'Spring Data Neo4j'
  1. Spring Data Neo4j
  2. DATAGRAPH-159

Spring Data neo4j rest should not depend on jersey-server

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.RC1
    • Fix Version/s: None
    • Component/s: CORE
    • Labels:
      None

      Description

      Using jersey-server makes it unusable (without Java EE container configuration) to be deployed on JBoss AS7, or other application server that provides its own JAX-RS implementation.

      After excluding jersey-server, my application still works.

      <dependency>
      <groupId>org.springframework.data</groupId>
      <artifactId>spring-data-neo4j-rest</artifactId>
      <version>$

      {spring-data-neo4j.version}

      </version>
      <exclusions>
      ...
      <exclusion>
      <artifactId>jersey-server</artifactId>
      <groupId>com.sun.jersey</groupId>
      </exclusion>
      </exclusions>
      </dependency>

      Is there a reason to depend on jersey-server in the first place? Since spring-data-neo4j-rest is a REST client, not a REST server.

      If jersey-server is needed for compilation of spring data neo4j, but not by the application, please mark the jersey-server dependency as scope=provided or optional=true.

      Thank you.

        Activity

        Hide
        rickardoberg Rickard Oberg added a comment -

        Jersey Server was used by the REST tests. Marked the dependency as test scoped.

        Show
        rickardoberg Rickard Oberg added a comment - Jersey Server was used by the REST tests. Marked the dependency as test scoped.
        Hide
        ceefour Hendy Irawan added a comment -

        Go go Rickard!

        Show
        ceefour Hendy Irawan added a comment - Go go Rickard!

          People

          • Assignee:
            rickardoberg Rickard Oberg
            Reporter:
            ceefour Hendy Irawan
            Last updater:
            Trevor Marshall
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: