Bug Fixes

[adamsclafani]

Changed 'bcrypt' to 'python3-bcrypt' which was causing the install script to fail #538.

Fixed bug where the hostname was None when checking for a preexisting internet connection

Updated flask caching due PyUpdev insecure warning.

[adamsclafani]

Update:

Pyupdev still complains about the latest version of flask caching so I'm reverting it back to what it was.

[codecov-commenter]

Codecov Report

Merging #547 (9697f93) into master (3bf8167) will decrease coverage by 1.39%.
The diff coverage is 54.08%.

@@            Coverage Diff             @@
##           master     #547      +/-   ##
==========================================
- Coverage   76.71%   75.32%   -1.40%     
==========================================
  Files          45       48       +3     
  Lines        2766     2946     +180     
==========================================
+ Hits         2122     2219      +97     
- Misses        644      727      +83     

Impacted Files	Coverage Δ
opwen_email_client/webapp/tasks.py	43.58% <23.07%> (-3.08%)	⬇️
opwen_email_client/util/network.py	30.00% <30.00%> (ø)
opwen_email_client/webapp/actions.py	34.34% <30.95%> (-0.92%)	⬇️
opwen_email_client/webapp/views.py	41.25% <33.33%> (-0.34%)	⬇️
opwen_email_client/webapp/forms/register.py	45.23% <45.23%> (ø)
opwen_email_client/webapp/forms/email.py	34.94% <60.00%> (-0.38%)	⬇️
opwen_email_server/services/sendgrid.py	91.79% <63.63%> (-2.61%)	⬇️
opwen_email_server/mailers/wikipedia.py	97.77% <97.77%> (ø)
opwen_email_client/webapp/config.py	100.00% <100.00%> (ø)
opwen_email_server/mailers/__init__.py	100.00% <100.00%> (ø)
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2785a32...9697f93. Read the comment docs.

[holta]

@adamsclafani many thanks!

Do you have any testing recommendations for @nzola @jvonau @deldesir to try this out?

[jvonau]

The fix for Flask-Caching is in 1.10.2 or later.
Have a look at https://pyup.io/repos/github/ascoderu/lokole/ and click on 'Requirements' for an overall perspective.

[jvonau]

good enough for now.

[holta]

@adamsclafani should this PR be merged today, to advance testing prior to the Lokole call 4PM PDT tomorrow?

(Or thanks for clarifying what next step's best!)

[holta]

@adamsclafani should this PR be merged today, to advance testing prior to the Lokole call 4PM PDT tomorrow?

(Or thanks for clarifying what next step's best!)

Please also check out the progress that occurred over the past 3 hours here:

• PR Create lokole user iiab/iiab#2782 "WIP lokole - Needs unique uuid for each new install"