Project

General

Profile

Bug #223

Net::FTP#sendport incompatible with Mathn library

Added by NoKarma (Arthur Schreiber) about 12 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
ruby -v:
[ruby-core:17557]

Description

=begin
The Mathn library modifies the #/ Operator to return a Rational instead of an Integer. But Net::FTP#sendport relies on getting an Integer and not a Rational value. Using #div instead of #/ for division fixes this error.

@ftp.send(:sendport, "192.168.0.1", 1234)

writes "PORT 192,168,0,1,4,210\r\n" to the socket

require "mathn"
@ftp.send(:sendport, "192.168.0.1", 1234)

writes "PORT 192,168,0,1,617/128,210\r\n" to the socket.
=end


Files

net_ftp_sendport_fix.patch (430 Bytes) net_ftp_sendport_fix.patch NoKarma (Arthur Schreiber), 07/04/2008 02:21 AM
#1

Updated by nobu (Nobuyoshi Nakada) about 12 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

=begin
Applied in changeset r17868.
=end

Also available in: Atom PDF