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.

Sign up for GitHub

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

Jump to bottom

Bad documentation of error handling in ParseWithClaims can lead to potentially dangerous situations #57135

Closed
hawkingrei opened this issue Nov 5, 2024 · 0 comments · Fixed by #57136
Closed

Bad documentation of error handling in ParseWithClaims can lead to potentially dangerous situations #57135

hawkingrei opened this issue Nov 5, 2024 · 0 comments · Fixed by #57136
Assignees
@hawkingrei
Labels
affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. affects-8.5 This bug affects the 8.5.x(LTS) versions. severity/moderate sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@hawkingrei
Copy link
Member

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

ref https://github.com/hawkingrei/tidb/security/dependabot/24

image

2. What did you expect to see? (Required)

3. What did you see instead (Required)

4. What is your TiDB version? (Required)
@hawkingrei hawkingrei added type/bug The issue is confirmed as a bug. sig/sql-infra SIG: SQL Infra labels Nov 5, 2024
@hawkingrei hawkingrei self-assigned this Nov 5, 2024
@hawkingrei hawkingrei mentioned this issue Nov 5, 2024
Merged
13 tasks
ti-chi-bot bot pushed a commit that referenced this issue Nov 5, 2024
@hawkingrei
*: upgrade golang-jwt/jwt to fix secruity alerms (#57136)
b0ba097 
close #57135
This was referenced Nov 6, 2024
Closed
Merged
Open
Merged
@ti-chi-bot ti-chi-bot bot added affects-8.1 This bug affects the 8.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.5 This bug affects the 8.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. labels Nov 7, 2024
ti-chi-bot bot pushed a commit that referenced this issue Nov 7, 2024
@ti-chi-bot
*: upgrade golang-jwt/jwt to fix secruity alerms (#57136) (#57183)
803b0ff 
close #57135
ti-chi-bot bot pushed a commit that referenced this issue Dec 11, 2024
@ti-chi-bot
*: upgrade golang-jwt/jwt to fix secruity alerms (#57136) (#57181)
5a6e570 
close #57135
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hawkingrei hawkingrei

Labels
affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. affects-8.5 This bug affects the 8.5.x(LTS) versions. severity/moderate sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

*: upgrade golang-jwt/jwt to fix secruity alerms hawkingrei/tidb
1 participant
@hawkingrei