Azure / Azure/azure-functions-java-worker

Optional route parameter without type is not correctly parsed

未关闭
#293 1 条评论 0 个 reaction 已指派 1 人 已被 @kamperiadis 认领 在 GitHub 查看
area:java-functions New Feature reviewed
主要语言
Java
星标
103
派生
74
平均合并
4 天 8 小时
30 天内合并 PR
2

描述

#### Investigative information

- Timestamp: 2019-05-26 05:43:59.033 UTC
- Invocation ID: 90a11823-2911-41aa-969c-f228f75865a0
- Region: France Central

#### Repro steps

*This behavior is only reproductible when the function app is executed on Azure, and it does not appear when running locally*

Run an Http triggered Function App which contains an optional route parameter without type specified with it, as for example : https://example.azurewebsites.net/api/functionname/{parameter?}.
By parameter with type I mean a parameter like {parameter:type} which is not used here.

#### Expected behavior

The expected behavior would be to get the actual value provided for the parameter.

#### Actual behavior

The optional parameter without type is not parsed correctly:
- if the function is executed without a value specified for this parameter, it ends with a NullPointerException :

`Result: Failure
Exception: NullPointerException:
Stack: java.lang.NullPointerException
at com.microsoft.azure.functions.worker.binding.BindingDataStore.getTriggerMetatDataByName(BindingDataStore.java:54)
at com.microsoft.azure.functions.worker.broker.ParameterResolver.resolve(ParameterResolver.java:62)
at com.microsoft.azure.functions.worker.broker.ParameterResolver.resolve(ParameterResolver.java:42)
at com.microsoft.azure.functions.worker.broker.JavaMethodExecutor.execute(JavaMethodExecutor.java:52)
at com.microsoft.azure.functions.worker.broker.JavaFunctionBroker.invokeMethod(JavaFunctionBroker.java:51)
at com.microsoft.azure.functions.worker.handler.InvocationRequestHandler.execute(InvocationRequestHandler.java:33)
at com.microsoft.azure.functions.worker.handler.InvocationRequestHandler.execute(InvocationRequestHandler.java:10)
at com.microsoft.azure.functions.worker.handler.MessageHandler.handle(MessageHandler.java:45)
at com.microsoft.azure.functions.worker.JavaWorkerClient$StreamingMessagePeer.lambda$onNext$0(JavaWorkerClient.java:92)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
`

- if a value is given to this parameter in the triggering URL, the parameter is incorrectly parsed and the returned value is the name of the parameter until the question mark.
E.g. with the example URL I gave before, if we call :
-- https://example.azurewebsites.net/api/functionname : the NPE is thrown ;
-- https://example.azurewebsites.net/api/functionname/avalue : instead of the value, I got "{parameter".

#### Known workarounds

Optional parameters with a type (e.g. "{parameter:alpha?}") cannot be obtained with the annotation `@BindingName` as it also returns a NPE, but it is nevertheless possible to retrieve their values by parsing the URL with `request.getUri().getPath()`.
Non-optional parameters, that is, without a question mark, work fine.

#### Related information

* Programming language used = Java 8
* Bindings used = http binding

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。