Skip to content

fix: bug that was caused by stoi

Giorgi Baramidze requested to merge slb/gb/stoi-bug into master

Type of change

  • Bug Fix
  • Feature
  • Pipeline
  • Test

Does this introduce a change in the core logic?

  • No
  • Yes

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this follow conventional commits spec?

  • [X ] No
  • Yes

Have you set the target Milestone?

  • No
  • Yes

Have you set the no-detached-pipeline label?

  • [] No
  • Yes

Updates description?

There was a bug in parsing the header file. If the number was bigger than ineteger std::stoi would throw exception. So the fix was to change it to std::stull which can handle bigger values.

Merge request reports