Opened 6 years ago

Closed 4 years ago

#1484529 closed Bugs (fixed)

Config directive, temp_dir, does not support relative path under Windows

Reported by: ekovarski Owned by: till
Priority: 5 Milestone: 0.2.2
Component: PHP backend Version: 0.1-rc1
Severity: normal Keywords: temp_dir, config
Cc:

Description (last modified by till)

It appears that using a relative path for temp_dir under Windows is not fully supported.

While Roundcube does function without an absolute path, some functions, such as attaching files fail intermittently without any errors.

For example, when attaching files the system would upload the attachment but when you hit send the message would be sent without any attachments.

Using an absolute path resolves the issue.

Change History (7)

comment:1 Changed 6 years ago by till

  • Description modified (diff)
  • Keywords temp_dir, config added; temp_dir removed

comment:2 Changed 6 years ago by till

Before I can accept the ticket, please provide the following:

a) the error occurred
b) your relative path
c) your absolute path

comment:3 Changed 6 years ago by till

  • Owner set to till
  • Status changed from new to assigned

comment:4 Changed 6 years ago by Stephan

I faced similar problems when my configuration contained a value for open_basedir.
Since I set it to none everything is working fine.
But without any logfiles its hard to get into it.
Regards, Stephan

comment:5 Changed 5 years ago by till

  • Milestone set to 0.2-stable

I need to test this - we might need to wrap the setting through realpath() but that is kind of expensive and I don't want to do this. Instead I'd like people to define absolute path.

This ticket still lacks examples, postponing. Apparently not as urgent.

comment:6 Changed 4 years ago by alec

  • Component changed from Client Scripts to PHP backend

Two points:
1) http://php.net/manual/en/function.move-uploaded-file.php read a comment from "Darrell 18-May-2005 09:51". Looks like we should build absolute path on Windows.
2) we should add file_exists() check after move_uploaded_file() use.

comment:7 Changed 4 years ago by alec

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in [779cbe00].

Note: See TracTickets for help on using tickets.