marksweb / django-bleach

Bleach is a Python module that takes any HTML input, and returns valid, sanitised HTML that contains only an allowed subset of HTML tags, attributes and styles. django-bleach is a Django app that makes using bleach extremely easy.
MIT License
148 stars 23 forks source link

Added option to allow relative links #17

Closed azaitsev closed 2 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #17 (33b3f42) into master (bb2a291) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          103       103           
  Branches        20        20           
=========================================
  Hits           103       103           
Impacted Files Coverage Δ
django_bleach/utils.py 100.00% <ø> (ø)

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 bb2a291...33b3f42. Read the comment docs.

marksweb commented 3 years ago

Could you please add some content to this PR to (at least) explain what it includes.

I have now added a PR template which might help you with what to include;

# Description

Describe:

* Content of the pull request
* Feature added / Problem fixed

## References

Provide any github issue fixed (as in ``Fix #XYZ``)

# Checklist

* [ ] I have ran `tox` to ensure tests pass
* [ ] Usage documentation added in case of new features
* [ ] Tests added / I have not lowered coverage from 100%
marksweb commented 3 years ago

@azaitsev it'd be good to get this merged. Can you take a look at the conflict please?

Laityned commented 2 years ago

allow_relative support is dropped by bleach in 2013 but there README is not up to date.

marksweb commented 2 years ago

Thanks @Laityned

In that case I'll close this.