We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I saw that in recent update, Http.headers had became EitherType<String, Array<String>>, I suppose to support the set-cookie header which is an array.
Http.headers
EitherType<String, Array<String>>
set-cookie
But I'm wondering why this haven't been changed in IncomingMessage. Was there a compatibility issue? If not, I'll make a PR to change it as well.
IncomingMessage
Thank you
The text was updated successfully, but these errors were encountered:
Looks like an oversight, PR welcome :)
Sorry, something went wrong.
Change IncomingMessage.headers to EitherType
fbcb80a
Follow the change of API for `HttpOptions.headers` to support Array headers. Closes HaxeFoundation#61
Change HttpRequestOptions.headers to DynamicAccess
ea3b1cf
Change the type to match `HttpOptions.headers` and better access to the headers. Same as HaxeFoundation#61
No branches or pull requests
I saw that in recent update,
Http.headers
had becameEitherType<String, Array<String>>
, I suppose to support theset-cookie
header which is an array.But I'm wondering why this haven't been changed in
IncomingMessage
. Was there a compatibility issue? If not, I'll make a PR to change it as well.Thank you
The text was updated successfully, but these errors were encountered: