This is an automated email from the ASF dual-hosted git repository.

acosentino pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 52b53bda872838ef95edb399e0df96ef55ee60d7
Author: Andrea Cosentino <anco...@gmail.com>
AuthorDate: Thu Mar 4 06:46:57 2021 +0100

    CAMEL-16273 - Camel-google-* cloud components: Make the serviceAccountKey 
explicitly configurable - Camel-google-pubsub, mark service account key as 
required
---
 .../org/apache/camel/catalog/docs/google-pubsub-component.adoc         | 2 +-
 .../org/apache/camel/component/google/pubsub/google-pubsub.json        | 2 +-
 .../camel-google-pubsub/src/main/docs/google-pubsub-component.adoc     | 2 +-
 .../org/apache/camel/component/google/pubsub/GooglePubsubEndpoint.java | 1 +
 .../camel/builder/endpoint/dsl/GooglePubsubEndpointBuilderFactory.java | 3 +++
 docs/components/modules/ROOT/pages/google-pubsub-component.adoc        | 2 +-
 6 files changed, 8 insertions(+), 4 deletions(-)

diff --git 
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/google-pubsub-component.adoc
 
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/google-pubsub-component.adoc
index ebfe967..51a99a1 100644
--- 
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/google-pubsub-component.adoc
+++ 
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/google-pubsub-component.adoc
@@ -91,7 +91,7 @@ with the following path and query parameters:
 | *concurrentConsumers* (common) | The number of parallel streams consuming 
from the subscription | 1 | Integer
 | *loggerId* (common) | Logger ID to use when a match to the parent route 
required |  | String
 | *maxMessagesPerPoll* (common) | The max number of messages to receive from 
the server in a single API call | 1 | Integer
-| *serviceAccountKey* (common) | The Service account key that can be used as 
credentials for the PubSub publisher/subscriber. It can be loaded by default 
from classpath, but you can prefix with classpath:, file:, or http: to load the 
resource from different systems. |  | String
+| *serviceAccountKey* (common) | *Required* The Service account key that can 
be used as credentials for the PubSub publisher/subscriber. It can be loaded by 
default from classpath, but you can prefix with classpath:, file:, or http: to 
load the resource from different systems. |  | String
 | *synchronousPull* (common) | Synchronously pull batches of messages | false 
| boolean
 | *bridgeErrorHandler* (consumer) | Allows for bridging the consumer to the 
Camel routing Error Handler, which mean any exceptions occurred while the 
consumer is trying to pickup incoming messages, or the likes, will now be 
processed as a message and handled by the routing Error Handler. By default the 
consumer will use the org.apache.camel.spi.ExceptionHandler to deal with 
exceptions, that will be logged at WARN or ERROR level and ignored. | false | 
boolean
 | *exceptionHandler* (consumer) | To let the consumer use a custom 
ExceptionHandler. Notice if the option bridgeErrorHandler is enabled then this 
option is not in use. By default the consumer will deal with exceptions, that 
will be logged at WARN or ERROR level and ignored. |  | ExceptionHandler
diff --git 
a/components/camel-google-pubsub/src/generated/resources/org/apache/camel/component/google/pubsub/google-pubsub.json
 
b/components/camel-google-pubsub/src/generated/resources/org/apache/camel/component/google/pubsub/google-pubsub.json
index 2d70f20..822febb 100644
--- 
a/components/camel-google-pubsub/src/generated/resources/org/apache/camel/component/google/pubsub/google-pubsub.json
+++ 
b/components/camel-google-pubsub/src/generated/resources/org/apache/camel/component/google/pubsub/google-pubsub.json
@@ -37,7 +37,7 @@
     "concurrentConsumers": { "kind": "parameter", "displayName": "Concurrent 
Consumers", "group": "common", "label": "", "required": false, "type": 
"integer", "javaType": "java.lang.Integer", "deprecated": false, "autowired": 
false, "secret": false, "defaultValue": "1", "description": "The number of 
parallel streams consuming from the subscription" },
     "loggerId": { "kind": "parameter", "displayName": "Logger Id", "group": 
"common", "label": "", "required": false, "type": "string", "javaType": 
"java.lang.String", "deprecated": false, "autowired": false, "secret": false, 
"description": "Logger ID to use when a match to the parent route required" },
     "maxMessagesPerPoll": { "kind": "parameter", "displayName": "Max Messages 
Per Poll", "group": "common", "label": "", "required": false, "type": 
"integer", "javaType": "java.lang.Integer", "deprecated": false, "autowired": 
false, "secret": false, "defaultValue": "1", "description": "The max number of 
messages to receive from the server in a single API call" },
-    "serviceAccountKey": { "kind": "parameter", "displayName": "Service 
Account Key", "group": "common", "label": "common", "required": false, "type": 
"string", "javaType": "java.lang.String", "deprecated": false, "autowired": 
false, "secret": false, "description": "The Service account key that can be 
used as credentials for the PubSub publisher\/subscriber. It can be loaded by 
default from classpath, but you can prefix with classpath:, file:, or http: to 
load the resource from different [...]
+    "serviceAccountKey": { "kind": "parameter", "displayName": "Service 
Account Key", "group": "common", "label": "common", "required": true, "type": 
"string", "javaType": "java.lang.String", "deprecated": false, 
"deprecationNote": "", "autowired": false, "secret": false, "description": "The 
Service account key that can be used as credentials for the PubSub 
publisher\/subscriber. It can be loaded by default from classpath, but you can 
prefix with classpath:, file:, or http: to load the r [...]
     "synchronousPull": { "kind": "parameter", "displayName": "Synchronous 
Pull", "group": "common", "label": "", "required": false, "type": "boolean", 
"javaType": "boolean", "deprecated": false, "autowired": false, "secret": 
false, "defaultValue": false, "description": "Synchronously pull batches of 
messages" },
     "bridgeErrorHandler": { "kind": "parameter", "displayName": "Bridge Error 
Handler", "group": "consumer", "label": "consumer", "required": false, "type": 
"boolean", "javaType": "boolean", "deprecated": false, "autowired": false, 
"secret": false, "defaultValue": false, "description": "Allows for bridging the 
consumer to the Camel routing Error Handler, which mean any exceptions occurred 
while the consumer is trying to pickup incoming messages, or the likes, will 
now be processed as a m [...]
     "exceptionHandler": { "kind": "parameter", "displayName": "Exception 
Handler", "group": "consumer (advanced)", "label": "consumer,advanced", 
"required": false, "type": "object", "javaType": 
"org.apache.camel.spi.ExceptionHandler", "optionalPrefix": "consumer.", 
"deprecated": false, "autowired": false, "secret": false, "description": "To 
let the consumer use a custom ExceptionHandler. Notice if the option 
bridgeErrorHandler is enabled then this option is not in use. By default the 
con [...]
diff --git 
a/components/camel-google-pubsub/src/main/docs/google-pubsub-component.adoc 
b/components/camel-google-pubsub/src/main/docs/google-pubsub-component.adoc
index ebfe967..51a99a1 100644
--- a/components/camel-google-pubsub/src/main/docs/google-pubsub-component.adoc
+++ b/components/camel-google-pubsub/src/main/docs/google-pubsub-component.adoc
@@ -91,7 +91,7 @@ with the following path and query parameters:
 | *concurrentConsumers* (common) | The number of parallel streams consuming 
from the subscription | 1 | Integer
 | *loggerId* (common) | Logger ID to use when a match to the parent route 
required |  | String
 | *maxMessagesPerPoll* (common) | The max number of messages to receive from 
the server in a single API call | 1 | Integer
-| *serviceAccountKey* (common) | The Service account key that can be used as 
credentials for the PubSub publisher/subscriber. It can be loaded by default 
from classpath, but you can prefix with classpath:, file:, or http: to load the 
resource from different systems. |  | String
+| *serviceAccountKey* (common) | *Required* The Service account key that can 
be used as credentials for the PubSub publisher/subscriber. It can be loaded by 
default from classpath, but you can prefix with classpath:, file:, or http: to 
load the resource from different systems. |  | String
 | *synchronousPull* (common) | Synchronously pull batches of messages | false 
| boolean
 | *bridgeErrorHandler* (consumer) | Allows for bridging the consumer to the 
Camel routing Error Handler, which mean any exceptions occurred while the 
consumer is trying to pickup incoming messages, or the likes, will now be 
processed as a message and handled by the routing Error Handler. By default the 
consumer will use the org.apache.camel.spi.ExceptionHandler to deal with 
exceptions, that will be logged at WARN or ERROR level and ignored. | false | 
boolean
 | *exceptionHandler* (consumer) | To let the consumer use a custom 
ExceptionHandler. Notice if the option bridgeErrorHandler is enabled then this 
option is not in use. By default the consumer will deal with exceptions, that 
will be logged at WARN or ERROR level and ignored. |  | ExceptionHandler
diff --git 
a/components/camel-google-pubsub/src/main/java/org/apache/camel/component/google/pubsub/GooglePubsubEndpoint.java
 
b/components/camel-google-pubsub/src/main/java/org/apache/camel/component/google/pubsub/GooglePubsubEndpoint.java
index 4c8b8bd..a57937a 100644
--- 
a/components/camel-google-pubsub/src/main/java/org/apache/camel/component/google/pubsub/GooglePubsubEndpoint.java
+++ 
b/components/camel-google-pubsub/src/main/java/org/apache/camel/component/google/pubsub/GooglePubsubEndpoint.java
@@ -57,6 +57,7 @@ public class GooglePubsubEndpoint extends DefaultEndpoint {
     @UriParam(label = "common",
               description = "The Service account key that can be used as 
credentials for the PubSub publisher/subscriber. It can be loaded by default 
from "
                             + " classpath, but you can prefix with classpath:, 
file:, or http: to load the resource from different systems.")
+    @Metadata(required = true)
     private String serviceAccountKey;
 
     @UriParam(name = "loggerId", description = "Logger ID to use when a match 
to the parent route required")
diff --git 
a/core/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/GooglePubsubEndpointBuilderFactory.java
 
b/core/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/GooglePubsubEndpointBuilderFactory.java
index 82882f5..d42733f 100644
--- 
a/core/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/GooglePubsubEndpointBuilderFactory.java
+++ 
b/core/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/GooglePubsubEndpointBuilderFactory.java
@@ -165,6 +165,7 @@ public interface GooglePubsubEndpointBuilderFactory {
          * 
          * The option is a: &lt;code&gt;java.lang.String&lt;/code&gt; type.
          * 
+         * Required: true
          * Group: common
          * 
          * @param serviceAccountKey the value to set
@@ -469,6 +470,7 @@ public interface GooglePubsubEndpointBuilderFactory {
          * 
          * The option is a: &lt;code&gt;java.lang.String&lt;/code&gt; type.
          * 
+         * Required: true
          * Group: common
          * 
          * @param serviceAccountKey the value to set
@@ -792,6 +794,7 @@ public interface GooglePubsubEndpointBuilderFactory {
          * 
          * The option is a: &lt;code&gt;java.lang.String&lt;/code&gt; type.
          * 
+         * Required: true
          * Group: common
          * 
          * @param serviceAccountKey the value to set
diff --git a/docs/components/modules/ROOT/pages/google-pubsub-component.adoc 
b/docs/components/modules/ROOT/pages/google-pubsub-component.adoc
index 7df1b0d..e180625 100644
--- a/docs/components/modules/ROOT/pages/google-pubsub-component.adoc
+++ b/docs/components/modules/ROOT/pages/google-pubsub-component.adoc
@@ -93,7 +93,7 @@ with the following path and query parameters:
 | *concurrentConsumers* (common) | The number of parallel streams consuming 
from the subscription | 1 | Integer
 | *loggerId* (common) | Logger ID to use when a match to the parent route 
required |  | String
 | *maxMessagesPerPoll* (common) | The max number of messages to receive from 
the server in a single API call | 1 | Integer
-| *serviceAccountKey* (common) | The Service account key that can be used as 
credentials for the PubSub publisher/subscriber. It can be loaded by default 
from classpath, but you can prefix with classpath:, file:, or http: to load the 
resource from different systems. |  | String
+| *serviceAccountKey* (common) | *Required* The Service account key that can 
be used as credentials for the PubSub publisher/subscriber. It can be loaded by 
default from classpath, but you can prefix with classpath:, file:, or http: to 
load the resource from different systems. |  | String
 | *synchronousPull* (common) | Synchronously pull batches of messages | false 
| boolean
 | *bridgeErrorHandler* (consumer) | Allows for bridging the consumer to the 
Camel routing Error Handler, which mean any exceptions occurred while the 
consumer is trying to pickup incoming messages, or the likes, will now be 
processed as a message and handled by the routing Error Handler. By default the 
consumer will use the org.apache.camel.spi.ExceptionHandler to deal with 
exceptions, that will be logged at WARN or ERROR level and ignored. | false | 
boolean
 | *exceptionHandler* (consumer) | To let the consumer use a custom 
ExceptionHandler. Notice if the option bridgeErrorHandler is enabled then this 
option is not in use. By default the consumer will deal with exceptions, that 
will be logged at WARN or ERROR level and ignored. |  | ExceptionHandler

Reply via email to