micky 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ecea094f30e0551b84949260cd3c78b34673cb69bcefdcd746b506938adffeab
-  data.tar.gz: 98874aec481b78fdb48fe0ae50248ec51683e299b3a6c6770706a706b6266894
+  metadata.gz: bfd554068deaa3274c2da419bde9eb75ab57a766990237c5ca80480acab41f14
+  data.tar.gz: 2457466811bcc288ea57b1cd3e90eb3e5425eefb3e5e409cae6eb7e51420723e
 SHA512:
-  metadata.gz: 6a0f768ceb90965a4203f30c1d68de2a82f328f3fa5cbf18a68ebe94f54c0018cd2df1c2ef5399af64bae8221868368407bd98d60cbdf78beb59c31ee80a7ae7
-  data.tar.gz: a277768598966377bd039e5a4df2340ac679d10728d9b913b45f3882159b1812d8eccf768424a43b6f4b4244a9b2e57c400a09b4dc685d2ab1eda56040a66f0f
+  metadata.gz: 8a134919cf3a80ac8014f62c0cf48fca4519d3a680af289c8ae06e01f11e39a0279e571c4d0ee9dd6d4e30008d7cdc4190fff4d45a8f88195fdd1a365ff04615
+  data.tar.gz: 866941a834365dcb0d3d3dff0f37228e52a7412299d13ee8867329aa688025b103fc625e0f12765ab3184b1e10f2bc9a79d24b9e664c4c2dd8dd459b6b1a6058

data/lib/micky/version.rb

@@ -1,3 +1,3 @@
 module Micky
-  VERSION = '0.8.1'
+  VERSION = '0.8.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: micky
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Rafael Masson
@@ -88,21 +88,14 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/workflows/ci.yml"
 - LICENSE.txt
 - README.md
-- Rakefile
 - lib/micky.rb
 - lib/micky/errors.rb
 - lib/micky/request.rb
 - lib/micky/response.rb
 - lib/micky/uri.rb
 - lib/micky/version.rb
-- test/max_response_size_test.rb
-- test/redirect_test.rb
-- test/test_helper.rb
-- test/total_timeout_test.rb
-- test/uri_test.rb
 homepage: https://github.com/rafbm/micky
 licenses:
 - MIT

data/.github/workflows/ci.yml

@@ -1,20 +0,0 @@
-name: CI
-
-on:
-  push:
-  pull_request:
-
-jobs:
-  test:
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false
-      matrix:
-        ruby: ['3.3', '3.4', '4.0']
-    steps:
-      - uses: actions/checkout@v4
-      - uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: ${{ matrix.ruby }}
-          bundler-cache: true
-      - run: bundle exec rake test

data/Rakefile

@@ -1,9 +0,0 @@
-require 'bundler/gem_tasks'
-require 'rake/testtask'
-
-Rake::TestTask.new(:test) do |t|
-  t.libs << 'test'
-  t.pattern = 'test/*_test.rb'
-end
-
-task default: :test

data/test/max_response_size_test.rb

@@ -1,89 +0,0 @@
-require 'test_helper'
-
-describe 'Micky max_response_size' do
-  it 'reads the whole body when the option is not set' do
-    response = Micky.get(server.url('/ok'))
-    assert_equal 1000, response.body.bytesize
-    refute response.truncated?
-  end
-
-  it 'leaves a body under the limit alone' do
-    response = Micky.get(server.url('/ok'), max_response_size: 100_000)
-    assert_equal 1000, response.body.bytesize
-    refute response.truncated?
-  end
-
-  it 'accepts a body exactly at the limit' do
-    response = Micky.get(server.url('/ok'), max_response_size: 1000)
-    assert_equal 1000, response.body.bytesize
-    refute response.truncated?
-  end
-
-  describe 'when the server sends more than the limit' do
-    let(:limit) { 50_000 }
-
-    it 'returns nil' do
-      assert_nil Micky.get(server.url('/flood'), max_response_size: limit)
-    end
-
-    it 'raises Micky::TooLargeResponse when asked to raise' do
-      error = assert_raises(Micky::TooLargeResponse) do
-        Micky.get(server.url('/flood'), max_response_size: limit, raise_errors: true)
-      end
-      assert_equal 'Response larger than 50000 bytes', error.message
-    end
-
-    it 'gives up as soon as it has read the limit, rather than draining the server' do
-      # Unbounded, this reads a gigabyte
-      assert_operator elapsed { Micky.get(server.url('/flood'), max_response_size: limit) }, :<, 1.0
-    end
-
-    describe 'with truncate: true' do
-      let(:response) { Micky.get(server.url('/flood'), max_response_size: limit, truncate: true) }
-
-      it 'returns the first bytes up to the limit' do
-        assert_equal limit, response.body.bytesize
-        assert response.truncated?
-      end
-
-      it 'does not resume reading when the body is asked for again' do
-        response.body
-        assert_equal limit, response.body.bytesize
-      end
-
-      it 'reports a content length that matches what it holds' do
-        assert_equal limit, response.content_length
-      end
-
-      it 'returns as soon as it has enough, rather than draining the server' do
-        assert_operator elapsed { response }, :<, 1.0
-      end
-
-      it 'counts what it keeps, not what came off the wire, when Net::HTTP decompresses' do
-        response = Micky.get(server.url('/gzipped-flood'), max_response_size: limit, truncate: true)
-        assert_equal limit, response.body.bytesize
-        assert response.body.start_with?('xxx')
-        assert response.truncated?
-      end
-    end
-  end
-
-  it 'caps a redirect body without treating it as too large' do
-    # The 302 itself carries a gigabyte; only the hop after it is a real response
-    time = elapsed {
-      response = Micky.get(server.url('/fat-redirect'), max_response_size: 50_000)
-      assert_equal 1000, response.body.bytesize
-      refute response.truncated?
-    }
-    assert_operator time, :<, 2.0
-  end
-
-  it 'keeps an empty body empty rather than nil' do
-    assert_equal '', Micky.get(server.url('/chunked-empty')).body
-    assert_equal '', Micky.get(server.url('/chunked-empty'), max_response_size: 100).body
-  end
-
-  it 'keeps a HEAD body nil' do
-    assert_nil Micky.head(server.url('/ok'), max_response_size: 100).body
-  end
-end

data/test/redirect_test.rb

@@ -1,22 +0,0 @@
-require 'test_helper'
-
-describe 'Micky redirects' do
-  # The test server listens on a random port, so a redirect that loses the port
-  # goes to 127.0.0.1:80 and fails
-  it 'keeps the port when the Location is host-relative' do
-    response = Micky.get(server.url('/host-relative-redirect'))
-    assert_equal 1000, response.body.bytesize
-  end
-
-  it 'keeps the port when the Location is path-relative' do
-    response = Micky.get(server.url('/dir/path-relative-redirect'))
-    assert_equal 1000, response.body.bytesize
-  end
-
-  it 'redirects to the resolved URL' do
-    log = StringIO.new
-    Micky.logger = Logger.new(log, level: Logger::DEBUG)
-    Micky.get(server.url('/host-relative-redirect'))
-    assert_includes log.string, "302 redirect to #{server.url('/ok')}"
-  end
-end

data/test/test_helper.rb

@@ -1,115 +0,0 @@
-require 'minitest/autorun'
-require 'micky'
-require 'socket'
-require 'logger'
-
-# WebMock and friends intercept above Net::HTTP’s body reading, so they can’t
-# test how many bytes come off the socket or how long we wait. This serves real
-# HTTP over a real socket instead.
-class TestServer
-  attr_reader :port
-
-  RESPONSES = {
-    # A small, ordinary response
-    '/ok' => ->(socket) {
-      body = 'hello' * 200
-      socket.write "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nContent-Length: #{body.bytesize}\r\n\r\n"
-      socket.write body
-    },
-    '/dir/ok' => ->(socket) { RESPONSES['/ok'].call(socket) },
-    # A 200 whose chunked body is empty; Net::HTTP yields no chunk for it
-    '/chunked-empty' => ->(socket) {
-      socket.write "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nTransfer-Encoding: chunked\r\n\r\n0\r\n\r\n"
-    },
-    # Redirects with a host-relative Location
-    '/host-relative-redirect' => ->(socket) {
-      socket.write "HTTP/1.1 302 Found\r\nLocation: /ok\r\nContent-Length: 0\r\n\r\n"
-    },
-    # Redirects with a path-relative Location, to /dir/ok
-    '/dir/path-relative-redirect' => ->(socket) {
-      socket.write "HTTP/1.1 302 Found\r\nLocation: ok\r\nContent-Length: 0\r\n\r\n"
-    },
-    # Streams for as long as anyone reads
-    '/flood' => ->(socket) {
-      socket.write "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nContent-Length: 1000000000\r\n\r\n"
-      loop { socket.write('x' * 8192) }
-    },
-    # One byte at a time, never idle long enough to trip a per-read timeout
-    '/drip' => ->(socket) {
-      socket.write "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nContent-Length: 100000\r\n\r\n"
-      loop { socket.write('x'); socket.flush; sleep 0.2 }
-    },
-    # A redirect that itself carries a huge body
-    '/fat-redirect' => ->(socket) {
-      socket.write "HTTP/1.1 302 Found\r\nLocation: /ok\r\nContent-Length: 1000000000\r\n\r\n"
-      loop { socket.write('x' * 8192) }
-    },
-    # gzip, so Net::HTTP’s transparent decoding is in the path
-    '/gzipped-flood' => ->(socket) {
-      require 'zlib'
-      socket.write "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\nContent-Encoding: gzip\r\n\r\n"
-      writer = Zlib::GzipWriter.new(socket)
-      loop { writer.write('x' * 8192) }
-    },
-  }
-
-  def initialize
-    @server = TCPServer.new('127.0.0.1', 0)
-    @port = @server.addr[1]
-    @thread = Thread.new { accept_loop }
-    @thread.abort_on_exception = false
-  end
-
-  def url(path) = "http://127.0.0.1:#{@port}#{path}"
-
-  def stop
-    @thread.kill
-    @server.close
-  end
-
-private
-
-  def accept_loop
-    loop do
-      socket = @server.accept
-      Thread.start(socket) do |s|
-        begin
-          path = s.gets.to_s.split(' ')[1]
-          while (line = s.gets) && line != "\r\n"; end
-          RESPONSES.fetch(path, ->(so) { so.write("HTTP/1.1 404 Not Found\r\nContent-Length: 0\r\n\r\n") }).call(s)
-        rescue StandardError, Timeout::Error
-          # The client hung up, as most of these specs expect
-        ensure
-          s.close rescue nil
-        end
-      end
-    end
-  end
-end
-
-class MickyTest < Minitest::Spec
-  def self.server
-    @server ||= begin
-      server = TestServer.new
-      Minitest.after_run { server.stop }
-      server
-    end
-  end
-
-  def server = self.class.server
-
-  def setup
-    Micky::DEFAULTS.each { |key, value| Micky.public_send("#{key}=", value) }
-    Micky.logger = Logger.new(nil)
-    Micky.skip_resolve = true
-  end
-
-  def elapsed
-    t0 = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-    yield
-    Process.clock_gettime(Process::CLOCK_MONOTONIC) - t0
-  end
-end
-
-# Every `describe` block inherits the server and the defaults reset above
-Minitest::Spec.register_spec_type(//, MickyTest)

data/test/total_timeout_test.rb

@@ -1,44 +0,0 @@
-require 'test_helper'
-
-describe 'Micky total_timeout' do
-  it 'defaults to 20 seconds' do
-    assert_equal 20, Micky.total_timeout
-  end
-
-  it 'leaves a fast response alone' do
-    assert_equal 1000, Micky.get(server.url('/ok'), total_timeout: 5).body.bytesize
-  end
-
-  describe 'against a server that drips bytes slower than it is read' do
-    # Every read succeeds, so `timeout` never fires however low it is set
-    let(:opts) { { timeout: 5, total_timeout: 1 } }
-
-    it 'gives up at the deadline' do
-      assert_in_delta 1, elapsed { Micky.get(server.url('/drip'), **opts) }, 0.5
-    end
-
-    it 'returns nil by default' do
-      assert_nil Micky.get(server.url('/drip'), **opts)
-    end
-
-    it 'raises Micky::TotalTimeout when asked to raise' do
-      assert_raises(Micky::TotalTimeout) do
-        Micky.get(server.url('/drip'), raise_errors: true, **opts)
-      end
-    end
-
-    it 'never returns without the deadline, which is what makes it necessary' do
-      assert_raises(Timeout::Error) do
-        Timeout.timeout(2) { Micky.get(server.url('/drip'), timeout: 5, total_timeout: nil) }
-      end
-    end
-  end
-
-  it 'does not relabel an enclosing block’s expiry as its own' do
-    assert_raises(ArgumentError) do
-      Timeout.timeout(0.2, ArgumentError) do
-        Micky.get(server.url('/drip'), timeout: 5, total_timeout: 30)
-      end
-    end
-  end
-end

data/test/uri_test.rb

@@ -1,26 +0,0 @@
-require 'test_helper'
-
-describe 'Micky.URI' do
-  # Ruby only prints the deprecation under $VERBOSE, so turn it on to see it
-  def without_warnings
-    verbose, $VERBOSE = $VERBOSE, true
-    _, err = capture_io { yield }
-    err
-  ensure
-    $VERBOSE = verbose
-  end
-
-  it 'escapes a URL that ::URI.parse rejects' do
-    uri = nil
-    err = without_warnings { uri = Micky.URI('http://example.com/a b?x=1|2') }
-    assert_equal 'http://example.com/a%20b?x=1%7C2', uri.to_s
-    assert_empty err
-  end
-
-  it 'extracts http URLs from text' do
-    urls = nil
-    err = without_warnings { urls = Micky::URI.extract('see http://a.com and ftp://c.com') }
-    assert_equal ['http://a.com'], urls
-    assert_empty err
-  end
-end