Skip to content

Commit 9e41781

Browse files
committed
Update option descriptions
1 parent e370276 commit 9e41781

1 file changed

Lines changed: 23 additions & 17 deletions

File tree

src/NServiceBus.Transport.RabbitMQ.CommandLine/SharedOptions.cs

Lines changed: 23 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ static class SharedOptions
66
{
77
const string ConnectionStringEnvironmentVariable = "RabbitMQTransport_ConnectionString";
88

9+
//Standard options
10+
911
public static Option<string> CreateConnectionStringOption()
1012
{
1113
var connectionStringOption = new Option<string>("--connectionString", "-c")
@@ -126,17 +128,19 @@ public static Option<string> CreateCertPassphraseOption()
126128
{
127129
var certPassphraseOption = new Option<string>("--certPassphrase")
128130
{
129-
Description = "The passphrase for client certificate file for when using a client certificate"
131+
Description = "The passphrase for the client certificate file when using a client certificate"
130132
};
131133

132134
return certPassphraseOption;
133135
}
134136

137+
// Options for source broker
138+
135139
public static Option<string> CreateSourceConnectionStringOption()
136140
{
137141
var connectionStringOption = new Option<string>("--sourceConnectionString")
138142
{
139-
Description = "Force this command to use the specified connection string"
143+
Description = "Force this command to use the specified connection string for the source broker"
140144
};
141145

142146
return connectionStringOption;
@@ -146,7 +150,7 @@ public static Option<string> CreateSourceConnectionStringEnvOption()
146150
{
147151
var connectionStringEnvOption = new Option<string>("--sourceConnectionStringEnv")
148152
{
149-
Description = "Specifies the environment variable where the connection string can be found. --connectionString, if specified, will take precedence over this option.",
153+
Description = "Specifies the environment variable where the connection string for the source broker can be found. --sourceConnectionString, if specified, will take precedence over this option.",
150154
DefaultValueFactory = _ => "RabbitMQTransport_Source_ConnectionString"
151155
};
152156

@@ -157,7 +161,7 @@ public static Option<string> CreateSourceManagementApiUrlOption()
157161
{
158162
var managementApiUrlOption = new Option<string>("--sourceManagementApiUrl")
159163
{
160-
Description = "Overrides the value inferred from the connection string"
164+
Description = "Overrides the value inferred from the connection string for the source broker"
161165
};
162166

163167
return managementApiUrlOption;
@@ -167,7 +171,7 @@ public static Option<string> CreateSourceManagementApiUserNameOption()
167171
{
168172
var managementApiUserNameOption = new Option<string>("--sourceManagementApiUserName")
169173
{
170-
Description = "Overrides the value inferred from the connection string. If provided, --managementApiPassword must also be provided or this option will be ignored."
174+
Description = "Overrides the value inferred from the connection string for the source broker. If provided, --sourceManagementApiPassword must also be provided or this option will be ignored."
171175
};
172176

173177
return managementApiUserNameOption;
@@ -177,7 +181,7 @@ public static Option<string> CreateSourceManagementApiPasswordOption()
177181
{
178182
var managementApiPasswordOption = new Option<string>("--sourceManagementApiPassword")
179183
{
180-
Description = "Overrides the value inferred from the connection string. If provided, --managementApiUserName must also be provided or this option will be ignored."
184+
Description = "Overrides the value inferred from the connection string for the source broker. If provided, --sourceManagementApiUserName must also be provided or this option will be ignored."
181185
};
182186

183187
return managementApiPasswordOption;
@@ -187,7 +191,7 @@ public static Option<bool> CreateSourceUseExternalAuthOption()
187191
{
188192
var useExternalAuthOption = new Option<bool>("--sourceUseExternalAuth")
189193
{
190-
Description = "Use the external authorization option when connecting to the broker",
194+
Description = "Use the external authorization option when connecting to the source broker",
191195
DefaultValueFactory = _ => false
192196
};
193197

@@ -198,7 +202,7 @@ public static Option<string> CreateSourceCertPathOption()
198202
{
199203
var certPathOption = new Option<string>("--sourceCertPath")
200204
{
201-
Description = "Set the path to the client certificate file for connecting to the broker"
205+
Description = "Set the path to the client certificate file for connecting to the source broker"
202206
};
203207

204208
return certPathOption;
@@ -208,17 +212,19 @@ public static Option<string> CreateSourceCertPassphraseOption()
208212
{
209213
var certPassphraseOption = new Option<string>("--sourceCertPassphrase")
210214
{
211-
Description = "The passphrase for client certificate file for when using a client certificate"
215+
Description = "The passphrase for the client certificate file when using a client certificate to connect to the source broker"
212216
};
213217

214218
return certPassphraseOption;
215219
}
216220

221+
// Options for destination broker
222+
217223
public static Option<string> CreateDestinationConnectionStringOption()
218224
{
219225
var connectionStringOption = new Option<string>("--destinationConnectionString")
220226
{
221-
Description = "Force this command to use the specified connection string"
227+
Description = "Force this command to use the specified connection string for the destination broker"
222228
};
223229

224230
return connectionStringOption;
@@ -228,7 +234,7 @@ public static Option<string> CreateDestinationConnectionStringEnvOption()
228234
{
229235
var connectionStringEnvOption = new Option<string>("--destinationConnectionStringEnv")
230236
{
231-
Description = "Specifies the environment variable where the connection string can be found. --connectionString, if specified, will take precedence over this option.",
237+
Description = "Specifies the environment variable where the connection string for the destination broker can be found. --destinationConnectionString, if specified, will take precedence over this option.",
232238
DefaultValueFactory = _ => "RabbitMQTransport_Destination_ConnectionString"
233239
};
234240

@@ -239,7 +245,7 @@ public static Option<string> CreateDestinationManagementApiUrlOption()
239245
{
240246
var managementApiUrlOption = new Option<string>("--destinationManagementApiUrl")
241247
{
242-
Description = "Overrides the value inferred from the connection string"
248+
Description = "Overrides the value inferred from the connection string for the destination broker"
243249
};
244250

245251
return managementApiUrlOption;
@@ -249,7 +255,7 @@ public static Option<string> CreateDestinationManagementApiUserNameOption()
249255
{
250256
var managementApiUserNameOption = new Option<string>("--destinationManagementApiUserName")
251257
{
252-
Description = "Overrides the value inferred from the connection string. If provided, --managementApiPassword must also be provided or this option will be ignored."
258+
Description = "Overrides the value inferred from the connection string for the destination broker. If provided, --destinationManagementApiPassword must also be provided or this option will be ignored."
253259
};
254260

255261
return managementApiUserNameOption;
@@ -259,7 +265,7 @@ public static Option<string> CreateDestinationManagementApiPasswordOption()
259265
{
260266
var managementApiPasswordOption = new Option<string>("--destinationManagementApiPassword")
261267
{
262-
Description = "Overrides the value inferred from the connection string. If provided, --managementApiUserName must also be provided or this option will be ignored."
268+
Description = "Overrides the value inferred from the connection string for the destination broker. If provided, --destinationManagementApiUserName must also be provided or this option will be ignored."
263269
};
264270

265271
return managementApiPasswordOption;
@@ -269,7 +275,7 @@ public static Option<bool> CreateDestinationUseExternalAuthOption()
269275
{
270276
var useExternalAuthOption = new Option<bool>("--destinationUseExternalAuth")
271277
{
272-
Description = "Use the external authorization option when connecting to the broker",
278+
Description = "Use the external authorization option when connecting to the destination broker",
273279
DefaultValueFactory = _ => false
274280
};
275281

@@ -280,7 +286,7 @@ public static Option<string> CreateDestinationCertPathOption()
280286
{
281287
var certPathOption = new Option<string>("--destinationCertPath")
282288
{
283-
Description = "Set the path to the client certificate file for connecting to the broker"
289+
Description = "Set the path to the client certificate file for connecting to the destination broker"
284290
};
285291

286292
return certPathOption;
@@ -290,7 +296,7 @@ public static Option<string> CreateDestinationCertPassphraseOption()
290296
{
291297
var certPassphraseOption = new Option<string>("--destinationCertPassphrase")
292298
{
293-
Description = "The passphrase for client certificate file for when using a client certificate"
299+
Description = "The passphrase for the client certificate file when using a client certificate to connect to the destination broker"
294300
};
295301

296302
return certPassphraseOption;

0 commit comments

Comments
 (0)