[sssd PR#5264][opened] Utils: White space replace with another character

[elkoniu]

URL: https://github.com/SSSD/sssd/pull/5264
Author: elkoniu
 Title: #5264: Utils: White space replace with another character
Action: opened

PR body:
"""
Function responsible for replacing white space
has wrong log levels set. It can create false positive
critical error messages broadcasted across the system
without any real reason.

This patch adjusts those log levels to be on the debug
level rather than criticar error level.

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1818671
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5264/head:pr5264
git checkout pr5264

[sssd PR#5264][+Bugzilla] Utils: White space replace with another character

[elkoniu]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

Label: +Bugzilla

[sssd PR#5264][+Waiting for review] Utils: White space replace with another character

[elkoniu]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

Label: +Waiting for review

[sssd PR#5264][comment] Utils: White space replace with another character

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

pbrezina commented:
"""
I think this is not a correct solution since having the override character in name is in fact a real problem in the environment. It will either make the user unresolvable or even worse it will return different user if you have both names "user_1" and "user 1" in LDAP.

I think the root cause is that this option is set globally for the whole SSSD and ssh_keys is a local user. If it is a local user then we should probably figure out if we can somehow avoid using this override for files provider. If it is an LDAP user then they need to fix it on their side.
"""

See the full comment at https://github.com/SSSD/sssd/pull/5264#issuecomment-678200050

[sssd PR#5264][comment] Utils: White space replace with another character

[alexey-tikhonov]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

alexey-tikhonov commented:
"""
Setting flags based on https://github.com/SSSD/sssd/pull/5264#issuecomment-678200050
"""

See the full comment at https://github.com/SSSD/sssd/pull/5264#issuecomment-697408609

[sssd PR#5264][-Waiting for review] Utils: White space replace with another character

[alexey-tikhonov]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

Label: -Waiting for review

[sssd PR#5264][+Changes requested] Utils: White space replace with another character

[alexey-tikhonov]

URL: https://github.com/SSSD/sssd/pull/5264
Title: #5264: Utils: White space replace with another character

Label: +Changes requested

[sssd PR#5264][closed] Utils: White space replace with another character

[elkoniu]

URL: https://github.com/SSSD/sssd/pull/5264
Author: elkoniu
 Title: #5264: Utils: White space replace with another character
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5264/head:pr5264
git checkout pr5264