search

busy-beaver-dev / busy-beaver

The Chicago Python Community Engagement Slack bot
https://busybeaverbot.com
MIT License
70 stars 23 forks source link

Add secure.py #321

Closed alysivji closed 4 years ago

alysivji commented 4 years ago

What does this do

Remove Flask-Talisman and add secure.py.

Why are we doing this

I use the secure.py library for Django projects and it works as advertised. Flask Talisman requires configuration; secure.py does not.

How should this be tested

Tested in staging

Migrations

no

Dependencies

no

Callouts

no

codecov-commenter commented 4 years ago

Codecov Report

Merging #321 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #321   +/-   ##
=======================================
  Coverage   92.48%   92.48%           
=======================================
  Files          71       71           
  Lines        2394     2394           
=======================================
  Hits         2214     2214           
  Misses        180      180
Impacted Files Coverage Δ
busy_beaver/apps/debug/api/healthcheck.py 81.81% <ø> (-3.90%) :arrow_down:
...y_beaver/apps/github_integration/summary/blocks.py 100.00% <ø> (ø)
busy_beaver/extensions.py 100.00% <ø> (ø)
busy_beaver/app.py 98.03% <100.00%> (+0.26%) :arrow_up:

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 c23797b...2e73e40. Read the comment docs.