Skip to content

enable server certificate vaildation #9#40

Merged
1 commit merged into
devfrom
make/9-ssl-validation
May 13, 2026
Merged

enable server certificate vaildation #9#40
1 commit merged into
devfrom
make/9-ssl-validation

Conversation

@yurademyan-netizen
Copy link
Copy Markdown
Contributor

@yurademyan-netizen yurademyan-netizen commented May 10, 2026

JIRA

Code reviewers

Second Level Review

  • @github_username

Summary of issue

SonarQube знайшов вразливість безпеки (Security Vulnerability) через примусово вимкнену валідацію SSL/TLS сертифікатів при HTTP-запитах до зовнішніх ресурсів.

Summary of change

  1. Видалено ServerCertificateCustomValidationCallback = true у файлі WebParsingUtils.cs (тепер використовується стандартна системна перевірка сертифікатів).
  2. Змінено TrustServerCertificate=True на False в appsettings.IntegrationTests.json.
  3. Виправлено стилістичні попередження від StyleCop.

Closes #9

Testing approach

Локальний запуск (F5). Перевірено, що бекенд успішно стартує, Swagger доступний на порту 5001 (HTTPS), а звернення до зовнішніх API тепер вимагають валідного сертифіката. Білд проходить без помилок.

CHECK LIST

  • CI passed
  • Code coverage >=95%
  • PR is reviewed manually again (to make sure you have 100% ready code)
  • All reviewers agreed to merge the PR
  • I've checked new feature as logged in and logged out user if needed
  • PR meets all conventions

@yurademyan-netizen yurademyan-netizen self-assigned this May 10, 2026
@yurademyan-netizen yurademyan-netizen added the validation validation of something label May 10, 2026
@sonarqubecloud
Copy link
Copy Markdown

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

@Vasia1101 Vasia1101 self-requested a review May 11, 2026 05:59
@Vasia1101 Vasia1101 self-requested a review May 11, 2026 06:00
@yurademyan-netizen yurademyan-netizen closed this pull request by merging all changes into dev in b05c229 May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

validation validation of something

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable server certificate validation on this SSL/TLS connection

3 participants