Bug 1036160 - Add more documentation about Login Modules in Reference in Security Guide
Summary: Add more documentation about Login Modules in Reference in Security Guide
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Documentation
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: GA
: EAP 6.2.2
Assignee: Russell Dickenson
QA Contact: Peter Skopek
URL:
Whiteboard:
: 1035894 (view as bug list)
Depends On:
Blocks: 1035353
TreeView+ depends on / blocked
 
Reported: 2013-11-29 15:31 UTC by Ondrej Lukas
Modified: 2014-08-14 15:19 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
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-06-02 12:50:47 UTC
Type: Bug


Attachments (Terms of Use)

Description Ondrej Lukas 2013-11-29 15:31:51 UTC
Reference section in Security Guide contains a list of Login Modules with their description and options. Description is much briefer than EAP 5 Security Guide. The request for extending the section in EAP 6 came from Common Criteria consultants from AtSec.

Comment 2 Tom WELLS 2014-04-22 04:11:13 UTC
*** Bug 1035894 has been marked as a duplicate of this bug. ***

Comment 3 Josef Cacek 2014-04-24 07:24:08 UTC
Verification failed.

There is a lot of EAP 5 specific information in the new chapter. It will need a detail review.

After a quick look can I see following problems (but there will be much more IMO):

* XML examples with security domains (application-policies) are not valid for EAP 6

* examples in the Password Hashing section should use rather SHA-256 than MD5

* remove module options from the chapter and include a reference to corresponding tables in "Included Authentication Modules"

[1] http://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_JBoss_Enterprise_Application_Platform_Common_Criteria_Certification/6.2.2/html-single/Security_Guide/index.html#Included_Authentication_Modules1

Comment 5 Russell Dickenson 2014-04-27 23:05:40 UTC
In discussion with Peter Skopek we agreed that he, or another SME, would review the content of chapter 16 and provide feedback. To make this task as easy as possible, I have copied the chapter's text to an Etherpad [1].

Once we have that feedback, we will work on editing the chapter as required.

[1] http://etherpad.corp.redhat.com/jboss-eap6-cc-login-modules-chapter-review

Comment 6 Peter Skopek 2014-05-03 15:17:47 UTC
Russell, you can find my changes to the text in etherpad [1].

[1] http://etherpad.corp.redhat.com/jboss-eap6-cc-login-modules-chapter-review

Comment 7 Russell Dickenson 2014-05-07 03:11:40 UTC
Attention: Peter

All feedback in the Etherpad has been incorporated into the CC edition of the Security Guide, which can be verified at [1].

[1] http://docbuilder.usersys.redhat.com/22671/

Comment 8 Russell Dickenson 2014-05-07 03:19:56 UTC
Attention: Peter

In the text, it seems that instead of "Login Modules", this chapter should be titled "Authentication Modules". What do you think?

Comment 9 Peter Skopek 2014-05-07 08:08:45 UTC
@Russell here is my feedback:

1. "Procedure 16.1. Secure Web Applications with Certificates and Role-based Authorization": change name of secured application to something different than jmx-console.war it evokes older versions if EAP. My suggestion "user-app.war"

2.  Can we make chapter "16.1.11. RunAsIdentity Creation" move one level deeper as it really is related to "16.1.10. RunAs Login Module"?

3. "16.1.13. SPNEGOUsers Login Module": this name of SPNEGO is an alias. I would like to see there "SPNEGO" used instead of "SPNEGOUsers". Sorry for not noticing it the first time.

4. missing class names in login module tables:

"Table A.1. RealmDirect" class is org.jboss.as.security.RealmDirectLoginModule
"Table A.3. Client" class is org.jboss.security.ClientLoginModule
"Table A.5. Remoting" class is org.jboss.as.security.remoting.RemotingLoginModule

5. login modules Certificate and CertificateUsers are the same. You can delete "Table A.9. CertificateUsers" and "Table A.10. CertificateUsers Module Options"

6. "Table A.25. RunAs" class cell contains "Class:" which redundant in this place.

7. In "Table A.37. SPNEGOUsers" and "Table A.38. SPNEGOUsers Module Options" rename SPNEGOUsers to SPNEGO as suggested in #3

WRT: Renaming the chapter to "Authentication Modules": I think that we should stay with the current name (Login Modules) as couple of them are also performing authorization tasks, so we might put more confusion to the whole thing than necessary.

Comment 10 Peter Skopek 2014-05-07 14:03:57 UTC
@Russell: I have checked fixes to my feed back from comment #9. 
All is fine and #3 is not possible due to technical problems in docs build system (as discussed on IRC).

Comment 12 Russell Dickenson 2014-05-15 20:58:57 UTC
The problem with the missing example from the SPNEGO section, mentioned in comment 11, has been resolved. This can be verified in revision 6.2.2-11 (or higher) at [1].


[1] http://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_JBoss_Enterprise_Application_Platform_Common_Criteria_Certification/6.2.2/html-single/Security_Guide/index.html#SPNEGOLoginModule

Comment 13 Ondrej Lukas 2014-05-16 06:31:55 UTC
Verified in Revision 6.2.2-12 of the CC edition of the Security Guide.


Note You need to log in before you can comment on or make changes to this bug.