Skip to content
This repository has been archived by the owner on Feb 27, 2022. It is now read-only.

Cache:false prevents caching the template #5

Closed
wants to merge 2 commits into from

Conversation

adamghill
Copy link
Contributor

Should fix issue #4.

@isaacs
Copy link
Owner

isaacs commented Oct 15, 2012

I landed this on 54fefe8.

The content-length:0 thing is weird. I need that to make it work on my system, both on node 0.8 and master. What version of node are you using?

@isaacs isaacs closed this Oct 15, 2012
@adamghill
Copy link
Contributor Author

Hmm, content-length:0 fails for me on both 0.8.5 and 0.8.12 in OSX. Here are the headers I see for test/basic.js "/foo first" test:

{ etag: '"KuRI//bxAVWXwkkHpV5fzXLZU8I="',
  'x-templar-stamp': 'pid88285',
  'content-type': 'text/html',
  date: 'Tue, 16 Oct 2012 11:54:36 GMT',
  connection: 'keep-alive',
  'transfer-encoding': 'chunked' }

@isaacs
Copy link
Owner

isaacs commented Oct 16, 2012

Aha, looks like the version of request changes things. I just updated,
I'll remove them now.

On Tue, Oct 16, 2012 at 4:57 AM, Adam notifications@github.com wrote:

Hmm, content-length:0 fails for me on both 0.8.5 and 0.8.12 in OSX. Here
are the headers I see for test/basic.js "/foo first" test:

{ etag: '"KuRI//bxAVWXwkkHpV5fzXLZU8I="',
'x-templar-stamp': 'pid88285',
'content-type': 'text/html',
date: 'Tue, 16 Oct 2012 11:54:36 GMT',
connection: 'keep-alive',
'transfer-encoding': 'chunked' }


Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-9481943.

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.

None yet

2 participants