Dt/oc 2204/node state error checking

chef-boneyard / opscode-pushy-server

Chef Push Jobs Server

https://docs.chef.io/push_jobs.html

Apache License 2.0

16 stars 10 forks source link

Dt/oc 2204/node state error checking #80

Closed doubt72 closed 11 years ago

doubt72 commented 11 years ago

Changes to pushy server for various error cases (i.e., when trying to access non-existent organizations or when erchef is down).

markan commented 11 years ago

Looks good. Out of curiosity are we now returning 502 generally when one of the federated services in erchef fails?

doubt72 commented 11 years ago

Specifically we're returning 502 when not able to reach the principal endpoint (which was tested by taking the whole server down). It only really affects the REST endpoints, but I think missing authz, etc., will cause 403s or 500s depending.