-
Notifications
You must be signed in to change notification settings - Fork 1.1k
periodic pull failure #461
Copy link
Copy link
Closed
Labels
Milestone
Description
Pulling periodically produces null because it deserializes empty json object into class with null fields. Here is the tcp dump of flow
HTTP/1.1 200 OK
Content-Type: application/json
Server: Docker/1.8.2-fc22 (linux)
Date: Fri, 12 Feb 2016 15:13:26 GMT
Transfer-Encoding: chunked
4f
{"stream":"Trying to pull repository docker.io/kostyasha/jenkins-slave ... "}
43
{"status":"Pulling from kostyasha/jenkins-slave","id":"jdk-wget"}
43
{"status":"Already exists","progressDetail":{},"id":"f1b10cd84249"}
43
{"status":"Already exists","progressDetail":{},"id":"172633e38420"}
43
{"status":"Already exists","progressDetail":{},"id":"fab4b1df8eb1"}
43
{"status":"Already exists","progressDetail":{},"id":"cd8265f6bf77"}
43
{"status":"Already exists","progressDetail":{},"id":"2f4d0019b032"}
43
{"status":"Already exists","progressDetail":{},"id":"c5ed3ebafd2e"}
5e
{"status":"Digest: sha256:fc90780df68dd345db58a39900174ae98c47332aa7a33361966d432493aae825"}
59
{"status":"Status: Image is up to date for docker.io/kostyasha/jenkins-slave:jdk-wget"}
4
{}
0
So the last json entry is {}, but
https://github.com/docker-java/docker-java/blob/master/src/main/java/com/github/dockerjava/core/async/JsonStreamProcessor.java#L39-L45
continue parsing.
According to java debugger this token is START_OBJECT
@marcuslinke should it be also excluded?
Reactions are currently unavailable