Fix SonarQube error

[GeoDerp]

No description provided.

[github-advanced-security]

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

[GeoDerp]

@davidusb-geek , given time ill look into SonarQube web recommendations later. Im also looking into how we can send more data via rest. ie: curl ... -H "Transfer-Encoding: chunked"

[davidusb-geek]

Ok

Im also looking into how we can send more data via rest. ie: curl ... -H "Transfer-Encoding: chunked"

What do you mean by this? A new method to make it simpler to send data un a curl command?

[GeoDerp]

Ok

Im also looking into how we can send more data via rest. ie: curl ... -H "Transfer-Encoding: chunked"

What do you mean by this? A new method to make it simpler to send data un a curl command?

So currently users are hitting the limitations of sending too much data in one packet / request. using -H "Transfer-Encoding: chunked" in the curl command tells the call to split the data into chunks (so multiple packets for one call). However I defiantly still need to look into it.

[GeoDerp]

This is an example of this: 7ca105f

[codecov]

Codecov Report

Attention: Patch coverage is 54.54545% with 10 lines in your changes missing coverage. Please review.

Project coverage is 65.83%. Comparing base (93f7d30) to head (61cff61).
Report is 4 commits behind head on master.

Files with missing lines	Patch %	Lines
src/emhass/command_line.py	54.54%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #450      +/-   ##
==========================================
+ Coverage   65.79%   65.83%   +0.04%     
==========================================
  Files           8        8              
  Lines        3049     3053       +4     
==========================================
+ Hits         2006     2010       +4     
  Misses       1043     1043              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[davidusb-geek]

Hi @GeoDerp, is this ready to merge?

[GeoDerp]

Hi @GeoDerp, is this ready to merge?

Should be, would like to do one more double check that nothing is broken in the js files however.

[GeoDerp]

@davidusb-geek , cant change google osv just yet. They havent released a new version. Will keep en eye out for when it happens.

[GeoDerp]

Quick look looks good on this PR.

[GeoDerp]

@davidusb-geek there may be a cpu maxed out issue from the most reagent release. I not 100% sure what the cause is. But it sounds like it's a CBC package issue.

[davidusb-geek]

@davidusb-geek there may be a cpu maxed out issue from the most reagent release. I not 100% sure what the cause is. But it sounds like it's a CBC package issue.

Did you saw this on your own install? Or it was reported to you?
My install is running fine on the latest release, no maxed out CPU issue

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud