[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-07-06 Thread Matt Gilman (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14615496#comment-14615496
 ] 

Matt Gilman commented on NIFI-679:
--

There is no existing guide for source format outside of checkstyle which is 
configured in the parent pom [1]. You can verify your contribution is compliant 
by running the contrib-check profile [2]. This however does not do anything for 
verifying local mutability. That is something we try to do, but there are 
certainly cases where that is missing throughout the codebase.

Generally, we would suggest creating a new ticket if focusing on cleaning up a 
portion of code. What likely happened in this instance is that an IDE was 
configured to automatically format and make checkstyle-type modifications and 
those were included into a patch with another functional change. 

[1] 
https://github.com/apache/incubator-nifi/blob/develop/nifi-parent/pom.xml#L260
[2] 
https://github.com/apache/incubator-nifi/blob/develop/nifi-parent/pom.xml#L380

 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0

 Attachments: nifi-679.patch


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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


[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-07-03 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14613178#comment-14613178
 ] 

ASF subversion and git services commented on NIFI-679:
--

Commit 9d6458ab2800add00312e129bf60cd04ead350f9 in incubator-nifi's branch 
refs/heads/develop from [~jskora]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-nifi.git;h=9d6458a ]

NIFI-679: Incorporate Basic Authentication handling into InvokeHTTP processor.

Signed-off-by: Matt Gilman matt.c.gil...@gmail.com


 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0

 Attachments: nifi-679.patch


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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


[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-07-03 Thread Matt Gilman (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14613179#comment-14613179
 ] 

Matt Gilman commented on NIFI-679:
--

Looks good... Thanks!

 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0

 Attachments: nifi-679.patch


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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


[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-07-03 Thread Joe Skora (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14612881#comment-14612881
 ] 

Joe Skora commented on NIFI-679:


I have attached a proposed patch for Basic Auth in InvokeHTTP.

 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0

 Attachments: nifi-679.patch


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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


[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-06-30 Thread Matt Gilman (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14608175#comment-14608175
 ] 

Matt Gilman commented on NIFI-679:
--

Awesome! Sorry about the vague description, but the intent at this time is to 
just add support for basic authentication.

 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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


[jira] [Commented] (NIFI-679) InvokeHTTP - Add support for basic authentication

2015-06-30 Thread Joe Skora (JIRA)

[ 
https://issues.apache.org/jira/browse/NIFI-679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14608138#comment-14608138
 ] 

Joe Skora commented on NIFI-679:


I'm working on this and have a version supporting Basic Auth functionality.

The title sounds specific to Basic Auth but the description is more generic, is 
the intent to support anything besides Basic Auth?

 InvokeHTTP - Add support for basic authentication
 -

 Key: NIFI-679
 URL: https://issues.apache.org/jira/browse/NIFI-679
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Matt Gilman
Priority: Minor
 Fix For: 0.2.0


 InvokeHTTP should optionally support username/password for web services that 
 do not employ certificate based authentication.



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