Nethserver-sssd-remove-provider : accentuated characters not correctly handled

pagaille · November 13, 2017, 12:17pm

NS 7.4 final
Hi,

nethserver-sssd-remove-provider doesn’t handle the accentuated characters correctly when creating users.tsv and groups.tsv.

The file’s charset is unknown,

file -i users.tsv 
users.tsv: text/plain; charset=unknown-8bit

Actually the contents looks like a mixup of Latin1 and UTF-8. Please try with names containing apostrophes ( ’ ) and accents ( é è ç, …)

The apostrophe looks coded as UTF-8, while the accentuated chars are in latin1. See screenshots.

Txs

giacomo · November 13, 2017, 2:02pm

This is the script which does the job:

github.com

NethServer/nethserver-sssd/blob/master/root/etc/e-smith/events/actions/nethserver-sssd-export-accounts

#!/usr/bin/perl

#
# Copyright (C) 2017 Nethesis S.r.l.
# http://www.nethesis.it - nethserver@nethesis.it
#
# This script is part of NethServer.
#
# NethServer is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License,
# or any later version.
#
# NethServer is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with NethServer.  If not, see COPYING.

This file has been truncated. show original

Anyone want to try to hack it?

pagaille · November 13, 2017, 8:00pm

Me

Clap your hands for pagaille’s first splash into the perl world

giacomo · November 14, 2017, 8:21am

Excellent work!!

Thank you pagaille!

davidep · December 5, 2017, 10:47am

In nethserver-testing:

yum install http://packages.nethserver.org/nethserver/7.4.1708/testing/x86_64/Packages/nethserver-sssd-1.3.3-1.3.gfbbf95a.ns7.noarch.rpm

pagaille · December 5, 2017, 1:59pm

I tested the patch (manually applied) on two different systems recently and didn’t noticed any problem.

alefattorini · January 3, 2018, 2:47pm

Can I close that? Ehi @pagaille sorry for coming late to the party. Great shoot!

pagaille · January 3, 2018, 3:22pm

Yes you can ! It is solved and merged. Txs

alefattorini · January 3, 2018, 5:55pm