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

Fix content-type handing for JSON requests #684

Merged
merged 1 commit into from
Jan 3, 2023
Merged

Conversation

frenck
Copy link
Owner

@frenck frenck commented Jan 3, 2023

Proposed Changes

Fixes a regression introduced in #677. Content headers for JSON data should be handled.
This fixes the controls of, e.g., protection mode.

@frenck frenck added the bugfix Inconsistencies or issues which will cause a problem for users or implementers. label Jan 3, 2023
@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #684 (e3e7b6c) into main (2c3e9e6) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #684   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          296       296           
  Branches        25        25           
=========================================
  Hits           296       296           
Impacted Files Coverage Δ
src/adguardhome/adguardhome.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@frenck frenck merged commit ac38f32 into main Jan 3, 2023
@frenck frenck deleted the frenck-2023-0028 branch January 3, 2023 12:32
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix Inconsistencies or issues which will cause a problem for users or implementers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant