[GitHub] [nifi] exceptionfactory commented on a change in pull request #5438: NIFI-9029 Document Missing Properties in the Sys Admin Guide

[GitBox <gi...@apache.org>]

exceptionfactory commented on a change in pull request #5438:
URL: https://github.com/apache/nifi/pull/5438#discussion_r722637417



##########
File path: nifi-docs/src/main/asciidoc/administration-guide.adoc
##########
@@ -3760,7 +3764,8 @@ Providing three total network interfaces, including  `nifi.web.http.network.inte
 |`nifi.web.https.port.forwarding`|Same as `nifi.web.http.port.forwarding`, but with HTTPS for secure communication. It is blank by default.
 |`nifi.web.https.ciphersuites.include`|Cipher suites used to initialize the SSLContext of the Jetty HTTPS port.  If unspecified, the runtime SSLContext defaults are used.
 |`nifi.web.https.ciphersuites.exclude`|Cipher suites that may not be used by an SSL client to establish a connection to Jetty.  If unspecified, the runtime SSLContext defaults are used.
-
+|`nifi.web.max.access.token.requests.per.second`| Maximum amount of requests per second the token endpoints will accept before filtering requests. The default value is `25`
+|`nifi.web.should.send.server.version`| Whether the Server header should be included in HTTP responses. The default value is `TRUE` +

Review comment:
       Following the convention of other `boolean` properties, the default value is `true`, not `TRUE`.
   ```suggestion
   |`nifi.web.should.send.server.version`| Whether the Server header should be included in HTTP responses. The default value is `true` +
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org