search

laminas-api-tools / api-tools-admin

Laminas API Tools Admin module
https://api-tools.getlaminas.org/documentation
BSD 3-Clause "New" or "Revised" License
13 stars 21 forks source link

Fixes Accept header to allow legacy and laminas-api-tools #54

Closed michalbundyra closed 4 years ago

michalbundyra commented 4 years ago
Q A
Bugfix yes/no

Description

On migration accept/content-type header has been changed to invalid value. Now we accepts both:

michalbundyra commented 4 years ago

@weierophinney We might want apply this patch to all tags. This is migration issue. Not sure what we should do with application/vnd.laminascampus.v1.config+json. I think we might want to keep it as well, as some people might already migrate and use that invalid value.

Thoughts?

froschdesign commented 4 years ago

@michalbundyra

I think we might want to keep it as well, as some people might already migrate and use that invalid value.

I had the same thought. I'm for keeping the wrong name and open an issue report to remove it with the next major version.