[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-07 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints tomaskrizek commented: """ master: * a06c71b1268850e485e89049ed3654f893edff0b Add SHA256 fingerprints for certs """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-284819750

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-07 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ Hm, apparently I had old `po/`, never mind, then. """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-284656476 -- Manage your subscription for the Freei

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-06 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints tomaskrizek commented: """ I think this is a translation issue that will resolve itself once we generate new translation files. Is that correct, @MartinBasti ? When using `make install` that regenerates `*.po

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-06 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ Please transform `sha256_fingerprint:` into `Fingerprint (SHA-256):` ``` $ ipa cert-show --all Serial number: 1 Issuing CA: ipa Certificate: Subject: CN=Certificate Authority,O=DOM-2

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-06 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ Please transform `sha256_fingerprint:` into `Fingerprint (SHA1):` ``` $ ipa cert-show --all Serial number: 1 Issuing CA: ipa Certificate: Subject: CN=Certificate Authority,O=DOM-245.

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-06 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ Please transform `sha256_fingerprint:` into `Fingerprint (SHA1):` ``` $ ipa cert-show --all Serial number: 1 Issuing CA: ipa Certificate: Subject: CN=Certificate Authority,O=DOM-245.

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-02 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints tomaskrizek commented: """ @pvomacka Thanks! Should be fixed now. """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-283673360 -- Manage your subscription for the Freeipa-de

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-02 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints pvomacka commented: """ @tomaskrizek so, inline comment is not possible to the line where file was not changed. So, please remove line 1979: delete command.options.all; . That should be enough to display fing

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-02 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints pvomacka commented: """ @tomaskrizek actually you did almost all necessary steps. Just please check inline comments where is described one another change. And in general you do not have to add anything into j

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-02 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints pvomacka commented: """ @stlaz , @tomaskrizek I will fix that today. """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-283662059 -- Manage your subscription for the Freeip

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-03-02 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ I wanted to test this but nothing is currently shown for either SHA-1 or SHA256 fingerprints in the WebUI so you can either fix it or we'll wait till @pvomacka has that done. """ See the

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-02-24 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints tomaskrizek commented: """ I've dropped the commit that modified the deprecated file. """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-282247242 -- Manage your subscriptio

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-02-24 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints MartinBasti commented: """ Do not touch `install/share/copy-schema-to-ca.py` ever (this will be removed soon from master, just waiting for ACKs) """ See the full comment at https://github.com/freeipa/freeipa

[Freeipa-devel] [freeipa PR#504][comment] Add SHA256 fingerprints

2017-02-24 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/504 Title: #504: Add SHA256 fingerprints stlaz commented: """ As discussed about hundred times before, do not touch `install/share/copy-schema-to-ca.py`. """ See the full comment at https://github.com/freeipa/freeipa/pull/504#issuecomment-282244201