Ochan Kwon comments

Results 8 comments of


                                            Ochan Kwon

memleak during multiple file upload when authentication is required

https://github.com/studio-b12/gowebdav/blob/425530b55ef4712a38177d133693c7601824c8dd/requests.go#L16 TeeReader requires as much memory as the requested file size. This code needs to be improved to avoid out of memory when uploading large-size files.

http2: add a MinConcurrentConns field to Transport for improving throughput

@googlebot I signed it!

Ability to sort images by creation date

I also really really want this feature.

exec stdout w/o newlines breaks health checks by logging on closed pipe

> Did you have more than one failed health check with this error? With a 10 sec poll and a 30 sec TTL it should take more than a single...

exec stdout w/o newlines breaks health checks by logging on closed pipe

I got the debug logs concerning the error. It occurred 26 hours after the service started. According to the logs, the logger was not closed. It was broken due to...

exec stdout w/o newlines breaks health checks by logging on closed pipe

I found that Logrus 1.0 also has the same issue related to token size in [here](https://github.com/sirupsen/logrus/issues/564). I think Logrus 1.0 wouldn't be safe from this error.

exec stdout w/o newlines breaks health checks by logging on closed pipe

I think I figured out what caused it. The output string of the health check script did not contain a new line character at the end of the string, so...

exec stdout w/o newlines breaks health checks by logging on closed pipe

I confirmed that there was no error with a new line character in the health check script. You can easily reproduce the error with the below health check script. ```...