search

52North / WPS

**DEPRECATED** The 52°North Web Processing Service enables the deployment of geo-processes on the web in a standardized way.
GNU General Public License v2.0
63 stars 55 forks source link

ExecutionContext.getTempDirectoryPath() misses to put an "/" after the java.io.tmpdir property #228

Closed grmpfhmbl closed 8 years ago

grmpfhmbl commented 8 years ago

Not sure, if this is a bug in WPS or my Java environment is poorly configured, but ExecutionContext.getTempDirectoryPath() returns /var/tmp3a5d1d23-50db-4a9e-ade0-b741e1fade09" and I guess it should return /var/tmp/3a5d1d23-50db-4a9e-ade0-b741e1fade09

If that is a but, I'd fix it by changing Line 47 to

    return System.getProperty("java.io.tmpdir") + System.getProperty("file.separator") + this.tempFolderName;

and create a pull request.

bpross-52n commented 8 years ago

Thanks for this, it seems to be a bug. We would be glad, if you would contribute a fix. Note that we will need a signed Contributors Licence Agreement from you before we can accept a possible PR. You can find more info here: http://52north.org/about/licensing/cla-guidelines

grmpfhmbl commented 8 years ago

@bpross-52n I just created a PR. The signed CLA should hopefully arrive in the next couple of days at 52° North.