~measurement-factory/squid/ftp-native

From Revision 12816 to 12797

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
12816		Fixes and polishing in response to Amos' squid-dev review da... Fixes and polishing in response to Amos' squid-dev review dated 2014/08/10. Fixed handling of FTP commands terminated with LF instead of CRLF (regression introduced during migration to Tokenizer in r12812). Many polishing touches.	Alex Rousskov	9 years ago
12815		Changes in response to Amos' squid-dev review dated 2014/08/... Changes in response to Amos' squid-dev review dated 2014/08/08. See inner commit messages for details.	Alex Rousskov	9 years ago
12814		SourceFormat Enforcement SourceFormat Enforcement	Alex Rousskov	9 years ago
12813		Fixed HttpHdr::Private/NoCache(v) implementations and optimi... Fixed HttpHdr::Private/NoCache(v) implementations and optimized their API. These calls now avoid assertions and extra trailing commas when called with empty names. The API now allows calling with a String() object, but still needs more polishing work. Moved common code from Ftp::Server::setReply and Ftp::Relay::createHttpReply() into Ftp::HttpReplyWrapper(). Also removed the last non-job callbak from Ftp::Client, polished and synced new FTP comments with the modern client/server/gateway/relay terminology, as well as minimized changes compared to trunk.	Alex Rousskov	9 years ago
12812		Converted some of the new FTP code to use SBuf and Tokenizer Converted some of the new FTP code to use SBuf and Tokenizer instead of MemBuf, String, and c-string manipulations.	Alex Rousskov	9 years ago
12811		Tokenizer fixes and API improvements. Tokenizer fixes and API improvements. Taught Tokenizer to keep track of the number of parsed bytes. Many callers need to know that because they need to adjust/consume I/O offsets/buffers. Adjusted unused Parser::Tokenizer::token() to not treat NUL delimiter specially. Besides the fact that not all grammars can treat NUL that way, the special NUL treatment resulted in some token() calls returning true for empty tokens, which was confusing parsers. Callers that do not require trailing delimiters, should use prefix() instead. This change is based on experience writing Tokenizer-based FTP parser, although the final parser code uses prefix() instead of token(), for unrelated reasons. Split Parser::Tokenizer::skip(set) into skipOne() and skipAll(). All other skip() methods skip exactly one thing (either a given character or a given token) but the old skip(set) method was skipping multiple things. That confused callers. We now force the caller to make a choice.  Fixed Parser::Tokenizer::skip(char) to avoid out of bound access.	Alex Rousskov	9 years ago
12810		Added CharacterSet::complement() to create "all except those... Added CharacterSet::complement() to create "all except those in that set" sets handy for parsing (e.g., "get all characters until the end of line"). Added CharacterSet::rename() to label sets. Handy in const declarations that use expressions. For example: const CharacterSet AB = (A+B).renamed("AB").	Alex Rousskov	9 years ago
12809		Do not allow SBuf::toLower/toUpper() return a value. Do not allow SBuf::toLower/toUpper() return a value.  Some callers think toLower() method changes the string and some think it returns a changed value without changing the string. Without unportable hacks like __attribute__((warn_unused_result)), some of the callers will guess wrong and not know about it. It is easier/safer to change the object itself and return nothing.  Provided ToUpper/ToLower() functions (not methods!) that preserve their paramter and return a new object for callers that need the old functionality. These functions are a good candidate for __attribute__ hacks.	Alex Rousskov	9 years ago
12808		Polished for the official review, addressing several TODOs. Polished for the official review, addressing several TODOs. Use 1.1 version for FTP ports because FTP commands are sent on a "persistent by default" connection, just like HTTP/1.1. Cleaned up Ftp::CtrlChannel, Ftp::DataChannel, and Ftp::Client object construction and destruction. Do not insist on USER command when intercepting FTP. Interception support may still not work for other reasons, but USER does not seem to be required since Squid gets request destination from the intercepted connection info.	Alex Rousskov	9 years ago
12807		Done moving FTP code around. Polished moved code to match ne... Done moving FTP code around. Polished moved code to match new locations. src/clients/FtpNative is now src/clients/FtpRelay. It was awkward to use a non-noun "Native" as a class name, and the class was already described as "relaying FTP".	Alex Rousskov	9 years ago
12806		Major source layout change: Moved FTP code into servers/, cl... Major source layout change: Moved FTP code into servers/, clients/, and ftp/.   src/servers/FtpServer.*   # new FTP server, relaying FTP   src/servers/HttpServer.*  # old ConnStateData parts conflicting w/ FtpServer   src/clients/FtpClient.*   # code shared by old and new FTP clients   src/clients/FtpGateway.*  # old FTP client, translating back to HTTP   src/clients/FtpNative.*   # new FTP client, relaying FTP   src/ftp/*                 # FTP stuff shared by clients and servers This change attempts to preserve code in moved files to the extent possible: Only copied or added code was polished. Future changes will polish moved code. Unfortunately, bzr does not track code changes across file splits, so the code moved from ConnStateData (client_side.cc) into HttpServer and FtpServer classes appears as removed and added in bzr diff. If you want to see the branch log for the FtpServer or HttpServer code added in this revision, look in the previous branch revision of client_side.cc.	Alex Rousskov	9 years ago
12805		Fixed (missing DOC_END) and polished ftp_port documentation. Fixed (missing DOC_END) and polished ftp_port documentation.	Alex Rousskov	9 years ago
12804		Moved ftpAccept outside the irrelevant #if USE_OPENSSL claus... Moved ftpAccept outside the irrelevant #if USE_OPENSSL clause.	Alex Rousskov	9 years ago
12803		Merged from trunk (r13515). Merged from trunk (r13515). Needs more work to handle FTP adaptation failures better.	Alex Rousskov	9 years ago
12802		Fix assertion on bodyPipe != NULL inside ConnStateData::hand... Fix assertion on bodyPipe != NULL inside ConnStateData::handleFtpRequestData() ftp-native branch hits this assertion when ICAP server respond with error on a ftp request. This patch closes the data connection on these cases and also add a check in ConnStateData::clientReadFtpData to avoid calling handleFtpRequestData when the bodyPipe is gone for a reason.	Christos Tsantilas	9 years ago
12801		merge from trunk r13423 merge from trunk r13423	Christos Tsantilas	9 years ago
12800		Fix Ftp::Gateway::ServerStateData::savedReply initialization... Fix Ftp::Gateway::ServerStateData::savedReply initialization inside Ftp::Gateway::ServerStateData::ServerStateData constructor Ftp::Gateway::ServerStateData::savedReply::messages is a wordlist not a boolean	Christos Tsantilas	9 years ago
12799		Revised ftp-gw timeout handling to cope with very long data ... Revised ftp-gw timeout handling to cope with very long data downloads/uploads that triggered bogus ctrl connection timeouts due to ctrl channel inactivity. Unset ctrl timeout when we are done waiting for the ctrl response (ServerStateData::readControlReply). Removed code setting data timeout from Ftp::ServerStateData::dataRead() because the data timeout is set in ServerStateData::maybeReadVirginBody() that dataRead calls to read data. Removed switchTimeoutToDataChannel() from Ftp::Gateway::ServerStateData::startDataDownload because   * ctrl timeout should be cleared when we are done waiting for the ctrl     response (ServerStateData::readControlReply) and   * data timeout should be set when we start waiting for the data     (ServerStateData::maybeReadVirginBody) Unset channel timeout in FtpChannel::forget(). Same as ftp-gw r12798.	Alex Rousskov	9 years ago
12798		Merged from trunk (r13356). Merged from trunk (r13356). Needs more work to handle FTP adaptation failures better.	Alex Rousskov	9 years ago
12797		FEAT response with excessive whitespace FEAT response with excessive whitespace Some broken Microsoft FTP servers seem to be sending the following response to the FEAT command: 211-FEAT ....SIZE ....MDTM 211 END A single dot above represents a single space character, so instead of one space character they send 4 (four) space characters at the beginning of the internal FEAT lines. When this happens, the FTP client who sent the FEAT request to Squid gets nothing; its connection hangs. The problem is inside FtpHandleFeatReply function while parses the commands  list. Arguments corresponding to an emty string passes to String::substr call (e->value.substr(beg-raw, end-raw) call) causing failure to a Must clause. This patch:   - fixes and adds checks to avoid processing lines which does not include a    command.  - try to detect extra spaces before commands  - try to add the same number of spaces before commands inserted by squid     to FEAT commands list (EPSV and EPRT commands)	Christos Tsantilas	10 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1