David Cuadrado

Data races on http.DefaultTransport with rc15

15

``` WARNING: DATA RACE Read at 0x0000022e7450 by goroutine 12: net/http.(*Client).send() /usr/local/Cellar/go/1.14.1/libexec/src/net/http/client.go:176 +0x39d net/http.(*Client).do() /usr/local/Cellar/go/1.14.1/libexec/src/net/http/client.go:699 +0x2cc net/http.(*Client).Do() /usr/local/Cellar/go/1.14.1/libexec/src/net/http/client.go:567 +0x7f github.com/aws/aws-sdk-go/aws/corehandlers.sendFollowRedirects() /Users/dcuadrado/go/pkg/mod/github.com/aws/[email protected]/aws/corehandlers/handlers.go:120 +0x2f github.com/aws/aws-sdk-go/aws/corehandlers.glob..func3() /Users/dcuadrado/go/pkg/mod/github.com/aws/[email protected]/aws/corehandlers/handlers.go:112 +0xfd github.com/aws/aws-sdk-go/aws/request.(*HandlerList).Run() /Users/dcuadrado/go/pkg/mod/github.com/aws/[email protected]/aws/request/handlers.go:267 +0xd1 github.com/aws/aws-sdk-go/aws/request.(*Request).sendRequest() /Users/dcuadrado/go/pkg/mod/github.com/aws/[email protected]/aws/request/request.go:578...

Add support for Native Client

1

I'd like to use a go library in a ChromeOS app I got the following error when trying to compile: ``` go $ GOOS=nacl GOARCH=amd64p32 go build # sdk/vendor/github.com/boltdb/bolt vendor/github.com/boltdb/bolt/db.go:97:...

Check when spaces are not needed

2

I enabled this tool for my pipeline but now I'm seeing more spaces than needed like: ```go a := 1 if a == 1 { ... } ```

Support nested structs

2

The following struct is not supported: ```go type Inner struct { A string B string } type Query struct { Search string Page int PageSize int Categories []Inner `qs:"category"` }...

Sometimes you just want to check that the distance is

putting the global var in a struct that you can instantiate should fix this

Add support for retrying failed requests

1

## What would you like to be added? 429 and 5XX errors should be retried ## Why is this needed? it improves the UX otherwise the dev would have to...

### Proposal I propose exposing a chai.Row function to scan into a reflect.Value ### Motivation It makes easier to implement ORMs on top of the chai

Allow to use the package as library

3

there are some cases where using as library would be preferable than using as cli is there a reason to not allow it? thanks