src/library/Connect/LocalEnvironmentManager.cs (7 lines): - line 113: // TODO (lolodi): not sure this is actually necessary, and if it is, we should probably do it when the result of the operation is required. - line 149: IP = endpoint.LocalIP.ToString(), //TODO (lolodi): HostFileEntry should use IPAddress instead of string for IP. - line 241: /// TODO (lolodi): this should return a Task, we should ping the local agent to verify that it is indeed running and then complete the task - line 363: // TODO (lolodi): all this check should not be required since we already have mapped to ports that are not used (on linux/mac) or we did already blew up when we checked if the local port was free. - line 406: // TODO (lolodi): Once we verify that we don't actually need the checks above we can just pass the whole enpoint to the _servicePortForwardManager and iterate thorugh all the ports there. - line 438: result["DAPR_HTTP_PORT"] = endpoint.Ports[0].LocalPort.ToString(); // TODO (lolodi): this is a hack that relies on the HTTP port to always be the first and GRPC port the second. - line 543: // TODO: wait to support file token to actually enable this code. src/routingmanager/RoutingStateEstablisher.cs (4 lines): - line 179: // TODO USER STORY 1326960: Check if wildcard * hosts are supported by IngressRoute - line 404: // TODO: This is nasty and needs a better fix - line 592: // TODO: fix the logging of this event to have more info/be less noisy - line 641: // TODO USER STORY 1326960: Https support: We are assuming that the original manually created cert will be a wildcard src/library/Connect/KubernetesRemoteEnvironmentManager.cs (3 lines): - line 102: // TODO do we really want this?? - line 236: // TODO: This code needs further investigation - line 1157: /// TODO: Bug 1292855: Find a way to re-use code between deployment & statefulset patching logic src/library/Utilities/RemoteContainerConnectionDetailsResolver.cs (2 lines): - line 46: // TODO (lolodi): this needs to be refactored so that we don't need to use such workarounds. - line 100: // TODO: This system of setting the source entity multiple times (e.g. we start as "Service", get set to "Pod" and then later to "Deployment"/"StatefulSet") src/library/Client/ManagementClients/ConnectManagementClient.cs (2 lines): - line 343: // TODO (lolodi): The user is passing local ports as an array, and then we collect the container ports from the container object in kubernetes. - line 414: // TODO: (lolodi) this should be embedded in StartRemoteAgent since this is taking care of restarting the remote agent when it stops. src/common/IP/IPManager.cs (2 lines): - line 162: // TODO: Fix Bug 1125798: Need to be a good pf citizen - line 252: // TODO: Fix Bug 1125798: Need to be a good pf citizen src/library/Connect/WorkloadInformationProvider.cs (2 lines): - line 65: // TODO (lolodi): This method should already take care of avoiding possible duplicates between services that are already reachable and the ones defined by the user in the localProcessConfig. - line 66: // TODO (lolodi)(perf): This result of this method should be wrapped in an AsyncLazy so that we only pull it once src/dsc/Commands/Connect/ConnectCommand.cs (2 lines): - line 493: // TODO (lolodi): To speed things up we should deploy routing manager and remote agent at the same time, instead of awaiting on the deployment of one before deploying the other. - line 519: // TODO: report progress local agent started src/common/Models/ProgressUpdate.cs (1 line): - line 13: // TODO: Make the completion percentage deterministic src/routingmanager/Extensions.cs (1 line): - line 269: // TODO USER STORY 1326960: Add https support src/endpointmanager/HostsFile/HostsFileManager.cs (1 line): - line 115: // TODO (lolodi): kept around to kep tests happy, this foreach can probably be removed as we always assign from an IPAddress (in fact the string in hostEntry should be IPAddress instead of string src/dsc/Commands/Connect/PrepConnectCommand.cs (1 line): - line 87: // TODO (lolodi): Everything local (e.g. useKubernetesEnvVar or runContainerized can be grouped in a localConnectionDetails class) src/library/Connect/Environment/LocalProcessConfig.cs (1 line): - line 173: // TODO: t.Serialize() returns "Serialization Error" when port is specified. Need to fix this at some point, but it doesn't affect the user experience at all. src/library/Connect/Environment/EnvironmentEntry.cs (1 line): - line 47: //TODO: depending on the type concatenation is not the right way, e.g. volume mounts on windows. src/common/IO/Output/ConsoleLauncher.cs (1 line): - line 75: UseShellExecute = true, // TODO: use false gives the best experience but issues with Ctrl-C, Console.ReadLine from debugged app src/LocalAgent/LocalAgentApp.cs (1 line): - line 59: // TODO (lolodi): management clients need to be disposed for log flushing src/common/ProgressModule.cs (1 line): - line 50: // TODO: Make the completion percentage deterministic src/library/Client/ManagementClients/IConnectManagementClient.cs (1 line): - line 59: // TODO: (lolodi) we should use event or at least take a callback src/library/Connect/Environment/EnvironmentTokenParser.cs (1 line): - line 32: // TODO: implement real file token, in the documentation we talk about the possiblity of mounting files without specifying volumes but this doesn't actually work today