search

ring-clojure / ring-headers

Ring middleware for common response headers
26 stars 12 forks source link

Adds middleware that adds charset to responses #1

Closed Confusion closed 9 years ago

Confusion commented 9 years ago

Adds the specified charset or "utf-8" to response maps with a content-type header that specifies a text-based media-type, but no charset.

As first suggested in https://github.com/ring-clojure/ring-defaults/issues/13.

weavejester commented 9 years ago

Other than a small stylistic change and the version number (which needs to be changed back for now), it looks good to go. Could you squash your changes after the version number has been put back?

weavejester commented 9 years ago

Thanks. One more small change if you're able: can you use imperative tense in your commit summary? e.g.

Add default charset middleware