Fix the query datetime support issue #1884
Conversation
| import { RequestParameters } from "@azure-rest/core-client"; | ||
|
|
||
| export interface TestHeaders { | ||
| "execution-to"?: string; |
There was a problem hiding this comment.
why is this different with what has been defined in the typespec ?
There was a problem hiding this comment.
This handled by tsp compiler not us.
There was a problem hiding this comment.
@timotheeguerin Hi, do you have any context on this ? The only document I can found about header is that they are case insensitive, but it doesn't say this dash mark can also be ignored. https://datatracker.ietf.org/doc/html/rfc7230#section-3.2
There was a problem hiding this comment.
The header decorator automatically convert camelCase to camel-case as the header value if you don’t specify an explicit value
There was a problem hiding this comment.
Is this documented somewhere? I mean in TypeSpec documentation.
There was a problem hiding this comment.
I thought we did but I can't seem to find it. Will create an issue
Generate the datetime as
Date | stringwhen the parameter is in query & model property, asstringin header.