ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

#3066 closed defect (fixed)

管理者IDに日本語や記号が使用できる

Reported by: kiwa Owned by: mori
Priority: minor Milestone: OpenPNE2.12.6
Component: 指定しない Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.12 OpenPNE2.13.4 Cc:

Description (last modified by shingo)

■現象

セットアップ時や管理者追加時、管理者IDに半角英数以外の日本語や記号のみを入力しても管理者が作成されてしまう。

■発生バージョン

  • 2.10.10
  • 2.12.4
  • 2.13.3

■再現手順

■原因

セットアップ時や管理者追加時に適切な形式チェックを行っていない

■修正内容

セットアップ時や管理者追加時に適切な形式チェックを行うように修正

■関連情報

#2048テスト中に発見

Change History (18)

comment:1 Changed 11 years ago by kiwa

Priority: minormajor

comment:2 Changed 11 years ago by imamura623

Keywords: 再現待ち removed
Priority: majorminor

再現できるか試してみました。

概要

セットアップ時と管理者画面での管理者アカウント追加時での、以下の場合の挙動を調べた。

  1. 半角英数以外の日本語の管理者ID
  2. 記号のみの管理者ID
  3. 間に連続する半角SPを含む管理者ID
  4. 前後に半角SPを含む新規の管理者名
  5. 前後に半角SPを含む既存の管理者名
  6. 半角SPのみの管理者名
  7. 全角SPのみの管理者名

テストデータ

  1. "あああ"
  2. "\"
  3. "a a"
  4. " a "
  5. " admin "
  6. " "
  7. " "

結果

再現した。 1,2,3,4,5 は管理者IDとして登録できた。

version

  1. 2.10.11
  2. 2.12.5
  3. 2.13.3

手順

管理者画面での管理者アカウント追加の場合

[管理画面] > [管理画面設定] > [アカウント管理] > [アカウントを追加する] で追加し、 同じ画面の生成された表やエラー文言で登録の有無を確認した。

セットアップ時の場合

セットアップ画面からの管理者ID登録を行い、エラー文言の有無、作成したでのログインの可否、[管理画面] > [管理画面設定] で表示される一覧で確認した。

結果詳細

以下の通り。

NG:登録できる OK:登録できない

半角英数以外の日本語 ("あああ")

バージョン登録パターン結果
2.10.11管理画面NG
セットアップNG
2.12.5 管理画面NG
セットアップNG
2.13.3管理画面NG
セットアップNG

記号のみ ("\")

バージョン登録パターン結果
2.10.11管理画面NG
セットアップNG
2.12.5 管理画面NG
セットアップNG
2.13.3管理画面NG
セットアップNG

間に連続する半角SPを含む ("a a")

バージョン登録パターン結果
2.10.11管理画面NG
セットアップNG
2.12.5 管理画面NG
セットアップNG
2.13.3管理画面NG
セットアップNG

前後に半角SPを含む新規の管理者名 (" a ")

バージョン登録パターン結果
2.10.11管理画面NG
セットアップNG
2.12.5 管理画面NG
セットアップNG
2.13.3管理画面NG
セットアップNG

前後に半角SPを含む既存の管理者名 (" admin ")

バージョン登録パターン結果
2.10.11管理画面NG
セットアップNG
2.12.5 管理画面NG
セットアップNG
2.13.3管理画面NG
セットアップNG

半角SPのみ (" ")

バージョン登録パターン結果
2.10.11管理画面OK
2.10.11セットアップOK
2.12.5 管理画面OK
セットアップOK
2.13.3管理画面OK
セットアップOK

全角SPのみ (" ")

バージョン登録パターン結果
2.10.11管理画面OK
2.10.11セットアップOK
2.12.5 管理画面OK
セットアップOK
2.13.3管理画面OK
セットアップOK

前後に半角SPを含む既存の管理者名で登録した場合の注意点

前後に半角SPを含む既存の管理者名で登録した場合、以下の動作となった。

  1. 管理画面から"admin"で登録できる
  2. 登録時のパスワードが同じ場合は " admin " と "admin"は同一視される
  3. 登録時のパスワードが違う場合は " admin " と "admin"はパスワードで区別される(表示はどちらも"admin")

comment:3 Changed 11 years ago by mori

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 確認中 added
Milestone: OpenPNE2.13.4

comment:4 Changed 11 years ago by mori

Milestone: OpenPNE2.13.4OpenPNE2.12.6

comment:5 Changed 11 years ago by mori

Owner: changed from nobody to mori
Status: newassigned

comment:6 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

修正したリビジョンを示してください。

comment:7 Changed 11 years ago by mori

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンを再修正しました。

comment:8 Changed 11 years ago by shingo

Description: modified (diff)

Descriptionに追記

comment:9 Changed 11 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • webapp/modules/admin/validate/do/insert_c_admin_user.iniについて
    • 正規表現はpneid登録時の表現に倣ってください
    • オプション「i」をつけるなら[a-z0-9]でいい
    • 文字数の制限があった方がいい
  • webapp/modules/setup/validate/do/setup.ini
    • captionがテンプレートの項目名と一致していない。

comment:10 Changed 11 years ago by shingo

Owner: changed from mori to shingo
Status: assignednew

comment:11 Changed 11 years ago by shingo

以下のリビジョンで comment:9 にあげられた事項について修正しました。 ご確認ください。

comment:12 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
Owner: changed from shingo to mori

comment:13 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

エラーメッセージが、「アカウント名を正しく入力してください」となってしまうのでやや不親切かと思います。

comment:14 Changed 11 years ago by shingo

Owner: changed from mori to shingo

comment:15 Changed 11 years ago by shingo

以下のリビジョンで comment:13 にあげられた事項について修正しました。 ご確認ください。

comment:16 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
Owner: changed from shingo to mori

comment:17 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:18 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed

動作テスト中にバグを発見しましたが、このチケットの範囲外の内容のため別チケットを作成しました。

  • #3134 SlavePNEモードでセットアップが未完了の状態でもメンバーの登録ができる

これ以外は問題ありません。

Note: See TracTickets for help on using tickets.