m-cmp / mc-iam-manager

https://github.com/m-cmp/docs/tree/main

Apache License 2.0

3 stars 5 forks source link

iamtokenvaldator is not compaitable with echo-jwt/v4 #39

Closed raccoon-mh closed 3 months ago

raccoon-mh commented 3 months ago

iamtokenvaldator is not compaitable with echo-jwt/v4

github.com/golang-jwt/jwt/v5 This module is necessary because github.com/golang-jwt/jwt/v5 is imported in:

server -- github.com/labstack/echo-jwt/v4
echo-jwt/v4 에서 github.com/golang-jwt/jwt/v5 를 참조하지만, 현 iamtokenvalidator 는 v4 를 참조하고 있음.
echo 프레임워크 keyfunction 을 사용하기 위해 버전업 또는 두가지 버전을 제공하는 것을 고려중..

raccoon-mh commented 3 months ago

github.com/labstack/echo-jwt v0.0.0-20221127215225-c84d41a71003

위 echo-jwt 버전 사용시 가능..

yunkon-kim commented 3 months ago

특별한 사유가 없으시다면, iamtokenvalidator가 github.com/golang-jwt/jwt/v5 로 업그레이드 하시고 mc-iam-manager 내에서 동일 버전으로 관리하시면 좋을 것 같다는 의견을 말씀드립니다. 이후에 Tumblebug에서 이에 맞도록 업데이트 하겠습니다.

raccoon-mh commented 3 months ago

@yunkon-kim

확인했습니다. 빠른시일내로 업데이트 진행하고 현이슈에서 공유드리겠습니다.

raccoon-mh commented 3 months ago

fixed by https://github.com/m-cmp/mc-iam-manager/pull/44