Skip to content

Commit

Permalink
Added summary of #1 (#8)
Browse files Browse the repository at this point in the history
* Added summary of #1

* Update http-client-meta.md
  • Loading branch information
Nyholm authored Sep 13, 2016
1 parent 431feca commit e16988c
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions proposed/http-client/http-client-meta.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,19 @@ client will allow libraries to be decoupled from an implementation such as Guzzl
specify the default behaviours.
* The purpose is not to be opinionated about the use of middlewares (PSR-15).

## Approaches

### Exceptions

Our the domain exceptions `NetworkException`, `RequestException` and `HttpException` define
a contract very similar to eachother. The choosen approach is to not let them extend each other
because inheritance does not make sense in the domain model. A `RequestException` is not a
`NetworkException`.

Allowing exception to extend a `RequestAwareException` and/or `ResponseAwareException` interface
has been discussed but that is a convenience shortcut that one should not take. One should rather
catch the specific exceptions and handle them accordingly.

## People

### 5.1 Editor
Expand Down

0 comments on commit e16988c

Please sign in to comment.