jwt-cpp icon indicating copy to clipboard operation
jwt-cpp copied to clipboard

Use common base class for all exceptions thrown by jwt-cpp

Open sirzooro opened this issue 3 years ago • 1 comments

This was initially described in https://github.com/Thalhammer/jwt-cpp/issues/212. Please add new class derived from std::system_error, and derive all jwt-cpp exceptions from it. This would allow to write bigger block of code, and at the end add few catch sections for different kinds of errors (e.g. network error, jwt validation error, etc.). You also could store some extra information in additional fields of exception class, e.g. name of claim which did not pass validation.

Desktop:

  • OS: RedHat 8
  • Compiler g++ 10.3.1
  • Version jwt-cpp v0.6.0

sirzooro avatar Feb 28 '22 17:02 sirzooro

Extra bit for this is some of the algo families added latter on mix other algo error codes https://github.com/Thalhammer/jwt-cpp/pull/318#issuecomment-1849113137

more possible work in this direction https://github.com/Thalhammer/jwt-cpp/pull/318#pullrequestreview-1777600060

prince-chrismc avatar Dec 11 '23 00:12 prince-chrismc