queens regresion: _dn_to_id() not using utf8_encode/decode
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Identity (keystone) |
Triaged
|
Low
|
Corey Bryant | ||
Ubuntu Cloud Archive |
Invalid
|
Undecided
|
Unassigned | ||
Queens |
Fix Released
|
High
|
Corey Bryant | ||
keystone (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Bionic |
Fix Released
|
High
|
Corey Bryant |
Bug Description
[Impact]
There's a regression in the LDAP common backend code due to a recent stable/queens backport that shouldn't have been backported past stable/rocky. It was backported as part of the fixes for https:/
The following patch shouldn't have been backported to stable/queens:
https:/
The reason why is because the following patch, which switched to bytes_mode=False, doesn't exist in stable/queens:
https:/
In particular see the changes to _dn_to_id() in https:/
Those changes didn't happen in stable/queens so _dn_to_id should still be UTF-8 encoding/decoding the appropriate fields. In other words it should still be using the following in stable/queens:
if self.id_attr == utf8_decode(
return utf8_decode(
[Test Case]
See test case in https:/
[Regression Potential]
The code that will be fixed for this bug (ie. the code in the if statement) is being reverted to what it used to be prior to the bug fix for https:/
Changed in keystone (Ubuntu): | |
status: | New → Invalid |
Changed in keystone (Ubuntu Bionic): | |
status: | New → Triaged |
importance: | Undecided → High |
Changed in cloud-archive: | |
status: | New → Invalid |
Changed in keystone (Ubuntu Bionic): | |
assignee: | nobody → Corey Bryant (corey.bryant) |
description: | updated |
summary: |
- stable/queens regresion - _dn_to_id() should still be using - utf8_encode/utf8_decode in queens + queens regresion - _dn_to_id() not using utf8_encode/utf8_decode |
summary: |
- queens regresion - _dn_to_id() not using utf8_encode/utf8_decode + queens regresion: _dn_to_id() not using utf8_encode/utf8_decode |
summary: |
- queens regresion: _dn_to_id() not using utf8_encode/utf8_decode + queens regresion: _dn_to_id() not using utf8_encode/decode |
Changed in keystone: | |
status: | New → Triaged |
importance: | Undecided → Low |
milestone: | none → ussuri-1 |
assignee: | nobody → Corey Bryant (corey.bryant) |
tags: | added: bionic regression-proposed |
Fix proposed to branch: stable/queens /review. opendev. org/692128
Review: https:/