added colors to simple defaults for liberator #313
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix for save-logs! to avoid 'GC overhead limit exceeded'
Fixes clojure-liberator#199 Bad qvals in these headers will now cause a 400 status code to be returned.
This drops JDK7, as JDK8 has become the new baseline. Also, Clojure 1.10.0 drops support for JDK7.
…ld-matrix Expand build matrix and add new Clojure versions
``` (defresource example "This is my awesome resource!" ...) ```
…e-docstring Allow defresource to have a docstring
The `combine` function attempts to add new values to a collection by merging them. However, for lists and a vectors, a lazy sequence is returned rather than a list or a vector. For lists, `(apply list (concat ...` is used instead of `(into ...` in order to preserve element order for backwards compatibility, as `into` would put the new elements at the head of the list. For vectors, `into` is used, as new elements are added to the tail. The implementation for sets is simplified.
…mbine-results Improve `combine` to not return lazy sequences
Handle malformed Accept-Language Accept-Charset & Accept-Encoding
The default modified-since? decision would always evaluate to false when the request contained an "if-modified-since" header but the response didn't contain a "last-modified" header. This seems wrong because the decision is not actually able to determine whether the resource has been modified or not. This patch makes it evaluate to true instead which seems more reasonable in this situation.
|
There is also https://github.com/Tavistock/liberator/tree/trm.default-graph-2 which adds grayed out boxes, I was trying to emulate http://clojure-liberator.github.io/liberator/doc/decisions.html but idk how to do it automatically. Maybe adding a |
Tavistock
force-pushed
the
trm.default-graph
branch
2 times, most recently
from
d2e59d8
to
00415ac
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #308. Doesn't account for defaults that are richer than just
trueorfalse