Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

2.2upgrade #2

Open
wants to merge 3,180 commits into
base: nimbis
Choose a base branch
from
Open

2.2upgrade #2

wants to merge 3,180 commits into from

Conversation

KavinKaviarasan1
Copy link

Added md5 patches to the django 2.2 fork

felixxm and others added 30 commits June 10, 2019 16:57
… and updated example of custom aggregate functions.

Backport of 76b3fc5 from master
…ule doesn't have __file__ attribute.

Backport of 8454f6d from master
…PROXY_SSL_HEADER if set.

An HTTP request would not be redirected to HTTPS when the
SECURE_PROXY_SSL_HEADER and SECURE_SSL_REDIRECT settings were used if
the proxy connected to Django via HTTPS.

HttpRequest.scheme will now always trust the SECURE_PROXY_SSL_HEADER if
set, rather than falling back to the request scheme when the
SECURE_PROXY_SSL_HEADER did not have the secure value.

Thanks to Gavin Wahl for the report and initial patch suggestion, and
Shai Berger for review.

Backport of 54d0f5e from master
…to email addresses without single quotes.

Backport of c2f381e from master
…ve superusers.

Equivalent note for PermissionsMixin was added in d33864e.
Backport of 4b32d03 from master
…s for custom UserAdmin with a custom user model.

Backport of c13e371 from master
….to_python() means no match.

Backport of f197c3d from master
felixxm and others added 24 commits June 2, 2020 10:47
"term" role became case sensitive in Sphinx 3.0.1.
Backport of cc70a03 from master
Support for sqlparse < 0.2.2 was broken in
782d85b because is_whitespace property
was added in sqlparse 0.2.2.

Backport of 4b6db76 from master.
Support for sqlparse < 0.2.2 was broken in
40b0a58 because is_whitespace property
was added in sqlparse 0.2.2.

Backport of 4339f2a from master.
This applies a patch to allow the use of Django on a FIPS-compliant
system with MD5 disabled. This utilizes the optional
'usedforsecurity=False' parameter that is passed into the hashlib.md5
function. This allows the use of MD5 even when MD5 has been disabled at
the system level.
Had to add the try/except loop to make the md5 fixes
work/pass during testing.
@KavinKaviarasan1 KavinKaviarasan1 requested a review from mandan July 14, 2020 19:04
@KavinKaviarasan1 KavinKaviarasan1 self-assigned this Jul 14, 2020
@KavinKaviarasan1
Copy link
Author

Copy link

@kevinrobbins kevinrobbins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could not find anymore instances of hashlib.md5() that did not have usedforsecurity added (with the exception of the exception handling calls).

@mandan
Copy link

mandan commented Jul 31, 2020

Thanks for the review @kevinrobbins . I don't think we should actually merge this into the nimbis branch, though. The best way to do this would probably be to have our separate release-2.2 branch that we sync with upstream, and then cherry pick the md5 commits on top of, for every new patch release. Then make our own release. No merge involved.

@kevinrobbins
Copy link

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.