mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Clement Michaud <>
Subject Re: Review Request 66621: Add support for alg RS256 to JWT library.
Date Tue, 15 May 2018 21:05:12 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated mai 15, 2018, 9:05 apr├Ęs-midi)

Review request for mesos, Alexander Rojas and Till Toenshoff.


Remove useless empty line

Bugs: MESOS-8788

Repository: mesos


Add support for alg RS256 to JWT library.

Diffs (updated)

  3rdparty/libprocess/include/process/jwt.hpp 768cbf6fa91537ff9f45f236f4033097c5cea959 
  3rdparty/libprocess/include/process/ssl/utilities.hpp b7cc31c33fd35c93754407f8b350eeb993177f1d

  3rdparty/libprocess/src/jwt.cpp 921031e6fe3ced5a6be6bc96190fae6d8282ae26 
  3rdparty/libprocess/src/ssl/utilities.cpp 4d3727daf53ec62a19255da5a9804d342e770ec2 
  3rdparty/libprocess/src/tests/jwt_keys.hpp PRE-CREATION 
  3rdparty/libprocess/src/tests/jwt_tests.cpp eb36a9aed3b11208c7cdc6f20b5347f46821a207 




make check

I added the same tests than the ones for HS256 (i.e., validation in following cases: bad header,
bad payload, unknown alg, unsupported alg, valid token etc.. and creation of a valid token).
I also added a test to verify that the validation of a RS256 token fails when using the wrong
public key.


Clement Michaud

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message