openactive / realtime-paged-data-exchange

OpenActive Realtime Paged Data Exchange Specification
https://www.openactive.io/realtime-paged-data-exchange/
Other
6 stars 1 forks source link

Handling of HTTP errors when harvesting a feed #81

Open ldodds opened 6 years ago

ldodds commented 6 years ago

The specification could give some indication about what consumers should do if/when HTTP errors are encountered when processing a feed, e.g:

ldodds commented 6 years ago

I see Section 5.1.4 addresses some of this already.

I'd suggest adding 500 as an option for other server side errors than just overloading/maintenance. Helps clarify with clients that there may be something to report.

Spec currently constrains implementations from not using redirects, so that isn't likely to be an option. Although I think this should be allowed.

ldodds commented 6 years ago

Decided not to specify 500 error as the expected behaviour is close to 503.

@nickevansuk how would you feel about revising this section slightly to say:

It's a minor point but gives servers a bit more flexibility. I think in practice they're likely to issue a range of 5XX errors. Pretty sure I've seen a gateway or vanilla 500 error.

nickevansuk commented 6 years ago

Agree @ldodds, 5XX errors allow more flexibility and have also seen gateways and CDNs produce 500s.