Skip to content

Prevent sending second WWW-Authenticate header - #5372

Merged
MorrisJobke merged 1 commit into
masterfrom
we-shall-monkey-patch-auth-headers-for-clients-that-dont-follow-specs
Jun 13, 2017
Merged

Prevent sending second WWW-Authenticate header#5372
MorrisJobke merged 1 commit into
masterfrom
we-shall-monkey-patch-auth-headers-for-clients-that-dont-follow-specs

Conversation

@LukasReschke

Copy link
Copy Markdown
Member

Overrides \Sabre\DAV\Auth\Backend\AbstractBearer::challenge to prevent sending a second WWW-Authenticate header which is standard-compliant but most DAV clients simply fail hard.

Fixes#5088

Signed-off-by: Lukas Reschke lukas@statuscode.ch

Overrides \Sabre\DAV\Auth\Backend\AbstractBearer::challenge to prevent sending a second WWW-Authenticate header which is standard-compliant but most DAV clients simply fail hard.
Fixes#5088
Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschkeLukasReschke added the 3. to review Waiting for reviews label Jun 13, 2017
@LukasReschke

Copy link
Copy Markdown
MemberAuthor

Backport to stable12 at #5386

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🙈

@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 13, 2017
@MorrisJobke
MorrisJobke merged commit e7b5c5c into masterJun 13, 2017
@MorrisJobke
MorrisJobke deleted the we-shall-monkey-patch-auth-headers-for-clients-that-dont-follow-specs branch June 13, 2017 17:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@LukasReschke@nickvergessen@MorrisJobke