filesize() to return size of symlink rather than the target file size about doc-en HOT 13 OPEN

I cannot reproduce with php 8.3.6 nor with 8.3.8, would you have a reproducer or give more details regarding your case ?

from doc-en.

I also have this issue on another server with centos 7 and php 8.3.8.
It's just a symlink created using symlink() that I want to delete after I get the size.

from doc-en.

There might be more to it. e.g. are you getting a symlink from a shared (virtualised ?) folder ? network mountpoint ?

from doc-en.

Nope it's a symlink to a file in the same folder.
But it might be that the target file is deleted before the symlink itself.

from doc-en.

It might thus

would you have a reproducer or give more details regarding your case ?

from doc-en.

$file = tempnam(sys_get_temp_dir(), '');
$symlink = $file . '.symlink';
symlink('./' . basename($file), $symlink);
unlink($file);
var_dump(filesize($symlink));

from doc-en.

sure ... looking at php_stat, it seems to behave "as expected', now I m torn between bug and "feature request to change longtime behavior".

from doc-en.

Yeah I didn't expect it to behave like this, I thought filesize would show size of the file supplied and not the target file of a symlink, would've made more sense to do filesize(realpath()) if target file size was needed🤔

Well probably because in most cases filesize of a symlink would be irrelevant😅

from doc-en.

I don't think we can change this. I would not consider it as a bug. This would require RFC to get changed.

from doc-en.

@RV7PR, are you interested in pursuing the RFC process?

from doc-en.

@cmb69 Not really... maybe we should just add a note for filesize() and SplFileInfo::getSize() specifiying that it always uses the target file of a symlink and if the filesize is needed for a symlink that lstat()['size'] should be used and that if target file does not exists anymore it generates a warning.

from doc-en.

Well, I'm fine with transferring this to doc-en (so the documentation could be updated). Any objections?

from doc-en.

I'm okay with that

from doc-en.