Skip to content
This repository was archived by the owner on Apr 4, 2026. It is now read-only.

fix issue 289: Transfer-Encoding chunked header duplicated#290

Merged
dlecan merged 1 commit into
develop-2.2.xfrom
issue-#289_duplicated_chunked_header
Jul 8, 2015
Merged

fix issue 289: Transfer-Encoding chunked header duplicated#290
dlecan merged 1 commit into
develop-2.2.xfrom
issue-#289_duplicated_chunked_header

Conversation

@yanns

@yanns yanns commented Jul 1, 2015

Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we sure this header is set by all containers ?

Why did that work before ?
More, before what ? With Play 2.1.x ? Or with earlier versions of Tomcat ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually, it is working because the header is duplicated. But most of HTTP clients deal with that.
It is only a trouble for some picky clients... ;)

As far as I know, the containers set this header automatically.
But I have not searched the JEE specs to check if the container must set this header.

It is a minor issue in my opinion.

dlecan added a commit that referenced this pull request Jul 8, 2015
…eader

fix issue 289: Transfer-Encoding chunked header duplicated
@dlecan dlecan merged commit f75bcca into develop-2.2.x Jul 8, 2015
@yanns yanns deleted the issue-#289_duplicated_chunked_header branch July 9, 2015 09:20
@dlecan dlecan added this to the 1.2.1 milestone Jul 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants