Skip to content

Commit 044ac99

Browse files
author
Ahmet Alp Balkan
committed
Merge pull request #7 from jgarverick/NamedContainers
Adding support for naming a container when using CreateContainer
2 parents b8390f9 + 2b3ca06 commit 044ac99

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

Docker.DotNet/ContainerOperations.cs

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,18 +54,24 @@ public async Task<ContainerResponse> InspectContainerAsync(string id)
5454

5555
public async Task<CreateContainerResponse> CreateContainerAsync(CreateContainerParameters parameters)
5656
{
57+
IQueryString qs = null;
58+
5759
if (parameters == null)
5860
{
5961
throw new ArgumentNullException("parameters");
6062
}
6163

64+
if (!string.IsNullOrEmpty(parameters.ContainerName)) {
65+
qs = new QueryString<CreateContainerParameters>(parameters);
66+
}
67+
6268
string path = "containers/create";
6369
JsonRequestContent<Config> data = null;
6470
if (parameters.Config != null)
6571
{
6672
data = new JsonRequestContent<Config>(parameters.Config, this.Client.JsonSerializer);
6773
}
68-
DockerApiResponse response = await this.Client.MakeRequestAsync(new[] {NoSuchContainerHandler}, HttpMethod.Post, path, null, data);
74+
DockerApiResponse response = await this.Client.MakeRequestAsync(new[] {NoSuchContainerHandler}, HttpMethod.Post, path, qs, data);
6975
return this.Client.JsonSerializer.DeserializeObject<CreateContainerResponse>(response.Body);
7076
}
7177

Docker.DotNet/Models/CreateContainerParameters.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ public class CreateContainerParameters
44
{
55
public Config Config { get; set; }
66

7+
[QueryStringParameter("name", false)]
8+
public string ContainerName { get; set; }
9+
710
public CreateContainerParameters()
811
{
912
}

0 commit comments

Comments
 (0)