Bug 901193 – add-user check is not i18n, while prompt is

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 901193 - (JBPAPP6-1639) add-user check is not i18n, while prompt is

Summary:	add-user check is not i18n, while prompt is

Status:	VERIFIED

Aliases:	JBPAPP6-1639

Product:	JBoss Enterprise Application Platform 6
Classification:	JBoss
Component:	Scripts and Commands, Security (Show other bugs)
Sub Component:
Version:	6.0.0
Hardware:	Unspecified Unspecified

Priority	urgent Severity urgent
Target Milestone:	DR3
Target Release:	EAP 6.4.0
Assigned To:	Darran Lofthouse
QA Contact:	Petr Kremensky
Docs Contact:

URL:	http://jira.jboss.org/jira/browse/JBP...
Whiteboard:	CLI
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2012-11-19 08:47 EST by Carlo de Wolf
Modified:	2018-10-19 17:36 EDT (History)
CC List:	5 users (show)

See Also:	1148775
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:	The add-user utility in JBoss EAP 6 prompts users to confirm the action they are about to take based on the current locale. Where user input was being compared with the different translations, the user input was being converted to lower case whilst the translated value was not. This meant the add-user utility was not able to identify which option the user had selected due to the difference in case for comparison and was reporting an error to the user to tell them they had not selected a valid value. In this release both the user's input and the translated values being compared are converted to lower case before comparison. Internationalized responses are now correctly accepted for the confirmation dialogues within the add-user utility.
Story Points:	---
Clone Of:
Environment:	Fedora 16
Last Closed:
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
JBoss Issue Tracker	JBPAPP6-1639	Major	Closed	add-user check is not i18n, while prompt is	2014-09-09 06:53:56 EDT
JBoss Issue Tracker	WFCORE-124	Major	Coding In Progress	The yes/no checking in add-user converts the users input to lower case but not the translated values.	2014-09-24 12:26:53 EDT

Groups: None (edit)

Description Carlo de Wolf 2012-11-19 08:47:10 EST

Workaround: Workaround Exists
Workaround Description: Use English

{noformat}
LANG=en_US.UTF-8 ./add-user.sh
{noformat}
project_key: JBPAPP6

{noformat}
Informationen des neu hinzuzufügenden Nutzers eingeben.
Bereich (ManagementRealm) : 
Benutzername (admin) : 
Passwort : 
Passwort erneut eingeben : 
Der Benutzername 'admin' ist leicht zu erraten
Sind Sie sicher, dass Sie Benutzer 'admin' hinzufügen möchten ja/nein? j

 * Fehler * 
JBAS015240: Invalid response. (Valid responses are yes, y, no and n)

{noformat}

Comment 2 Darran Lofthouse 2013-06-27 08:23:14 EDT

I will try and re-visit this one - I did think we had this covered but it is possible that if we do not have a full set of translations some of it is dropping back to English.

Comment 7 Petr Kremensky 2014-07-09 08:10:03 EDT

Still valid issue (EAP 6.3.0.ER9).

Der Benutzername 'admin' ist leicht zu erraten
Sind Sie sicher, dass Sie Benutzer 'admin' hinzufügen möchten ja/nein? ja

 * Fehler * 
JBAS015240: Ungültige Antwort. (Gültige Antworten sind  Ja, j, Nein and n)

Der Benutzername 'admin' ist leicht zu erraten
Sind Sie sicher, dass Sie Benutzer 'admin' hinzufügen möchten ja/nein? nein

 * Fehler * 
JBAS015240: Ungültige Antwort. (Gültige Antworten sind  Ja, j, Nein and n)

Comment 8 Darran Lofthouse 2014-09-24 12:20:08 EDT

Ok after debugging further this is an issue with the case conversion, the users input is converted to lower case but the translation values are not - ideally the translated values would be lower case but that makes this easily breakable so I will add a lower case conversion of the translated values before the check.

Comment 12 Kabir Khan 2014-09-26 08:05:48 EDT

https://github.com/jbossas/jboss-eap/pull/1713

Comment 13 Petr Kremensky 2014-10-02 05:40:58 EDT

Verified on EAP 6.4.0.DR3.

I found some minor issues in translations, I'll file a separate bz for this and link it here.

Note You need to log in before you can comment on or make changes to this bug.