Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change API in jwt-cpp #11

Open
neon1ks opened this issue Mar 13, 2023 · 4 comments
Open

Change API in jwt-cpp #11

neon1ks opened this issue Mar 13, 2023 · 4 comments

Comments

@neon1ks
Copy link
Contributor

neon1ks commented Mar 13, 2023

Take note!

In release v0.7.0-rc.0 from Dec 5, 2022:

  • get_payload_claims() was replaced by get_payload_json()

https://github.com/Thalhammer/jwt-cpp/releases

When updating the library, the following code is affected:

diff --git a/src/l6/jwt-cpp-example/main.cpp b/src/l6/jwt-cpp-example/main.cpp
index a2b6c8c..864499e 100644
--- a/src/l6/jwt-cpp-example/main.cpp
+++ b/src/l6/jwt-cpp-example/main.cpp
@@ -20,6 +20,7 @@ int main()
 
     verifier.verify(decoded);
 
-    for(auto& e : decoded.get_payload_claims())
+    // for(auto& e : decoded.get_payload_claims()) // old api
+    for(auto& e : decoded.get_payload_json())
         std::cout << e.first << " = " << e.second << std::endl;
 }
@neon1ks
Copy link
Contributor Author

neon1ks commented Mar 13, 2023

Не увидел макроса, указывающего версию библиотеки jwt-cpp. Поэтому затрудняюсь как подправить код без поломки обратной совместимости.

@artiomn
Copy link
Owner

artiomn commented Mar 14, 2023

Не увидел макроса, указывающего версию библиотеки jwt-cpp. Поэтому затрудняюсь как подправить код без поломки обратной совместимости.

Его нет, похоже: Thalhammer/jwt-cpp#273
Тут важный момент: если оно собирается в контейнере, имеет смысл править.
Я не обновляю GB образ, т.к. не работаю с ними уже, к тому же, не хочется ничего сломать.

В любом случае, проверю и волью, если работает. К тому же, есть образы для книги, там вариант библиотеки новее.

@neon1ks
Copy link
Contributor Author

neon1ks commented Mar 16, 2023

К тому же, есть образы для книги, там вариант библиотеки новее.

А что за книга? Есть ли возможность её купить?

@artiomn
Copy link
Owner

artiomn commented Apr 25, 2023

Да, конечно. В основу положены курсы. Но там значительно переработан материал и
Образы уже доступны (начинаются с nb-, остальное название такое же, но совместимость не обещаю).
Книга выйдет, похоже, в конце лета, сначала в РФ.

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

No branches or pull requests

2 participants