Unexpected default vendor accept header.

[yoonsy]

According to a middleware spec, ApiVersions::Middleware should add a default vendor accept to a nil Accept header.

But, in my opinion, it adds default vendor only if request path contains prefix path like /api. Otherwise, It adds default vendor when request path is / and Accept header is */*.

What do you think of this?

Thank you for sharing this project. 👍

[yoonsy]

I forked this project and changed the code like this.

I fixed the api prefix to /api, so I didn't open PR.

[rposborne]

Agreed, adding the header to all can make controller tests suddenly think they are getting json. (when they have nothing to do with the API)