gyselroth / balloon-client-desktop

balloon desktop client for Windows, Mac OS X and Linux
GNU General Public License v3.0
10 stars 3 forks source link

Temp folder does not get cleaned up #218

Closed raffis closed 4 years ago

raffis commented 4 years ago

Describe the bug

The feedback contains no data. Probably because the .balloon folder is too big.

The other issue is that the .balloon folder may have loads of files in temp which not seemed to get cleaned up even tough there should have been a fix in v1.2.

To Reproduce

?

Expected behavior

Environment

pixtron commented 4 years ago

Cleaning up temp files seems to have worked in v1.2.0 as temp folder in current instance (instance-4) has a combined size of 0B.

  0B    /Users/.../.balloon/temp
  0B    /Users/.../.balloon/instance-4/temp
 26M    /Users/.../.balloon/instance-2/temp
607M    /Users/.../.balloon/instance-1/temp
pixtron commented 4 years ago

temp folders are excluded while creating the zip:

https://github.com/gyselroth/balloon-client-desktop/blob/3d434124489fd1315520b14f9df439e8463f32f8/app/ui/feedback/controller.js#L172-L175

But the size of the zipped report seems to be still bigger then the allowed limit in redmine (nodes.db of instance-4 is already ~14MB).

pixtron commented 4 years ago

In the logs the http request for sending the feedback returns status code 200.

pixtron commented 4 years ago

After our discussion, closing this issue. Problem seems to be the proxy between client and redmine.