Ticket UUID: | 1006157 | |||
Title: | ftpd LIST response mis-formatted | |||
Type: | Bug | Version: | None | |
Submitter: | blacksqr | Created on: | 2004-08-09 19:28:35 | |
Subsystem: | ftpd | Assigned To: | gwlester | |
Priority: | 5 Medium | Severity: | ||
Status: | Closed | Last Modified: | 2004-10-06 02:16:02 | |
Resolution: | Fixed | Closed By: | andreas_kupries | |
Closed on: | 2004-10-05 19:16:02 | |||
Description: |
::ftpd's response to the LIST command should be formatted similar to the output of "ls -l", including owner and group values. But when ftpd is run on Windows, the owner and group fields are simply omitted, causing the ftp clients I am working with to mis-parse the response. I investigated how the ftp server at ftp.microsoft.com formats its response to LIST, and I found that it uses the generic literal strings "owner" and "group" to fill the necessary fields. So I patched ::ftpd to behave in the same way. The procedure ::ftpd::fsFile::fs needs the "list" option of the switch command replaced with the following: list { # [ 766112 ] report . and .. directories (linux) # Copied the code from 'nlst' above to handle this. foreach f [lsort $fileList] { if {[string equal [file tail $f] "."] || \ [string equal [file tail $f] ".."]} { continue } file stat $f stat if {[string equal $tcl_platform(platform) "unix"]} { set user [file attributes $f -owner] set group [file attributes $f -group] } else { set user owner set group group } puts $outchan [format "%s %3d %s %8s %11s %s %s" \ [PermBits $f $stat(mode)] $stat(nlink) \ $user $group $stat(size) \ [FormDate $stat(mtime)] [file tail $f]] } } | |||
User Comments: |
andreas_kupries added on 2004-10-06 02:16:02:
Logged In: YES user_id=75003 Accepted and committed to head. afaupell added on 2004-10-05 15:20:18: File Added - 103870: diff afaupell added on 2004-10-05 15:19:36: Logged In: YES user_id=812825 i agree with the submitter, the format should be constant across platforms. patch generated and attached. andreas_kupries added on 2004-08-10 02:38:57: Logged In: YES user_id=75003 Notes for future reports: Note I: Setting the 'category' information will in many cases auto-assign a maintainer for review and application. Note II: Changes sources, or patches are best not inlined into the bug report. Attaching them as separate files is better for two reasons: * The notification mails for the report are shorter, as SF doesn't have to send all this code, only the comments. * The changes are easier to get out of the report, via click on links. Right now I can either try copy/paste a mangled version out of the webbrowser, or I have to go to a notification mail to get an unmangled version I'd rather simply click on a link for the attachment. |
Attachments:
- diff [download] added by afaupell on 2004-10-05 15:20:18. [details]