https://issues.apache.org/bugzilla/show_bug.cgi?id=48685

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

--- Comment #5 from Mark Thomas <ma...@apache.org> 2010-03-08 19:45:38 UTC ---
The patch will not be applied in it's current form.

There are a number of issues. In no particular:
- format (bracket placement, random white-space changes)
- use of a system property that makes SPNEGO configuration global
- lack of documentation

This last point is particularly important given that this depends on a
specifically configured JAAS realm.

I am still leaning towards creating a new authenticator for this. The spec
allows containers to add additional methods beyond BASIC, FORM, etc.

Given the effort required to test this, I'm not going to start setting up a
domain and the like until there is some documentation, particularly on the JAAS
configuration required. The other issues with the patch I am less concerned
with given that it is likely to be extensively re-worked before it is included.
I'm also marking this as an enhancement.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to