Skip to content

Commit

Permalink
Preven Net::FTPServer limits.
Browse files Browse the repository at this point in the history
They lead to mysterious crashes.
  • Loading branch information
theory committed Jul 29, 2010
1 parent ea4c643 commit c9fd33a
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lib/Bric/Changes.pod
Expand Up @@ -70,6 +70,12 @@ report (Bug # 184). [David]
Worked around a bug in L<Net::SSH2> where sockets weren't being closed,
leading to L<bric_queued> running out of file descriptors. [David]

=item *

The virtual FTP server no longer limits itself, but leaves resource limits to
the OS. This prevents apparently unrelated (and mysterious!) errors from the
FTP server. [Alex Krohn].

=back

=head1 Version 2.0.0 (2010-04-12)
Expand Down
11 changes: 11 additions & 0 deletions lib/Bric/Util/FTP/Server.pm
Expand Up @@ -171,6 +171,17 @@ sub post_bind_hook {
unless $> == SYS_USER;
}

=item process_limits_hook()
Overrides FTP::Server's implementation to ensure no limits are set. Better to
just let the OS set them.
=cut

sub process_limits_hook {
return 1;
}

sub find_workflow {
my ($self, $site_id) = @_;
my $user = $self->{user_obj};
Expand Down

0 comments on commit c9fd33a

Please sign in to comment.