Skip to content

Conversation

CheungJ
Copy link
Contributor

@CheungJ CheungJ commented Jun 14, 2016

... and search for those file types


will serve the relative folder via port 5000

If you want canned to iterate through all accepted content types in the `Accept` header, use
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe better

If you want canned to accept the first matching content type when sending multiple Accept headers you can pass

$canned --relaxed-accept ./my/responses

I would also prefer to have that be a --relaxed-accept without the true, as well as not having the word iterate as it seems very implementation specific.

Copy link
Contributor Author

@CheungJ CheungJ Jun 23, 2016

Choose a reason for hiding this comment

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

Yes, I agree that it shouldn't be implementation specific. I'm OK with --relaxed_accept but I think that it should be --relaxed_accept=true to remain consistent with how the flag for cors is used -- i.e. --cors=true. Of course, it's up to you since you're the owner of the library :)

@sideshowcoder
Copy link
Owner

Sorry took me a long time to review, it has been a crazy week. But I like the PR a lot some small comments and I'm gonna merge this happily. Thanks so much for your work!

@sideshowcoder sideshowcoder merged commit 1b22287 into sideshowcoder:master Jun 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants