Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Allow explicitly set content-type via head method when status code allows it according to the RFCs #6201

Merged
merged 2 commits into from May 8, 2012

Conversation

Projects
None yet
3 participants
Contributor

whistlerbrk commented May 8, 2012

Creative title.

@josevalim as per your request, the same pull request made against master which are you already pulled into 3-2-stable ( in #6198 )

@twinturbo addresses your concern, no conflict, tests pass and compatible with your accepted pull ( #6148 )

@josevalim josevalim added a commit that referenced this pull request May 8, 2012

@josevalim josevalim Merge pull request #6201 from whistlerbrk/master
Allow explicitly set content-type via head method when status code allows it according to the RFCs
fc5c41e

@josevalim josevalim merged commit fc5c41e into rails:master May 8, 2012

Contributor

josevalim commented May 8, 2012

Awesome, thanks <3

ahawkins commented May 8, 2012

K cool! Glad that is sorted out. Now #head works like it should. Cheers @whistlerbrk @josevalim

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment