use response.content (bytes) instead of response.text (encoded text)

[ryanmelt]

No description provided.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 75.00%. Comparing base (e825c03) to head (9fa24a8).
Report is 2 commits behind head on main.

Files	Patch %	Lines
openc3/python/openc3/script/storage.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1148      +/-   ##
==========================================
+ Coverage   74.98%   75.00%   +0.02%     
==========================================
  Files         594      594              
  Lines       43900    43900              
  Branches      757      757              
==========================================
+ Hits        32917    32928      +11     
+ Misses      10899    10888      -11     
  Partials       84       84              

Flag	Coverage Δ
frontend	55.31% <ø> (+0.01%)	⬆️
python	83.04% <0.00%> (+0.07%)	⬆️
ruby-api	48.73% <ø> (ø)
ruby-backend	80.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[jmthomas]

How did this ever work in the docker compose version? Why was this unique to k8?

[ryanmelt]

The difference is in docker compose we are using local mode which is a different code path than k8s which doesn't support local mode.