Repository navigation
Conversation
| { | ||
| AllowPrereleaseVersions = false, | ||
| DependencyVersion = DependencyVersion.Highest, | ||
| ApplicationRepository = @"https://artifact.finam.ru/artifactory/api/nuget/Etna.Applications", |
There was a problem hiding this comment.
this url is unreachable outside finam network.
There was a problem hiding this comment.
Sorry, probably I should remove this test.
|
|
||
| private static void ConfigureSupportedSslProtocols() | ||
| { | ||
| ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls | SecurityProtocolType.Tls11 | SecurityProtocolType.Tls12; |
There was a problem hiding this comment.
Why do you need it?
Looks like attempt to change appserver code to adopt for obsolete or misconfigured environment...
There was a problem hiding this comment.
Maybe you're right, but this is the way nuget team handle issue in future versions:
NuGet.Clients/NuGet.CommandLine/Program.cs
NuGet.Core/NuGet.Common/NetworkProtocolUtility.cs
I think that's a problem with HttpClient default configuration for work with secure protocols.
|
To my opinion, ServicePointManager is configured on application-wide level, therefore it's not a responsibility of NugetApplication repository to configure it. |
support fo https nuget feeds