Merge #472

472: feat(http): Distinct app and internal error codes in the handleError function r=48d90782 a=48d90782

This PR introduces distinct error codes for the app and internal RR errors.
Errors: 
```go
roadrunner.ErrNoAssociatedPool
roadrunner.ErrAllocateWorker
roadrunner.ErrWorkerNotReady
roadrunner.ErrEmptyPayload
roadrunner.ErrPoolStopped
roadrunner.ErrWorkerAllocateTimeout
roadrunner.ErrAllWorkersAreDead
```
now associated with the internal error codes. All other errors are application errors.
Some types of errors are impossible to distinguish in the RR1, for example `json.Unmarshall` internal error or similar.
The `.rr.yaml` now contain two more options for the errors.
```yaml
http:
    internalErrorCode: 502,
    appErrorCode: 502
```
Default behavior unchanged (500 error code as before), but now might be overridden.
closes #471 

Co-authored-by: Valery Piashchynski <[email protected]>

2 files changed, 5 insertions, 3 deletions

diff --git a/service/env/config_test.go b/service/env/config_test.go
index a526990d..cc2bdf97 100644
--- a/service/env/config_test.go
+++ b/service/env/config_test.go
@@ -1,15 +1,16 @@
 package env
 
 import (
+	"testing"
+
 	json "github.com/json-iterator/go"
 	"github.com/spiral/roadrunner/service"
 	"github.com/stretchr/testify/assert"
-	"testing"
 )
 
 type mockCfg struct{ cfg string }
 
-func (cfg *mockCfg) Get(name string) service.Config  { return nil }
+func (cfg *mockCfg) Get(name string) service.Config { return nil }
 func (cfg *mockCfg) Unmarshal(out interface{}) error {
 	j := json.ConfigCompatibleWithStandardLibrary
 	return j.Unmarshal([]byte(cfg.cfg), out)
diff --git a/service/env/service_test.go b/service/env/service_test.go
index 19cc03c7..a354214c 100644
--- a/service/env/service_test.go
+++ b/service/env/service_test.go
@@ -1,8 +1,9 @@
 package env
 
 import (
-	"github.com/stretchr/testify/assert"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func Test_NewService(t *testing.T) {