This is probably an edge case (it was only exposed by a separate bug (not in Tim) causing data to be posted to the wrong URL), but the factory_keys method in TargetImagesController begins by checking params[:target_image][:percent_complete].
params[:target_image] can be nil, at least in the case of bad data being passed in, causing the method to raise an undefined method[]' for nil:NilClass` exception.
This is probably an edge case (it was only exposed by a separate bug (not in Tim) causing data to be posted to the wrong URL), but the
factory_keysmethod in TargetImagesController begins by checkingparams[:target_image][:percent_complete].params[:target_image]can be nil, at least in the case of bad data being passed in, causing the method to raise anundefined method[]' for nil:NilClass` exception.