È thread-safe impostare l'autenticazione HTTP Active Resource su base per utente?

Question

Active Resource può utilizzare l'autenticazione HTTP impostata a livello di classe. Per esempio:

class Resource < ActiveResource::Base 
end 

Resource.user = 'user' 
Resource.password = 'password' 

o

Resource.site = "http://user:password@site.com/" 

Ma cosa succede se io uso l'autenticazione HTTP diversa in base a quale utente è loggato? Se cambio Resource.user e Resource.password, ciò causerà una condizione di competizione in cui le richieste da un thread iniziano improvvisamente a utilizzare l'autenticazione di un utente le cui richieste sono eseguite simultaneamente in un thread diverso? Oppure si tratta di un problema (a condizione di reimpostare l'autenticazione tra le richieste) perché i server dei binari non sono multithread?

Anche se non ci sono problemi di sicurezza dei thread, sembra comunque rischioso che se non riesco a ripristinarli, le credenziali dell'utente precedente verranno utilizzate automaticamente dalle richieste future.

Aggiornamento: Dopo essere stato frustrato con ActiveResource, ho scritto la mia libreria RIPOSO: https://github.com/DeepWebTechnologies/well_rested

Answer 1

scimmia patchare il host, user e password metodi della ActiveResource::Base classe:

class ActiveResource::Base 
    # store the attribute value in a thread local variable 
    class << self 
    %w(host user password).each do |attr|    

     define_method(attr) do 
     Thread.current["active_resource.#{attr}"] 
     end 

     define_method("#{attr}=") do |val| 
     Thread.current["active_resource.#{attr}"] = val 
     end 
    end 
    end 
end 

Ora impostare il credenziali in ogni richiesta

class ApplicationController < ActionController::Base 

    around_filter :set_api_credentials 

private 

    # set the credentials in every request 
    def set_api_credentials 
    ActiveResource::Base.host, 
     ActiveResource::Base.user, 
     ActiveResource::Base.password = current_user_credentials 
    yield 
    ensure 
    ActiveResource::Base.host = 
     ActiveResource::Base.user = 
     ActiveResource::Base.password = nil 
    end 

    DEFAULT_HOST, DEFAULT_USER, DEFAULT_PASSWORD= [ 
    "http://www.foo.com", "user1", "user78102" ] 

    def current_user_credentials 
    current_user.present? ? 
     [ current_user.host, current_user.login, current_user.password] : 
     [ DEFAULT_HOST, DEFAULT_USER, DEFAULT_PASSWORD] 
    end 

end 

Answer 2

A partire da Active Resource 4.1.0, quelle impostazioni sono thread locali, quindi questo esempio non causerebbe più una condizione di competizione.

Questo è il rilevante commit se qualcuno è interessato: https://github.com/rails/activeresource/commit/538588ddba9ffc9bf356790e9186dc7e6adad12f