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

20240517 develop to release ( v4.0.0 ) #493

Merged
merged 725 commits into from
May 17, 2024
Merged

20240517 develop to release ( v4.0.0 ) #493

merged 725 commits into from
May 17, 2024

Conversation

ktkfree
Copy link
Contributor

@ktkfree ktkfree commented May 17, 2024

20240517 develop to release ( v4.0.0 )

ktkfree and others added 30 commits April 16, 2024 13:35
trivial. fix update systemNotificationRUle
trivial. support array filter on users
GetPolicyForEdit fill parameter 버그 수정
trivial. add user roles to audit
trivial. bugfix for stack node count
trivial. add status to systemNotificationRule
정책 업데이트 버그 픽스
ktkfree and others added 28 commits May 7, 2024 14:02
temp-fix: make appserving dashboard not being refreshed for now
trivial. add attribute to audit
feature. add force deleting logic when the cluster status was invalid
minor fix. fix url of kubeconfig
trivial. disable email on default systemnotification
파라미터를 명시하지 않은 정책을 적용할 때 버그 수정
feature. add policyName to systemNotification
bugfix. User role에 대한 권한 부여 버그 수정
trivial. fix messageActionProposal on policy_notifications
trivial. fix filter for policyNotifications
데이터 동기화 설정 필드 추가
@@ -37,3 +38,18 @@
}
return token, err
}

func StringToTokenWithoutVerification(tokenString string) (*jwt.Token, error) {
token, _, err := new(jwt.Parser).ParseUnverified(tokenString, jwt.MapClaims{})

Check failure

Code scanning / CodeQL

Missing JWT signature check High

This JWT is parsed without verification and received from
this user-controlled source
.
This JWT is parsed without verification and received from
this user-controlled source
.
@ktkfree ktkfree merged commit b5cb7f0 into release May 17, 2024
7 of 9 checks passed
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

Successfully merging this pull request may close these issues.

8 participants