Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem inheriting from WebClientFileDownloader class and overriding CreateHttpClient method #575

Closed
bdovaz opened this issue Apr 22, 2024 · 1 comment
Labels
Milestone

Comments

@bdovaz
Copy link

bdovaz commented Apr 22, 2024

In the 2 constructors of this class a virtual method is invoked (PrepareToDownloadFile > CreateHttpClient) and this is not good practice and is causing me problems:

In sonar you can see why it should not be done and that it is just my exact problem:

https://rules.sonarsource.com/csharp/RSPEC-1699/?search=virtual

@Deadpikle Deadpikle added this to the 3.x milestone Apr 24, 2024
@Deadpikle Deadpikle added the bug label Apr 24, 2024
@Deadpikle
Copy link
Collaborator

Thanks for the heads up! Fixed in 420f961 and preview version rolling out now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants