🚀 go-pugleaf

Medyk wrote:
> Mam problem z timeout'em
> Jest skrypt, który jest wywoływany przez cron'a, który ma pobrać czy po 
> prostu odpalić stronę www.
> Przykładowo:
> 
> resp, data = Net::HTTP.new('www.jakis-tam.com').get('/')
> 

Musisz złapać fakt samego timeoutu:

begin
   timeout(600) do
      resp, data = Net::HTTP.new('www.jakis-tam.com').get('/')
   end
rescue TimeoutError
   # cokolwiek	
end

lopex

Marcin Mielżyński wrote:
> Medyk wrote:
>> Mam problem z timeout'em
>> Jest skrypt, który jest wywoływany przez cron'a, który ma pobrać czy 
>> po prostu odpalić stronę www.
>> Przykładowo:
>>
>> resp, data = Net::HTTP.new('www.jakis-tam.com').get('/')
>>
> 
> Musisz złapać fakt samego timeoutu:
> 
> begin
>   timeout(600) do
>      resp, data = Net::HTTP.new('www.jakis-tam.com').get('/')
>   end
> rescue TimeoutError
>   # cokolwiek   
> end
> 
> lopex

Co masz dokładnie na myśli?
Mi nie chodzi o wyłapanie tego błędu - tylko o to by dać taki timeout by 
strona się w pełni odpaliła, czyli by tego błędu nie było (innymi słowy 
by był, kiedy faktycznie czas ładowania jest niepokojący np. po 10 
minutach ale nie po 30 sekundach czy 2 minutach).
Jeśli domyślnie Ruby przerywa operację po 30 sekundach ja chcę by w tym 
przypadku przerwał ją po 10 minutach..
U mnie wygląda na to, że timeout się wywawala po 2-3 minutach, mimo 
następującego zapisu (który jak ja rozumiem, powinien pozwalać na 
odpalanie wewnętrznego bloku przez maksimum 10 minut):

Timeout::timeout(600) {
     resp, data = Net::HTTP.new('www.jakis-tam.com').get('/')
}


-- 
Medyk
http://www.medikoo.com/

Medyk wrote:

> 
> Co masz dokładnie na myśli?
> Mi nie chodzi o wyłapanie tego błędu - tylko o to by dać taki timeout by 
> strona się w pełni odpaliła, czyli by tego błędu nie było (innymi słowy 
> by był, kiedy faktycznie czas ładowania jest niepokojący np. po 10 
> minutach ale nie po 30 sekundach czy 2 minutach).

Eh, zmyliłeś mnie tym timeout - to wogóle nie do tego służy


require 'net/http'

h = Net::HTTP.new("www.onet.pl")
h.read_timeout = 600
puts h.get('/').body



lopex

Marcin Mielżyński wrote:
> Eh, zmyliłeś mnie tym timeout - to wogóle nie do tego służy
> 
> 
> require 'net/http'
> 
> h = Net::HTTP.new("www.onet.pl")
> h.read_timeout = 600
> puts h.get('/').body
> 

Dzięki! wygląda na to, że działa :)
Nie mniej nie bardzo rozumiem Twoje tłumaczenie, do czego jeśli nie do 
tego w takim razie służy ten blok:

Timeout::timeout(s) { }

Dokumentacja mówi:
Executes the method‘s block. If the block execution terminates before 
sec seconds has passed, it returns true. If not, it terminates the 
execution and raises exception (which defaults to Timeout::Error).

Co jak rozumiem powinno dawać to czego oczekiwałem, pozwolenie na 
wykonywanie się wewnętrznego kodu do czasu s.
Rozumiem, że błąd u mnie polegał na tym, że wsadzenie kodu w blok 
Timeout nie miało wpływu na wewnętrzny timeout obiektu Net::HTTP, który 
jedynie mogłem ustawić poprzez właściwość read_timeout. Dobrze rozumuję?

-- 
Medyk
http://www.medikoo.com/

Medyk wrote:

> Nie mniej nie bardzo rozumiem Twoje tłumaczenie, do czego jeśli nie do 
> tego w takim razie służy ten blok:

trochę zamieszałem, przyznaję się ;)

> Co jak rozumiem powinno dawać to czego oczekiwałem, pozwolenie na 
> wykonywanie się wewnętrznego kodu do czasu s.
> Rozumiem, że błąd u mnie polegał na tym, że wsadzenie kodu w blok 
> Timeout nie miało wpływu na wewnętrzny timeout obiektu Net::HTTP, który 
> jedynie mogłem ustawić poprzez właściwość read_timeout. Dobrze rozumuję?
> 

Tak, get robi to samo trochę niżej dla socketa, a jeszcze niżej w 
'protocol.rb':

     def rbuf_fill
       timeout(@read_timeout) {
         @rbuf << @io.sysread(1024)
       }
     end


lopex