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

Auth: Fix email verification bypass when using basic authentication #82914

Merged
merged 6 commits into from
Feb 16, 2024
Merged

Auth: Fix email verification bypass when using basic authentication #82914

merged 6 commits into from
Feb 16, 2024

Conversation

volcanonoodle
Copy link
Contributor

What is this feature?

Fixes CVE-2023-6152

@volcanonoodle volcanonoodle requested review from zserge, diegommm, mildwonkey, gillesdemey, konrad147 and soniaAguilarPeiron and removed request for a team February 16, 2024 16:00
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.4.x milestone Feb 16, 2024
@volcanonoodle volcanonoodle marked this pull request as draft February 16, 2024 16:02
@volcanonoodle volcanonoodle marked this pull request as ready for review February 16, 2024 17:16
@volcanonoodle volcanonoodle changed the title Fix email verification bypass when using basic authentication Auth: Fix email verification bypass when using basic authentication Feb 16, 2024
@volcanonoodle volcanonoodle merged commit 46c26bb into main Feb 16, 2024
22 checks passed
@volcanonoodle volcanonoodle deleted the email-verification branch February 16, 2024 17:55
@aangelisc aangelisc removed this from the 10.4.x milestone Mar 6, 2024
@aangelisc aangelisc added this to the 10.4.0 milestone Mar 6, 2024
@BrewTestBot BrewTestBot mentioned this pull request Mar 6, 2024
Closed
@volcanonoodle volcanonoodle mentioned this pull request Mar 7, 2024
Merged
@kalleep kalleep mentioned this pull request Mar 13, 2024
Merged
3 tasks
@BrewTestBot BrewTestBot mentioned this pull request Mar 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IevaVasiljeva IevaVasiljeva IevaVasiljeva approved these changes

@grafanabot grafanabot Awaiting requested review from grafanabot grafanabot is a code owner

@zserge zserge Awaiting requested review from zserge zserge was automatically assigned from grafana/backend-platform

@diegommm diegommm Awaiting requested review from diegommm diegommm was automatically assigned from grafana/backend-platform

@mildwonkey mildwonkey Awaiting requested review from mildwonkey mildwonkey was automatically assigned from grafana/backend-platform

@gillesdemey gillesdemey Awaiting requested review from gillesdemey gillesdemey is a code owner automatically assigned from grafana/alerting-frontend

@konrad147 konrad147 Awaiting requested review from konrad147 konrad147 is a code owner automatically assigned from grafana/alerting-frontend

@soniaAguilarPeiron soniaAguilarPeiron Awaiting requested review from soniaAguilarPeiron soniaAguilarPeiron is a code owner automatically assigned from grafana/alerting-frontend

Assignees
No one assigned
Labels
add to changelog area/backend area/frontend type/bug
Projects
None yet
Milestone
10.4.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@volcanonoodle @IevaVasiljeva @aangelisc