.gitignore

@@ -1,4 +1,3 @@
 *.so
 
 _docs/
-vls.log

.woodpecker/lint.yml

@@ -1,6 +1,3 @@
-variables:
-  - &vlang_image 'git.rustybever.be/chewing_bever/vlang:0.3.3-alpine3.17'
-
 branches:
   exclude: [ main ]
 
@@ -8,14 +5,14 @@ platform: 'linux/amd64'
 
 pipeline:
   build:
-    image: *vlang_image
+    image: 'git.rustybever.be/chewing_bever/vlang:0.3.2'
     commands:
       - make
     when:
       event: [ push ]
 
   lint:
-    image: *vlang_image
+    image: 'git.rustybever.be/chewing_bever/vlang:0.3.2'
     commands:
       - make lint
     when:

Makefile

@@ -1,6 +1,6 @@
 # =====CONFIG=====
 V_PATH ?= v
-V := $(V_PATH) -showcc -d use_openssl
+V := $(V_PATH) -showcc
 
 all: vdocker
 
@@ -10,10 +10,6 @@ all: vdocker
 vdocker:
 	$(V) -g -shared .
 
-.PHONY: c
-c:
-	$(V) -o docker.c .
-
 
 # =====DOCS=====
 .PHONY: api-docs

ROADMAP.md

@@ -20,7 +20,7 @@ reference](https://docs.docker.com/engine/api/v1.41/).
     - [ ] Start a container
     - [ ] Stop a container
     - [ ] Restart a container
-    - [x] Kill a container
+    - [ ] Kill a container
     - [ ] Update a container
     - [ ] Rename a container
     - [ ] Pause a container
@@ -42,7 +42,7 @@ reference](https://docs.docker.com/engine/api/v1.41/).
     - [ ] Inspect an image
     - [ ] Get the history of an image
     - [ ] Push an image
-    - [x] Tag an image
+    - [ ] Tag an image
     - [ ] Remove an image
     - [ ] Search images
     - [ ] Delete unused images

containers.v

@@ -1,22 +1,81 @@
 module docker
 
+import json
 import time
-import types { ContainerListItem }
+import net.http { Method }
 
-[params]
-pub struct ContainerListConfig {
-	all     bool
-	limit   int
-	size    bool
-	filters map[string][]string
+pub struct Port {
+	ip           string [json: IP]
+	private_port u16    [json: PrivatePort]
+	public_port  u16    [json: PublicPort]
+	type_        string [json: Type]
 }
 
-pub fn (mut d DockerConn) container_list(c ContainerListConfig) ![]ContainerListItem {
-	d.request(.get, '/containers/json')
-	d.params(c)
-	d.send()!
+pub struct HostConfig {
+	network_mode string [json: NetworkMode]
+}
 
-	return d.read_json_response[[]ContainerListItem]()
+pub struct EndpointIpamConfig {
+	ipv4_address   string   [json: IPv4Address]
+	ipv6_address   string   [json: IPv6Address]
+	link_local_ips []string [json: LinkLocalIPs]
+}
+
+pub struct EndpointSettings {
+	ipam_config            EndpointIpamConfig [json: IPAMConfig]
+	links                  []string           [json: Links]
+	aliases                []string           [json: Aliases]
+	network_id             string             [json: NetworkID]
+	endpoint_id            string             [json: EndpointID]
+	gateway                string             [json: Gateway]
+	ip_address             string             [json: IPAddress]
+	ip_prefix_len          int                [json: IPPrefixLen]
+	ipv6_gateway           string             [json: IPv6Gateway]
+	global_ipv6_address    string             [json: GlobalIPv6Address]
+	global_ipv6_prefix_len i64                [json: GlobalIPv6PrefixLen]
+	mac_address            string             [json: MacAddress]
+	driver_opts            map[string]string  [json: DriverOpts]
+}
+
+pub struct NetworkSettings {
+	networks map[string]EndpointSettings [json: Networks]
+}
+
+pub struct MountPoint {
+	type_       string [json: Type]
+	name        string [json: Name]
+	source      string [json: Source]
+	destination string [json: Destination]
+	driver      string [json: Driver]
+	mode        string [json: Mode]
+	rw          bool   [json: RW]
+	propagation string [json: Propagation]
+}
+
+pub struct ContainerListItem {
+	id               string            [json: Id]
+	names            []string          [json: Names]
+	image            string            [json: Image]
+	image_id         string            [json: ImageID]
+	command          string            [json: Command]
+	created          i64               [json: Created]
+	ports            []Port            [json: Ports]
+	size_rw          i64               [json: SizeRw]
+	size_root_fs     i64               [json: SizeRootFs]
+	labels           map[string]string [json: Labels]
+	state            string            [json: State]
+	status           string            [json: Status]
+	host_config      HostConfig        [json: HostConfig]
+	network_settings NetworkSettings   [json: NetworkSettings]
+	mounts           []MountPoint      [json: Mounts]
+}
+
+pub fn (mut d DockerConn) container_list() ![]ContainerListItem {
+	d.send_request(Method.get, '/containers/json')!
+
+	data := d.read_json_response<[]ContainerListItem>()!
+
+	return data
 }
 
 pub struct NewContainer {
@@ -35,18 +94,30 @@ pub:
 }
 
 pub fn (mut d DockerConn) container_create(c NewContainer) !CreatedContainer {
-	d.request(.post, '/containers/create')
-	d.body_json(c)
-	d.send()!
+	d.send_request_with_json(Method.post, '/containers/create', c)!
+	head, res := d.read_response()!
 
-	return d.read_json_response[CreatedContainer]()
+	if head.status_code != 201 {
+		data := json.decode(DockerError, res)!
+
+		return error(data.message)
+	}
+
+	data := json.decode(CreatedContainer, res)!
+
+	return data
 }
 
 // start_container starts the container with the given id.
 pub fn (mut d DockerConn) container_start(id string) ! {
-	d.request(.post, '/containers/${id}/start')
-	d.send()!
-	d.read_response()!
+	d.send_request(Method.post, '/containers/$id/start')!
+	head, body := d.read_response()!
+
+	if head.status_code != 204 {
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
 }
 
 struct ContainerInspect {
@@ -68,10 +139,16 @@ pub mut:
 }
 
 pub fn (mut d DockerConn) container_inspect(id string) !ContainerInspect {
-	d.request(.get, '/containers/${id}/json')
-	d.send()!
+	d.send_request(Method.get, '/containers/$id/json')!
+	head, body := d.read_response()!
 
-	mut data := d.read_json_response[ContainerInspect]()!
+	if head.status_code != 200 {
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
+
+	mut data := json.decode(ContainerInspect, body)!
 
 	// The Docker engine API *should* always return UTC time.
 	data.state.start_time = time.parse_rfc3339(data.state.start_time_str)!
@@ -84,26 +161,27 @@ pub fn (mut d DockerConn) container_inspect(id string) !ContainerInspect {
 }
 
 pub fn (mut d DockerConn) container_remove(id string) ! {
-	d.request(.delete, '/containers/${id}')
-	d.send()!
-	d.read_response()!
-}
+	d.send_request(Method.delete, '/containers/$id')!
+	head, body := d.read_response()!
 
-pub fn (mut d DockerConn) container_kill(id string) ! {
-	d.request(.post, '/containers/${id}/kill')
-	d.send()!
-	d.read_response()!
+	if head.status_code != 204 {
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
 }
 
 pub fn (mut d DockerConn) container_get_logs(id string) !&StreamFormatReader {
-	d.request(.get, '/containers/${id}/logs')
-	d.params({
-		'stdout': 'true'
-		'stderr': 'true'
-	})
-	d.send()!
-	d.read_response_head()!
-	d.check_error()!
+	d.send_request(Method.get, '/containers/$id/logs?stdout=true&stderr=true')!
+	head := d.read_response_head()!
+
+	if head.status_code != 200 {
+		content_length := head.header.get(http.CommonHeader.content_length)!.int()
+		body := d.read_response_body(content_length)!
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
 
 	return d.get_stream_format_reader()
 }

docker.v

@@ -4,6 +4,7 @@ import net.unix
 import io
 import net.http
 import strings
+import net.urllib
 import json
 import util
 
@@ -16,21 +17,10 @@ const (
 	api_version          = 'v1.41'
 )
 
-[heap]
 pub struct DockerConn {
 mut:
 	socket &unix.StreamConn
 	reader &io.BufferedReader
-	// Data for the request that's currently being constructed.
-	method       http.Method
-	url          string
-	params       map[string]string
-	content_type string
-	// Before send: body of the request
-	// After send: body of response
-	body string
-	// HTTP head of the response
-	head http.Response
 }
 
 // new_conn creates a new connection to the Docker daemon.
@@ -50,71 +40,99 @@ pub fn (mut d DockerConn) close() ! {
 	d.socket.close()!
 }
 
+// send_request sends an HTTP request without body.
+fn (mut d DockerConn) send_request(method http.Method, url_str string) ! {
+	url := urllib.parse('/$docker.api_version$url_str')!
+	req := '$method $url.request_uri() HTTP/1.1\nHost: localhost\n\n'
+
+	d.socket.write_string(req)!
+
+	// When starting a new request, the reader needs to be reset.
+	d.reader = io.new_buffered_reader(reader: d.socket)
+}
+
+// send_request_with_body sends an HTTP request with the given body.
+fn (mut d DockerConn) send_request_with_body(method http.Method, url_str string, content_type string, body string) ! {
+	url := urllib.parse('/$docker.api_version$url_str')!
+	req := '$method $url.request_uri() HTTP/1.1\nHost: localhost\nContent-Type: $content_type\nContent-Length: $body.len\n\n$body\n\n'
+
+	d.socket.write_string(req)!
+
+	// When starting a new request, the reader needs to be reset.
+	d.reader = io.new_buffered_reader(reader: d.socket)
+}
+
+// send_request_with_json<T> is a convenience wrapper around
+// send_request_with_body that encodes the input as JSON.
+fn (mut d DockerConn) send_request_with_json<T>(method http.Method, url_str string, data &T) ! {
+	body := json.encode(data)
+
+	return d.send_request_with_body(method, url_str, 'application/json', body)
+}
+
 // read_response_head consumes the socket's contents until it encounters
 // '\r\n\r\n', after which it parses the response as an HTTP response.
 // Importantly, this function never consumes the reader past the HTTP
 // separator, so the body can be read fully later on.
-fn (mut d DockerConn) read_response_head() ! {
+fn (mut d DockerConn) read_response_head() !http.Response {
 	mut res := []u8{}
 
 	util.read_until_separator(mut d.reader, mut res, docker.http_separator)!
 
-	d.head = http.parse_response(res.bytestr())!
+	return http.parse_response(res.bytestr())
 }
 
-// read_response_body consumes the rest of the HTTP response and stores it as
-// the response body. This function should only be called after
-// read_response_head. This function always reads the entire response into
-// memory, even if it's chunked.
-fn (mut d DockerConn) read_response_body() ! {
-	if d.head.status() == .no_content {
-		return
+// read_response_body reads `length` bytes from the stream. It can be used when
+// the response encoding isn't chunked to fully read it.
+fn (mut d DockerConn) read_response_body(length int) !string {
+	if length == 0 {
+		return ''
 	}
 
+	mut buf := []u8{len: docker.buf_len}
+	mut c := 0
 	mut builder := strings.new_builder(docker.buf_len)
 
-	if d.head.header.get(.transfer_encoding) or { '' } == 'chunked' {
-		mut body_stream := d.get_chunked_response_reader()
+	for builder.len < length {
+		c = d.reader.read(mut buf) or { break }
 
-		util.reader_to_writer(mut body_stream, mut builder)!
-	} else {
-		content_length := d.head.header.get(.content_length)!.int()
-
-		if content_length == 0 {
-			d.body = ''
-
-			return
-		}
-
-		mut buf := []u8{len: docker.buf_len}
-		mut c := 0
-
-		for builder.len < content_length {
-			c = d.reader.read(mut buf)!
-
-			builder.write(buf[..c])!
-		}
+		builder.write(buf[..c])!
 	}
 
-	d.body = builder.str()
+	return builder.str()
 }
 
-// read_response is a convenience function that always consumes the entire
-// response and loads it into memory. It should only be used when we're certain
-// that the result isn't too large, as even chunked responses will get fully
-// loaded into memory.
-fn (mut d DockerConn) read_response() ! {
-	d.read_response_head()!
-	d.check_error()!
-	d.read_response_body()!
+// read_response is a convenience function which always consumes the entire
+// response & returns it. It should only be used when we're certain that the
+// result isn't too large.
+fn (mut d DockerConn) read_response() !(http.Response, string) {
+	head := d.read_response_head()!
+
+	if head.header.get(http.CommonHeader.transfer_encoding) or { '' } == 'chunked' {
+		mut builder := strings.new_builder(1024)
+		mut body := d.get_chunked_response_reader()
+
+		util.reader_to_writer(mut body, mut builder)!
+
+		return head, builder.str()
+	}
+
+	content_length := head.header.get(http.CommonHeader.content_length)!.int()
+	res := d.read_response_body(content_length)!
+
+	return head, res
 }
 
-// read_json_response<T> is a convenience function that runs read_response
-// before parsing its contents, which is assumed to be JSON, into a struct.
-fn (mut d DockerConn) read_json_response[T]() !T {
-	d.read_response()!
+fn (mut d DockerConn) read_json_response<T>() !T {
+	head, body := d.read_response()!
 
-	data := json.decode(T, d.body)!
+	if head.status_code < 200 || head.status_code > 300 {
+		data := json.decode(DockerError, body)!
+
+		return docker_error(head.status_code, data.message)
+	}
+
+	mut data := json.decode(T, body)!
 
 	//$for field in T.fields {
 	//$if field.typ is time.Time {
@@ -126,7 +144,7 @@ fn (mut d DockerConn) read_json_response[T]() !T {
 }
 
 // get_chunked_response_reader returns a ChunkedResponseReader using the socket
-// as reader. This function should only be called after check_error.
+// as reader.
 fn (mut d DockerConn) get_chunked_response_reader() &ChunkedResponseReader {
 	r := new_chunked_response_reader(d.reader)
 
@@ -134,28 +152,10 @@ fn (mut d DockerConn) get_chunked_response_reader() &ChunkedResponseReader {
 }
 
 // get_stream_format_reader returns a StreamFormatReader using the socket as
-// reader. This function should only be called after check_error.
+// reader.
 fn (mut d DockerConn) get_stream_format_reader() &StreamFormatReader {
 	r := new_chunked_response_reader(d.reader)
 	r2 := new_stream_format_reader(r)
 
 	return r2
 }
-
-struct DockerError {
-pub:
-	message string
-}
-
-// check_error should be called after read_response_head. If the status code of
-// the response is an error, the body is consumed and the Docker HTTP error is
-// returned as a V error. If the status isn't the error, this function is a
-// no-op, and the body can be read.
-fn (mut d DockerConn) check_error() ! {
-	if d.head.status().is_error() {
-		d.read_response_body()!
-		d_err := json.decode(DockerError, d.body)!
-
-		return error_with_code('${d.head.status()}: ${d_err.message}', d.head.status_code)
-	}
-}

errors.v

@@ -0,0 +1,21 @@
+module docker
+
+struct DockerError {
+	status  int    [skip]
+	message string
+}
+
+fn (err DockerError) code() int {
+	return err.status
+}
+
+fn (err DockerError) msg() string {
+	return err.message
+}
+
+fn docker_error(status int, message string) IError {
+	return IError(DockerError{
+		status: status
+		message: message
+	})
+}

images.v

@@ -1,26 +1,25 @@
 module docker
 
-import types { Image }
+import net.http { Method }
+import json
 
-pub fn (mut d DockerConn) image_inspect(image string) !Image {
-	d.request(.get, '/images/${image}/json')
-	d.send()!
-
-	data := d.read_json_response[Image]()!
-
-	return data
+struct Image {
+pub:
+	id string [json: Id]
 }
 
-// image_pull pulls the given image:tag.
-pub fn (mut d DockerConn) image_pull(image string, tag string) ! {
-	d.request(.post, '/images/create')
-	d.params({
-		'fromImage': image
-		'tag':       tag
-	})
-	d.send()!
-	d.read_response_head()!
-	d.check_error()!
+// pull_image pulls the given image:tag.
+pub fn (mut d DockerConn) pull_image(image string, tag string) ! {
+	d.send_request(Method.post, '/images/create?fromImage=$image&tag=$tag')!
+	head := d.read_response_head()!
+
+	if head.status_code != 200 {
+		content_length := head.header.get(http.CommonHeader.content_length)!.int()
+		body := d.read_response_body(content_length)!
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
 
 	// Keep reading the body until the pull has completed
 	mut body := d.get_chunked_response_reader()
@@ -33,32 +32,29 @@ pub fn (mut d DockerConn) image_pull(image string, tag string) ! {
 }
 
 // create_image_from_container creates a new image from a container.
-pub fn (mut d DockerConn) image_from_container(id string, repo string, tag string) !Image {
-	d.request(.post, '/commit')
-	d.params({
-		'container': id
-		'repo':      repo
-		'tag':       tag
-	})
-	d.body('application/json', '{}')
-	d.send()!
+pub fn (mut d DockerConn) create_image_from_container(id string, repo string, tag string) !Image {
+	d.send_request(Method.post, '/commit?container=$id&repo=$repo&tag=$tag')!
+	head, body := d.read_response()!
 
-	return d.read_json_response[Image]()!
+	if head.status_code != 201 {
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
+
+	data := json.decode(Image, body)!
+
+	return data
 }
 
 // remove_image removes the image with the given id.
-pub fn (mut d DockerConn) image_remove(id string) ! {
-	d.request(.delete, '/images/${id}')
-	d.send()!
-	d.read_response()!
-}
+pub fn (mut d DockerConn) remove_image(id string) ! {
+	d.send_request(Method.delete, '/images/$id')!
+	head, body := d.read_response()!
 
-pub fn (mut d DockerConn) image_tag(name string, repo string, tag string) ! {
-	d.request(.post, '/images/${name}/tag')
-	d.params({
-		'repo': repo
-		'tag':  tag
-	})
-	d.send()!
-	d.read_response()!
+	if head.status_code != 200 {
+		data := json.decode(DockerError, body)!
+
+		return error(data.message)
+	}
 }

request.v

@@ -1,71 +0,0 @@
-module docker
-
-import net.http
-import net.urllib
-import io
-import json
-
-fn (mut d DockerConn) request(method http.Method, url string) {
-	d.method = method
-	d.url = url
-	d.content_type = ''
-	d.body = ''
-
-	d.params.clear()
-}
-
-fn (mut d DockerConn) body(content_type string, body string) {
-	d.content_type = content_type
-	d.body = body
-}
-
-fn (mut d DockerConn) body_json[T](data T) {
-	d.content_type = 'application/json'
-	d.body = json.encode(data)
-}
-
-fn (mut d DockerConn) params[T](o T) {
-	$if T is map[string]string {
-		for key, value in o {
-			d.params[key] = urllib.query_escape(value.replace("'", '"'))
-		}
-	} $else {
-		$for field in T.fields {
-			v := o.$(field.name)
-
-			if !isnil(v) {
-				d.params[field.name] = urllib.query_escape(v.str().replace("'", '"'))
-			}
-		}
-	}
-}
-
-fn (mut d DockerConn) send() ! {
-	mut full_url := '/${docker.api_version}${d.url}'
-
-	if d.params.len > 0 {
-		mut fields := []string{cap: d.params.len}
-
-		for key, value in d.params {
-			fields << '${key}=${value}'
-		}
-		params_str := fields.join('&')
-		// params_str := d.params.keys().map('${it}=${d.params[it]}').join('&')
-		full_url += '?${params_str}'
-	}
-
-	// This is to make sure we actually created a valid URL
-	parsed_url := urllib.parse(full_url)!
-	final_url := parsed_url.request_uri()
-
-	req := if d.body == '' {
-		'${d.method} ${final_url} HTTP/1.1\nHost: localhost\n\n'
-	} else {
-		'${d.method} ${final_url} HTTP/1.1\nHost: localhost\nContent-Type: ${d.content_type}\nContent-Length: ${d.body.len}\n\n${d.body}\n\n'
-	}
-
-	d.socket.write_string(req)!
-
-	// When starting a new request, the reader needs to be reset.
-	d.reader = io.new_buffered_reader(reader: d.socket)
-}

types/container.v

@@ -1,111 +0,0 @@
-module types
-
-pub struct Port {
-pub:
-	ip           string [json: IP]
-	private_port u16    [json: PrivatePort]
-	public_port  u16    [json: PublicPort]
-	type_        string [json: Type]
-}
-
-pub struct HostConfig {
-pub:
-	network_mode string [json: NetworkMode]
-}
-
-pub struct EndpointIpamConfig {
-pub:
-	ipv4_address   string   [json: IPv4Address]
-	ipv6_address   string   [json: IPv6Address]
-	link_local_ips []string [json: LinkLocalIPs]
-}
-
-pub struct EndpointSettings {
-pub:
-	ipam_config            EndpointIpamConfig [json: IPAMConfig]
-	links                  []string           [json: Links]
-	aliases                []string           [json: Aliases]
-	network_id             string             [json: NetworkID]
-	endpoint_id            string             [json: EndpointID]
-	gateway                string             [json: Gateway]
-	ip_address             string             [json: IPAddress]
-	ip_prefix_len          int                [json: IPPrefixLen]
-	ipv6_gateway           string             [json: IPv6Gateway]
-	global_ipv6_address    string             [json: GlobalIPv6Address]
-	global_ipv6_prefix_len i64                [json: GlobalIPv6PrefixLen]
-	mac_address            string             [json: MacAddress]
-	driver_opts            map[string]string  [json: DriverOpts]
-}
-
-pub struct NetworkSettings {
-pub:
-	networks map[string]EndpointSettings [json: Networks]
-}
-
-pub struct MountPoint {
-pub:
-	type_       string [json: Type]
-	name        string [json: Name]
-	source      string [json: Source]
-	destination string [json: Destination]
-	driver      string [json: Driver]
-	mode        string [json: Mode]
-	rw          bool   [json: RW]
-	propagation string [json: Propagation]
-}
-
-pub struct ContainerListItem {
-pub:
-	id               string            [json: Id]
-	names            []string          [json: Names]
-	image            string            [json: Image]
-	image_id         string            [json: ImageID]
-	command          string            [json: Command]
-	created          i64               [json: Created]
-	ports            []Port            [json: Ports]
-	size_rw          i64               [json: SizeRw]
-	size_root_fs     i64               [json: SizeRootFs]
-	labels           map[string]string [json: Labels]
-	state            string            [json: State]
-	status           string            [json: Status]
-	host_config      HostConfig        [json: HostConfig]
-	network_settings NetworkSettings   [json: NetworkSettings]
-	mounts           []MountPoint      [json: Mounts]
-}
-
-pub struct HealthConfig {
-pub:
-	test         []string [json: Test]
-	interval     int      [json: Interval]
-	timeout      int      [json: Timeout]
-	retries      int      [json: Retries]
-	start_period int      [json: StartPeriod]
-}
-
-pub struct ContainerCreate {
-pub:
-	hostname      string [json: Hostname]
-	domain_name   string [json: Domainname]
-	user          string [json: User]
-	attach_stdin  bool   [json: AttachStdin]
-	attach_stdout bool   [json: AttachStderr] = true
-	// ExposedPorts
-	tty          bool         [json: Tty]
-	open_stdin   bool         [json: OpenStdin]
-	stdin_once   bool         [json: StdinOnce]
-	env          []string     [json: Env]
-	cmd          []string     [json: Cmd]
-	healthcheck  HealthConfig [json: Healthcheck]
-	args_escaped bool         [json: ArgsEscaped]
-	image        string       [json: Image]
-	// Volumes
-	working_dir      string            [json: WorkingDir]
-	entrypoint       []string          [json: Entrypoint]
-	network_disabled bool              [json: NetworkDisabled]
-	mac_address      string            [json: MacAddress]
-	on_build         []string          [json: OnBuild]
-	labels           map[string]string [json: Labels]
-	stop_signal      string            [json: StopSignal]
-	stop_timeout     int               [json: StopTimeout]
-	shell            []string          [json: Shell]
-}

types/image.v

@@ -1,6 +0,0 @@
-module types
-
-pub struct Image {
-pub:
-	id string [json: Id]
-}

types/volume.v

@@ -1,22 +0,0 @@
-module types
-
-import time
-
-pub struct UsageData {
-	size      int [json: Size]
-	ref_count int [json: RefCount]
-}
-
-pub struct Volume {
-pub mut:
-	created_at_str string            [json: CreatedAt]
-	created_at     time.Time         [skip]
-	name           string            [json: Name]
-	driver         string            [json: Driver]
-	mountpoint     string            [json: Mountpoint]
-	status         map[string]string [json: Status]
-	labels         map[string]string [json: Labels]
-	scope          string            [json: Scope]
-	options        map[string]string [json: Options]
-	usage_data     UsageData         [json: UsageData]
-}

util/util.v

@@ -21,7 +21,7 @@ pub fn reader_to_writer(mut reader io.Reader, mut writer io.Writer) ! {
 // match_array_in_array<T> returns how many elements of a2 overlap with a1. For
 // example, if a1 = "abcd" & a2 = "cd", the result will be 2. If the match is
 // not at the end of a1, the result is 0.
-pub fn match_array_in_array[T](a1 []T, a2 []T) int {
+pub fn match_array_in_array<T>(a1 []T, a2 []T) int {
 	mut i := 0
 	mut match_len := 0
 

volumes.v

@@ -1,7 +1,26 @@
 module docker
 
+import net.http { Method }
 import time
-import types { Volume }
+
+struct UsageData {
+	size      int [json: Size]
+	ref_count int [json: RefCount]
+}
+
+struct Volume {
+	created_at_str string [json: CreatedAt]
+pub mut:
+	created_at time.Time         [skip]
+	name       string            [json: Name]
+	driver     string            [json: Driver]
+	mountpoint string            [json: Mountpoint]
+	status     map[string]string [json: Status]
+	labels     map[string]string [json: Labels]
+	scope      string            [json: Scope]
+	options    map[string]string [json: Options]
+	usage_data UsageData         [json: UsageData]
+}
 
 [params]
 pub struct VolumeListFilter {
@@ -17,10 +36,9 @@ struct VolumeListResponse {
 }
 
 pub fn (mut d DockerConn) volume_list() !VolumeListResponse {
-	d.request(.get, '/volumes')
-	d.send()!
+	d.send_request(Method.get, '/volumes')!
 
-	mut data := d.read_json_response[VolumeListResponse]()!
+	mut data := d.read_json_response<VolumeListResponse>()!
 
 	for mut vol in data.volumes {
 		vol.created_at = time.parse_rfc3339(vol.created_at_str)!