oscerd opened a new pull request, #25563:
URL: https://github.com/apache/camel/pull/25563
`createFunction` fed three headers straight into protobuf setters:
```java
final String entryPoint =
exchange.getIn().getHeader(GoogleCloudFunctionsConstants.ENTRY_POINT,
String.class);
final String runtime =
exchange.getIn().getHeader(GoogleCloudFunctionsConstants.RUNTIME, String.class);
final String sourceArchiveUrl =
exchange.getIn().getHeader(GoogleCloudFunctionsConstants.SOURCE_ARCHIVE_URL,
String.class);
CloudFunction function = CloudFunction.newBuilder()
.setName(...).setEntryPoint(entryPoint)
.setRuntime(runtime)...
.setSourceArchiveUrl(sourceArchiveUrl).build();
```
Protobuf setters reject `null`, so a request missing any of them failed with
a bare
`NullPointerException` and no hint about which header the operation needs.
Each is now validated and
reported by name.
`listFunctions` also built its request with `setPageSize(Integer.MAX_VALUE)`
while collecting the
result through `iterateAll()`, which follows the pages itself. The oversized
page size is dropped —
the request the service sees is the paging default and the returned list is
unchanged.
Covered by a new test on the existing gRPC mock-service base.
_Claude Code on behalf of oscerd_
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
