search

arun11299 / cpp-jwt

JSON Web Token library for C++
MIT License
393 stars 112 forks source link

Fix tests to compile #9

Closed gramic closed 6 years ago

gramic commented 6 years ago

Currently at least test_jwt_decode.cc has errors. I am able to fix them, but it was suspicious to find that they do not compile. Are you plannig to keep them working?

arun11299 commented 6 years ago

Hello @gramic Can you give some more information about the failure ? Specifically:

  1. Compile time errors
  2. Platform
  3. OpenSSL version

Thanks.

gramic commented 6 years ago

I did not provide the error because it is too simple. However, here it is

include/jwt/test/test_jwt_decode.cc:17:32: error: too few arguments to function call, expected 3, have 2 auto res = sgn.encode(hdr, jp);

arun11299 commented 6 years ago

Ah...I see it now. These are internal tests that I was using while developing it. I didn't think anybody would use these instead of the tests which are there outside the include directory.

I will fix these. Thanks for reporting.

gramic commented 6 years ago

Well, I really did intent to use those outside, but somehow I made a mistake too.