Skip to content

Conversation

@youkoulayley
Copy link
Contributor

What does this PR do?

This PR removes the backoff for the HTTP Challenge as the Present method is synchronous, there should be no need to retry the request.

Motivation

Fixes #10214

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Co-authored-by: Romain rtribotte@users.noreply.github.com

Copy link
Member

@kevinpollet kevinpollet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@lbenguigui lbenguigui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nmengin nmengin requested a review from jspdown November 13, 2023 13:20
Copy link
Contributor

@jspdown jspdown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@traefiker traefiker force-pushed the fix/http-challenge-backoff branch from 2d5a5c0 to 6ec411f Compare November 13, 2023 14:37
@traefiker traefiker merged commit 84516f9 into traefik:v2.10 Nov 13, 2023
@nmengin nmengin changed the title Remove backoff for http challenge Remove backoff for http challenge (CVE-2023-47124) Nov 29, 2023
@rtribotte rtribotte mentioned this pull request Nov 29, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants