mirror of
https://git.anonymousland.org/anonymousland/synapse.git
synced 2024-12-26 01:19:21 -05:00
Remove unused compare_digest
function. (#10706)
This commit is contained in:
parent
e62cdbef1a
commit
029b7ad7b9
1
changelog.d/10706.misc
Normal file
1
changelog.d/10706.misc
Normal file
@ -0,0 +1 @@
|
|||||||
|
Remove unused `compare_digest` function.
|
@ -12,7 +12,6 @@
|
|||||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
# See the License for the specific language governing permissions and
|
# See the License for the specific language governing permissions and
|
||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
import hmac
|
|
||||||
import logging
|
import logging
|
||||||
import random
|
import random
|
||||||
from typing import List, Union
|
from typing import List, Union
|
||||||
@ -60,18 +59,6 @@ from synapse.util.threepids import (
|
|||||||
|
|
||||||
from ._base import client_patterns, interactive_auth_handler
|
from ._base import client_patterns, interactive_auth_handler
|
||||||
|
|
||||||
# We ought to be using hmac.compare_digest() but on older pythons it doesn't
|
|
||||||
# exist. It's a _really minor_ security flaw to use plain string comparison
|
|
||||||
# because the timing attack is so obscured by all the other code here it's
|
|
||||||
# unlikely to make much difference
|
|
||||||
if hasattr(hmac, "compare_digest"):
|
|
||||||
compare_digest = hmac.compare_digest
|
|
||||||
else:
|
|
||||||
|
|
||||||
def compare_digest(a, b):
|
|
||||||
return a == b
|
|
||||||
|
|
||||||
|
|
||||||
logger = logging.getLogger(__name__)
|
logger = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user