Return 'Forbidden' as a status code when user is not found in admin apis

[skymountain]

admin系API で user 情報がとれないときは Forbidden を返すようにしました． lib/api/admin_user.rb ではメッセージを出してましたが Forbidden だしいらないかと思い消しました．

[chiro]

エラーメッセージはあったほうが嬉しいと思うので、

return helper.forbidden("Unknown user: #{token}") unless user

こんな感じのあれでどうでしょう。 なかったことにしてください……

[chiro]

テストのほうも直してもらえますか? たぶんこのへん です

[skymountain]

了解です．

...って今思ったんですけど，user のチェックの前に app.su? をチェックしてるので Forbidden でなくメッセージ付き Bad Request が正当っぽいんですけど，どうでしょう？

cc: @sowa-snow

[westpaddy]

こっちはBad Requestが正当だと思います

[westpaddy]

もしかしたら向こうもBad requestでよかったのかもしれません

[westpaddy]

むこう（download.rb）は

• su が存在しないユーザを指定した場合（Bad request）
• 非suが他人のユーザを指定した場合（Forbidden）

を一か所で判定しているのが混乱の元な気がします。

[skymountain]

281