Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v0.17.3 #711

Merged
merged 4 commits into from
May 10, 2022
Merged

Release v0.17.3 #711

merged 4 commits into from
May 10, 2022

Conversation

alexanderkiel
Copy link
Member

No description provided.

This fixes a regression introduced by #703 where we don't return JSON
by default if an Accept header is present. This is relevant in
situations where browsers start with text/html but also will accept
application/xml with lower priority. Instead of returning a 406 in this
cases, XML is delivered now.

If one prefers JSON in browsers the _format query param has to be used.
@alexanderkiel alexanderkiel self-assigned this May 10, 2022
@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #711 (2591dc6) into master (d5b0c68) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #711   +/-   ##
=======================================
  Coverage   93.64%   93.64%           
=======================================
  Files         237      237           
  Lines       14174    14174           
  Branches      356      356           
=======================================
  Hits        13273    13273           
  Misses        545      545           
  Partials      356      356           
Impacted Files Coverage Δ
...dules/rest-api/src/blaze/rest_api/capabilities.clj 95.00% <ø> (ø)
.../rest-api/src/blaze/rest_api/middleware/output.clj 100.00% <100.00%> (ø)

@alexanderkiel alexanderkiel merged commit 9d9c9e2 into master May 10, 2022
@alexanderkiel alexanderkiel deleted the release-v0.17.3 branch May 10, 2022 15:55
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.

1 participant