Project

General

Profile

Actions

Bug #3475

closed

When Foreman URL (/users/login) is used as ErrorDocument target, status is not preserved

Added by Jan Pazdziora over 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Authentication
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

In the document http://projects.theforeman.org/projects/foreman/wiki/Foreman_and_mod_auth_kerb, configuration is shown which allows to configure mod_auth_kerb for Negotiate authentication for issue http://projects.theforeman.org/issues/3312.

However, the fallback configuration that I found working is a little bit cumbersome:

ErrorDocument 401 '<html><meta http-equiv="refresh" content="0; URL=/users/login"><body>Kerberos authentication did not pass.</body></html>'

It would be much better if we could use

ErrorDocument 401 /users/login

However, when the /users/login is served by Foreman's passenger, it does not preserve the 401 status, returning 200 instead. As a result, the Negotiation does not happen at all (and situation is the same when Basic authentication is configured). So that setup cannot be used.

Foreman shouldn't specify status (and force 200) if there already is one set in the response.


Related issues 1 (0 open1 closed)

Related to Foreman - Bug #10586: When /users/login is used as ErrorDocument for 401 with status=401, it does not provide that statusClosedJan Pazdziora05/22/2015Actions
Actions #2

Updated by Dominic Cleal about 10 years ago

  • Category set to Authentication
  • Status changed from New to Ready For Testing
  • Assignee set to Jan Pazdziora
  • Target version set to 1.9.1
Actions #3

Updated by Anonymous about 10 years ago

  • Target version changed from 1.9.1 to 1.9.0
Actions #4

Updated by Dominic Cleal about 10 years ago

  • translation missing: en.field_release set to 7
Actions #5

Updated by Jan Pazdziora about 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #6

Updated by The Foreman Bot almost 9 years ago

  • Pull request https://github.com/theforeman/foreman/pull/2401 added
Actions #7

Updated by Dominic Cleal almost 9 years ago

  • Related to Bug #10586: When /users/login is used as ErrorDocument for 401 with status=401, it does not provide that status added
Actions

Also available in: Atom PDF