Temp files should be created in a temp directory #423

Activity

1. bsideup commented on Aug 1, 2017

   

   bsideup

   on Aug 1, 2017

   Member

   More actions

   Hi @axelfontaine,

   Unfortunately, Docker doesn't allow mounts outside of user home directory, at least on Mac / Windows, this is why we do that. Any idea how to improve?
2. axelfontaine commented on Aug 1, 2017

   

   axelfontaine

   on Aug 1, 2017

   Author

   More actions

   The file I had was actually created in my source directory on the host Windows x64 machine and it doesn't lie under my user home. I have a feeling this temp file may not have been created by Docker, but by testcontainers itself. (I deleted before filing this issue and forgot to write down the exact name, but it was something like .testcontainers.tmp.something
3. axelfontaine commented on Aug 1, 2017

   

   axelfontaine

   on Aug 1, 2017

   Author

   More actions

   The culprit is the first line in org.testcontainers.utility.MountableFile.extractClassPathResourceToTempLocation() :

   File tmpLocation = new File(".testcontainers-tmp-" + Base58.randomString(5));
4. arcuri82 commented on Aug 8, 2017

   

   arcuri82

   on Aug 8, 2017

   More actions

   Having this same issue, where my folders get polluted by these tmp files. If for any reason the TMP folder of the OS cannot be used, a better default location than "." could be "./target" (at least for Maven builds)

   👍React with 👍2Reacted by Mark Bramnik and VladoDemcak
5. kiview commented on Dec 9, 2017

   

   kiview

   on Dec 9, 2017

   Member

   More actions

   Has been fixed with #450.
6. kiview

   closed this as completedon Dec 9, 2017
7. rnorth

   added a commit that references this issue on Dec 24, 2018

   Bump influxdb-java from 2.10 to 2.14 in /modules/influxdb (#1070)

   ...

   f986243