Skip to content
This repository was archived by the owner on Mar 6, 2020. It is now read-only.

Webapp service in Open Liberty #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kubik42
Copy link

@Kubik42 Kubik42 commented Apr 9, 2018

@arun-gupta From the old PR:

Added three new profiles: wildfly for starting the service in Wildfly, openliberty for starting the service in Open Liberty, docker-openliberty for building a Docker image for Open Liberty.

To start the service in Wildfly as before, run:

mvn wildfly-swarm:run -P wildfly

To start the service in Open Liberty, run:

mvn install liberty:start-server -P openliberty

Just in case, run the clean goal when switching between runtimes.

New changes:

The src/main/openliberty/config directory contains three files:

  • server.xml contains the server configuration. This is where you can enable various features in the server.
  • bootstrap.properties contains the runtime variables.
  • server.env contains environment variables.

Justification for code changes:

  • Added a slash to the url pattern in the web.xml file as outlined in the servlet spec.

  • Open Liberty also requires the JavaEE JAX-RS, so hence the reason for adding javax.ws.rs-api dependency and separating the two JAX-RS versions into the two profiles.

  • I also made some suggestive changes to WebappController.java. I can change these back if you prefer the old approach.

@Kubik42
Copy link
Author

Kubik42 commented Apr 9, 2018

@Kubik42
Copy link
Author

Kubik42 commented Apr 9, 2018

Let me know if you need the docker-compose yaml from the old PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant