gchq / Gaffer

A large-scale entity and relation database supporting aggregation of properties
Apache License 2.0
1.77k stars 350 forks source link

Road traffic demo REST port is not changed by POM property #3185

Closed GCHQDeveloper314 closed 6 months ago

GCHQDeveloper314 commented 6 months ago

Describe the bug The road-traffic-demo POM contains a property for configuring the port to use for Gaffer's REST API.

https://github.com/gchq/Gaffer/blob/7a34d83504c89a49b3c6bc0a26f833cda368cd41/example/road-traffic/road-traffic-demo/pom.xml#L32

However, changing this property doesn't in fact change the port used.

To Reproduce Steps to reproduce the behaviour:

  1. Edit the port used in the POM or override the POM property manually.
  2. Run the demo using mvn install -Proad-traffic-demo -pl :road-traffic-demo.
  3. The REST API will start using port 8080, and not the standalone-port value.

Expected behaviour Setting the standalone-port property value will set the REST API port used when the demo is run.