Skip to content

Harden parsing and curl escaping#53

Closed
ELF-Nigel wants to merge 5 commits intoKeyAuth:mainfrom
ELF-Nigel:main
Closed

Harden parsing and curl escaping#53
ELF-Nigel wants to merge 5 commits intoKeyAuth:mainfrom
ELF-Nigel:main

Conversation

@ELF-Nigel
Copy link

  • curl_easy_escape results are now freed safely to prevent leaks (auth.cpp).
  • VerifyPayload now handles invalid timestamps defensively (still fails closed).
  • safer substring extraction in get_str_between_two_str to avoid out‑of‑range when delimiters are missing.
  • utils::string_to_timet now validates parse failure.
  • minor loop safety in auth.hpp (size‑safe index and no copies).

@github-actions
Copy link

The KeyAuth owner has been notified of your pull request. Please wait for him to review it. This could take several days.

@ELF-Nigel ELF-Nigel closed this Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant