Bug 1036185

Summary: [Doc Bug Fix] Rewrite SPNEGO Module Options table in Reference in Security Guide
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Ondrej Lukas <olukas>
Component: DocumentationAssignee: Lucas Costi <lcosti>
Status: CLOSED CURRENTRELEASE QA Contact: Russell Dickenson <rdickens>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: hmlnarik, jcacek, lcosti, sgilda, twells
Target Milestone: GAKeywords: Documentation
Target Release: EAP 6.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Build Name: 14876, Security Guide-6.2-1 Build Date: 28-11-2013 11:27:34 Topic ID: 4732-549749 [Latest]
Last Closed: 2014-08-06 14:38:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1035353    

Description Ondrej Lukas 2013-11-29 15:53:31 UTC
Table A.35. SPNEGO Module Options in Reference has all options wrong - these options are not connected with SPNEGO Module Options, right options are removeRealmFromPrincipal, serverSecurityDomain and usernamePasswordDomain.

Comment 3 Josef Cacek 2014-04-23 17:11:52 UTC
Please change also the caption of the table. Use "SPNEGOUsers Module Options" instead of "SPNEGO Module Options"

The table content seems OK in SG and AG CC-edition and SG 6.3.0-edition.
The fixes are not in Administration and Configuration guide for 6.3.0. (Maybe a new push is needed).

Comment 5 sgilda 2014-05-21 19:01:36 UTC
I'm a little confused about this bug. 

I think mofification of Topic 4735 may be a typo as it is not used in any content spec. Should it be topic 4732?

I'm not sure which book(s) this impacts.

For EAP 6.3, I think it can be verified here:

http://documentation-devel.engineering.redhat.com/site/documentation/en-US/JBoss_Enterprise_Application_Platform/6.3/html-single/Security_Guide/index.html#Included_Authentication_Modules1

Comment 6 Lucas Costi 2014-05-21 22:52:38 UTC
You are right Sande, that is a typo. The correct topic is 4732.

However, the change I made in comment 4 has been superseded by a change made in BZ#1036160 (comment 9) for  Peter Skopek.

Josef, can you please confirm that the later change from the above bug is more correct, and that the current table name [1] is ok?

[1] http://documentation-devel.engineering.redhat.com/site/documentation/en-US/JBoss_Enterprise_Application_Platform/6.3/html-single/Security_Guide/index.html#topic4732_spnegomoduleoptions

Comment 7 Josef Cacek 2014-06-10 08:23:42 UTC
Lucas, the content seems fine. But we should stay consistent in naming. Either we use "SPNEGO" always or "SPNEGOUsers".

If we stick to "SPNEGO", then one occurrence of "SPNEGOUsers" in [1] should be fixed.

[1] http://documentation-devel.engineering.redhat.com/site/documentation/en-US/JBoss_Enterprise_Application_Platform/6.3/html-single/Security_Guide/index.html#LdapLoginModule1

Comment 8 Lucas Costi 2014-06-11 00:20:30 UTC
Thanks Josef. I have corrected that occurrence in "Ldap Login Module" [28310, revision 666517].

I have just checked the A&C and Developer guides too, and have confirmed there are no instances of "SPENGOUSers"

Comment 9 Lucas Costi 2014-06-11 06:23:44 UTC
Changes are now ready for review (Revision 6.3.0-25):

The original issue was verified in [1] as now correct by Josef in comment 7.

The last change requested in comment 7 ("SPNEGOUSers" -> "SPNEGO" in the 'Note' admonition), and fixed in comment 8, is here:

http://documentation-devel.engineering.redhat.com/site/documentation/en-US/JBoss_Enterprise_Application_Platform/6.3/html-single/Security_Guide/index.html#LdapLoginModule1

[1] http://documentation-devel.engineering.redhat.com/site/documentation/en-US/JBoss_Enterprise_Application_Platform/6.3/html-single/Security_Guide/index.html#topic4732_spnegomoduleoptions

Comment 10 Josef Cacek 2014-06-11 06:35:35 UTC
Verified on docs-stage.
Revision 6.3.0-25