-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v4 model non-string route parameters are undefined #285
Comments
Hi @e-karlsson, this appears to be working for me: Can you confirm which version of the |
Hi @ejizba ! Thank you for taking the time. Very interesting. Here is the version I have in the Also, I am running on Mac if that can make any difference here. |
Yeah I'm on a mac as well. If you check your package-lock.json file, it should display the specific version of the npm package being used. For example:
Could you send screenshots of what you're doing? It might allow me to catch a difference. |
Yeah two more things:
|
Okay, I figured out our difference! Projects created recently will have a new http stream feature enabled by default (more info here), but older projects (like I was testing on) will not. I can repro the issue with http streaming turned on and will look into a fix. Thanks for reporting! In the meantime, you can workaround this by turning http streaming off. In your |
Hi @ejizba ! I have confirmed another issue with |
Hi @e-karlsson, please file a new issue with detailed repro steps |
Hi @e-karlsson we just released a fix in v4.5.1. Thanks for your help! FYI @emilselroos |
We are migrating from v3 to v4 currently and have reached a critical problem. It seems like the routing does not work with numbers/int.
I took the basic example for Docs: https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-http-webhook-trigger?tabs=python-v2%2Cisolated-process%2Cnodejs-v4%2Cfunctionsv2&pivots=programming-language-typescript#customize-the-http-endpoint
We can see that the category is resolving perfectly fine, but the id is undefined.
No matter if I remove the route constraint it still is undefined.
System:
The text was updated successfully, but these errors were encountered: