Skip to content

Commit

Permalink
fix: Unpin SAML library.
Browse files Browse the repository at this point in the history
We were using an old version of the python3-saml library,
which was causing issues with newer versions of social-core.

The reason it was pinned was because our etree implementation
didn't support several fields that the saml library did, so
we are now importing those entities as well.
  • Loading branch information
dianakhuang committed May 2, 2023
1 parent bdd2a0d commit 75b4429
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion openedx/core/lib/safe_lxml/etree.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

from lxml.etree import XMLParser as _XMLParser
from lxml.etree import * # lint-amnesty, pylint: disable=redefined-builtin
from lxml.etree import _Element, _ElementTree
from lxml.etree import _Comment, _Element, _ElementTree, _Entity, _ProcessingInstruction

# This should be imported after lxml.etree so that it overrides the following attributes.
from defusedxml.lxml import XML, fromstring, parse
Expand Down
3 changes: 0 additions & 3 deletions requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,6 @@ matplotlib<3.4.0
# Major upgrade will be done in separate ticket.
pymongo<4.0.0

# python3-saml==1.10.0 version started breaking a11y tests
python3-saml<1.10.0

# greater version has breaking changes and requires some migration steps.
django-webpack-loader==0.7.0

Expand Down

0 comments on commit 75b4429

Please sign in to comment.