https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2016-06-12T02:01:49ZCeph Ceph - Bug #2904: ceph-authtool: Adds keys on typos, expected error messagehttps://tracker.ceph.com/issues/2904?journal_id=724252016-06-12T02:01:49ZBrad Hubbardbhubbard@redhat.com
<ul><li><strong>Assignee</strong> set to <i>Brad Hubbard</i></li></ul> Ceph - Bug #2904: ceph-authtool: Adds keys on typos, expected error messagehttps://tracker.ceph.com/issues/2904?journal_id=724292016-06-12T09:47:21ZBrad Hubbardbhubbard@redhat.com
<ul></ul><p>This case has been resolved by a previous commit.</p>
<p>$ ./ceph-authtool /tmp/keyring --name=ods.0 --cap mon "allow *" --cap osd "allow *" <br />error parsing 'ods.0': expected string of the form TYPE.ID, valid types are: auth, mon, osd, mds, client</p>
<p>I have a candidate patch which produces the following for commands which previously misbehaved.</p>
<p>$ ./ceph-authtool /tmp/keyring --create-keyring --name=mon. --add-key= --cap mon 'allow *'<br />Option --add-key= requires an argument.</p>
<p>$ ./ceph-authtool /tmp/keyring --cap mon "allow *" --cap osd "allow *" <br />Can't find existing key for client.admin and neither gen-key nor add-key specified</p>
<p>I think I have covered all options which can create an invalid key like "key = AAAAAAAAAAAAAAAA"</p>
<p>My patch needs a good deal of testing and then I'll submit a PR.</p> Ceph - Bug #2904: ceph-authtool: Adds keys on typos, expected error messagehttps://tracker.ceph.com/issues/2904?journal_id=724302016-06-12T09:48:14ZBrad Hubbardbhubbard@redhat.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>12</i></li></ul> Ceph - Bug #2904: ceph-authtool: Adds keys on typos, expected error messagehttps://tracker.ceph.com/issues/2904?journal_id=740052016-07-07T06:14:18ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Status</strong> changed from <i>12</i> to <i>Resolved</i></li></ul><p><a class="external" href="https://github.com/ceph/ceph/pull/9704">https://github.com/ceph/ceph/pull/9704</a></p>