Code review comment for lp:~dooferlad/linaro-android-frontend/proxy_lava-job-info

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I don't understand all the new_style_url/old_style_url gyrations. In particular, it sure looks as if both could go via the proxy, in which case both will succeed at the http level (/api/get-lava-job-info always returns http 200) and then its a race as to which response gets displayed.

The other bits look fine.

« Back to merge proposal