[ 
https://issues.apache.org/jira/browse/HADOOP-11031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15124457#comment-15124457
 ] 

Chris Nauroth commented on HADOOP-11031:
----------------------------------------

Hi [~lmccay].  This content looks great.  I have a few suggestions.

# Let's update site.xml to put a link to this page in the left navigation.  
This seems to make sense in the Common section, right after the link to HTTP 
Authentication.
# The last hyperlink in the table of contents doesn't work correctly.  I think 
it needs to be changed to this.
{code}
     - [The hadoop credential Command](#The_hadoop_credential_Command)
{code}
# Where the {{Configuration#getPassword}} method is mentioned, we can make it a 
hyperlink to the JavaDocs.
# The document mentions that it will cover "how to create custom providers" and 
"an example of its use will be included", but I didn't find that.  I assume it 
just wasn't done yet, so please consider adding it in the next patch revision.  
I also think it's acceptable to drop the mention of it and defer that kind of 
deep customization to a later patch.  I think the initial goal here was to make 
basic usage easier to understand.
# The table entry for SSL Passwords ends with "Components".  It looks like this 
was either a leftover word or an uncompleted sentence.
# I think we should discuss how to control the keystore password for the jceks 
provider ({{HADOOP_CREDSTORE_PASSWORD}} environment variable or 
{{hadoop.security.credstore.java-keystore-provider.password-file}} 
configuration property if not in environemnt).

bq. Being that this is a common facility being used by other projects, is it 
appropriate to include them all in the table of integration points and link to 
their specific documentation as appropriate?

Yes, I think this makes sense.  I also think it makes sense for the targeted 
pages to back-link over to CredentialProviderAPI.html, so that their users are 
guided to the documentation.  That would be a bigger change though obviously.

> Design Document for Credential Provider API
> -------------------------------------------
>
>                 Key: HADOOP-11031
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11031
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Larry McCay
>            Assignee: Larry McCay
>         Attachments: CredentialProviderAPI.md
>
>
> Provide detailed overview of the design, intent and use of the credential 
> management API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to