[Freeipa-devel] [freeipa PR#716][comment] Fix minor typos

2017-04-16 Thread abbra 
  URL: https://github.com/freeipa/freeipa/pull/716
Title: #716: Fix minor typos

abbra commented:
"""
Thanks for this pull request.

There are no tickets associated with these changes.

The changes themselves are controversial. Do not change `--forwarder-*` to 
`--forward-*` because you are dealing with well-known DNS term here, not a 
simple word.

Please normalize your `From:` line to be from the same email address. We do not 
accept something like `From: user 
`

Updates to translations should be done via 
https://fedora.zanata.org/project/view/freeipa?dswid=2118, see 
https://fedoraproject.org/wiki/L10N/Translate_on_Zanata for details.

Changes like `plugable` -> `pluggable` may be OK in the text when they are part 
of a normal sentence. However, do not change the code itself and references in 
the text to those code names. These constitute part of a released plugin API 
and should not be changed.


"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/716#issuecomment-294407938
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#433][synchronized] csrgen: Allow some certificate fields to be specified by the user

2017-04-16 Thread LiptonB 
   URL: https://github.com/freeipa/freeipa/pull/433
Author: LiptonB
 Title: #433: csrgen: Allow some certificate fields to be specified by the user
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/433/head:pr433
git checkout pr433
From bf80c8e7c5ae1e67a497b06819ef56bfedc4339a Mon Sep 17 00:00:00 2001
From: Ben Lipton 
Date: Thu, 28 Jul 2016 16:21:44 -0400
Subject: [PATCH 1/3] csrgen: Implement fields that prompt user for data

Allows some data to be user-specified rather than coming out of the
database. The provided data can be formatted with jinja2 rules just as
database values can.

https://fedorahosted.org/freeipa/ticket/4899
---
 ipaclient/csrgen.py| 35 --
 ipaclient/csrgen/rules/dataEmailUserSpecified.json | 16 ++
 ipaclient/plugins/csrgen.py| 10 +--
 ipatests/test_ipaclient/test_csrgen.py | 11 +++
 4 files changed, 63 insertions(+), 9 deletions(-)
 create mode 100644 ipaclient/csrgen/rules/dataEmailUserSpecified.json

diff --git a/ipaclient/csrgen.py b/ipaclient/csrgen.py
index 0f52a8b..67442f6 100644
--- a/ipaclient/csrgen.py
+++ b/ipaclient/csrgen.py
@@ -372,8 +372,9 @@ def __init__(self, rule_provider, formatter_class=OpenSSLFormatter):
 self.rule_provider = rule_provider
 self.formatter = formatter_class()
 
-def csr_config(self, principal, config, profile_id):
-render_data = {'subject': principal, 'config': config}
+def csr_config(self, principal, config, userdata, profile_id):
+render_data = {
+'subject': principal, 'config': config, 'userdata': userdata}
 
 rules = self.rule_provider.rules_for_profile(profile_id)
 template = self.formatter.build_template(rules)
@@ -387,6 +388,36 @@ def csr_config(self, principal, config, profile_id):
 
 return config
 
+def get_user_prompts(self, profile_id):
+prompts = {}
+rules = self.rule_provider.rules_for_profile(profile_id)
+
+for field_mapping in rules:
+for rule in field_mapping.data_rules:
+if 'prompt' in rule.options:
+try:
+var = rule.options['data_source']
+except KeyError:
+raise errors.CSRTemplateError(reason=_(
+'Certificate mapping rule %(rule)s has a prompt'
+' but no data_source set') % {'rule': rule.name})
+if var in prompts:
+raise errors.CSRTemplateError(reason=_(
+'More than one data rule in this profile prompts'
+' for the %(item)s data item') % {'item': var})
+var_parts = var.split('.')
+if len(var_parts) != 2 or var_parts[0] != 'userdata':
+raise errors.CSRTemplateError(
+reason=_(
+'Format of variable name in rule %(rule)s is'
+' incorrect. Rules that prompt for data must'
+' use a variable "userdata."') %
+{'rule': rule.name})
+
+prompts[var_parts[1]] = rule.options['prompt']
+
+return prompts
+
 
 class CSRLibraryAdaptor(object):
 def get_subject_public_key_info(self):
diff --git a/ipaclient/csrgen/rules/dataEmailUserSpecified.json b/ipaclient/csrgen/rules/dataEmailUserSpecified.json
new file mode 100644
index 000..3fb2fb1
--- /dev/null
+++ b/ipaclient/csrgen/rules/dataEmailUserSpecified.json
@@ -0,0 +1,16 @@
+{
+  "rules": [
+{
+  "helper": "openssl",
+  "template": "email = {{userdata.email}}"
+},
+{
+  "helper": "certutil",
+  "template": "email:{{userdata.email|quote}}"
+}
+  ],
+  "options": {
+"data_source": "userdata.email",
+"prompt": "Email address"
+  }
+}
diff --git a/ipaclient/plugins/csrgen.py b/ipaclient/plugins/csrgen.py
index 568a79f..6503f57 100644
--- a/ipaclient/plugins/csrgen.py
+++ b/ipaclient/plugins/csrgen.py
@@ -90,6 +90,9 @@ def execute(self, *args, **options):
 if not backend.isconnected():
 backend.connect()
 
+generator = csrgen.CSRGenerator(csrgen.FileRuleProvider())
+prompts = generator.get_user_prompts(profile_id, helper)
+
 try:
 if principal.is_host:
 principal_obj = api.Command.host_show(
@@ -106,9 +109,12 @@ def execute(self, *args, **options):
 principal_obj = principal_obj['result']
 config = api.Command.config_show()['result']
 
-generator = csrgen.CSRGenerator(csrgen.FileRuleProvider())
+userdata = {}
+for name, prompt in prompts.items():
+userdata[name] = self.Backend.textui.prompt(prompt)
 
-

[Freeipa-devel] [freeipa PR#717][opened] csrgen: Finish NSS support

2017-04-16 Thread LiptonB 
   URL: https://github.com/freeipa/freeipa/pull/717
Author: LiptonB
 Title: #717: csrgen: Finish NSS support
Action: opened

PR body:
"""
I took the approach of generating a new key for each request, as keys already 
stored in a database are difficult to name precisely. I also had to add another 
hook to `CSRLibraryAdaptor` that is called after the cert is returned from the 
server, so that we could add the cert to the database as desired.
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/717/head:pr717
git checkout pr717
From 171ead1704ef9f5d28b361cb6555dc31540fa4b5 Mon Sep 17 00:00:00 2001
From: Ben Lipton 
Date: Thu, 30 Mar 2017 22:39:40 -0400
Subject: [PATCH] csrgen: Finish NSS support

https://pagure.io/freeipa/issue/4899
---
 ipaclient/csrgen.py   | 130 +-
 ipaclient/plugins/cert.py |  21 +++-
 2 files changed, 102 insertions(+), 49 deletions(-)

diff --git a/ipaclient/csrgen.py b/ipaclient/csrgen.py
index 0f52a8b..7724099 100644
--- a/ipaclient/csrgen.py
+++ b/ipaclient/csrgen.py
@@ -10,16 +10,17 @@
 import os.path
 import pipes
 import subprocess
+import tempfile
 import traceback
 
 import pkg_resources
 
 from cryptography.hazmat.backends import default_backend
-from cryptography.hazmat.primitives.asymmetric import padding
+from cryptography.hazmat.primitives.asymmetric import padding, rsa
 from cryptography.hazmat.primitives import hashes
 from cryptography.hazmat.primitives.serialization import (
-load_pem_private_key, Encoding, PublicFormat)
-from cryptography.x509 import load_pem_x509_certificate
+load_pem_private_key, Encoding, NoEncryption, PrivateFormat, PublicFormat)
+from cryptography.x509 import load_der_x509_certificate
 import jinja2
 import jinja2.ext
 import jinja2.sandbox
@@ -389,39 +390,28 @@ def csr_config(self, principal, config, profile_id):
 
 
 class CSRLibraryAdaptor(object):
-def get_subject_public_key_info(self):
-raise NotImplementedError('Use a subclass of CSRLibraryAdaptor')
-
-def sign_csr(self, certification_request_info):
-"""Sign a CertificationRequestInfo.
+def key(self):
+"""Return the private key to be used in the cert.
 
-Returns: str, a DER-encoded signed CSR.
+Returns: cryptography.hazmat.primitives.asymmetric.rsa.RSAPrivateKey
+representing the private key.
 """
 raise NotImplementedError('Use a subclass of CSRLibraryAdaptor')
 
-
-class OpenSSLAdaptor(object):
-def __init__(self, key_filename, password_filename):
-self.key_filename = key_filename
-self.password_filename = password_filename
-
-def key(self):
-with open(self.key_filename, 'r') as key_file:
-key_bytes = key_file.read()
-password = None
-if self.password_filename is not None:
-with open(self.password_filename, 'r') as password_file:
-password = password_file.read().strip()
-
-key = load_pem_private_key(key_bytes, password, default_backend())
-return key
-
 def get_subject_public_key_info(self):
+"""Return the public key info for the cert.
+
+Returns: str, a DER-encoded SubjectPublicKeyInfo structure.
+"""
 pubkey_info = self.key().public_key().public_bytes(
 Encoding.DER, PublicFormat.SubjectPublicKeyInfo)
 return pubkey_info
 
 def sign_csr(self, certification_request_info):
+"""Sign a CertificationRequestInfo.
+
+Returns: str, a DER-encoded signed CSR.
+"""
 reqinfo = decoder.decode(
 certification_request_info, rfc2314.CertificationRequestInfo())[0]
 csr = rfc2314.CertificationRequest()
@@ -442,32 +432,78 @@ def sign_csr(self, certification_request_info):
 csr.setComponentByName('signature', asn1sig)
 return encoder.encode(csr)
 
+def process_cert(self, cert):
+"""Perform any required post-processing on the certificate."""
 
-class NSSAdaptor(object):
-def __init__(self, database, password_filename):
-self.database = database
+
+class OpenSSLAdaptor(CSRLibraryAdaptor):
+def __init__(self, key_filename, password_filename):
+self.key_filename = key_filename
 self.password_filename = password_filename
-self.nickname = base64.b32encode(os.urandom(40))
+self._key = None
 
-def get_subject_public_key_info(self):
-temp_cn = base64.b32encode(os.urandom(40))
+def key(self):
+if self._key is None:
+with open(self.key_filename, 'r') as key_file:
+key_bytes = key_file.read()
+password = None
+if self.password_filename is not None:
+with open(self.password_filename, 'r') as password_file:
+password = password_file.read().strip()
 
-password_args = []
-if

[Freeipa-devel] [freeipa PR#716][comment] Fix minor typos

2017-04-16 Thread realsobek 
  URL: https://github.com/freeipa/freeipa/pull/716
Title: #716: Fix minor typos

realsobek commented:
"""
There are 353 occurrences of 'plugable'. 2 in file names. 351 in code. 
I can make the change in a separate branch. But I cannot evaluate the outcome. 
Hence I would like to have your opinion before jumping to conclusions. Can I go 
ahead and make the change?
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/716#issuecomment-294373325
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/23] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/23] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/23] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/23] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#715][synchronized] use correct option name

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/715
Author: realsobek
 Title: #715: use correct option name
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/715/head:pr715
git checkout pr715
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 1/3] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From cb4250afda73a1ad3b3fb1d8a94f04b6dd944c72 Mon Sep 17 00:00:00 2001
From: user 
Date: Sat, 15 Apr 2017 23:03:19 +0200
Subject: [PATCH 2/3] use correct option name

---
 po/uk.po | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/po/uk.po b/po/uk.po
index 720ca2e..4669fcb 100644
--- a/po/uk.po
+++ b/po/uk.po
@@ -11044,7 +11044,7 @@ msgstr ""
 
 msgid ""
 "\n"
-" Semantics of the --forwarder-policy option:\n"
+" Semantics of the --forward-policy option:\n"
 "   * none - disable forwarding for the given zone.\n"
 "   * first - forward all queries to configured forwarders. If they fail,\n"
 "   do resolution using DNS root servers.\n"
@@ -11052,7 +11052,7 @@ msgid ""
 "   return failure.\n"
 msgstr ""
 "\n"
-" Семантика параметра --forwarder-policy:\n"
+" Семантика параметра --forward-policy:\n"
 "   * none — вимкнути переспрямовування для вказаної зони.\n"
 "   * first — спрямувати усі запити до налаштованих переспрямовувачів. Якщо\n"
 "   це не спрацює, виконати перетворення за допомогою кореневих серверів "
@@ -23359,7 +23359,7 @@ msgid ""
 "   queries, which cannot be answered from its local cache, to configured\n"
 "   forwarders.\n"
 "\n"
-" Semantics of the --forwarder-policy option:\n"
+" Semantics of the --forward-policy option:\n"
 "   * none - disable forwarding for the given zone.\n"
 "   * first - forward all queries to configured forwarders. If they fail,\n"
 "   do resolution using DNS root servers.\n"
@@ -23617,7 +23617,7 @@ msgstr ""
 "кешу, до\n"
 "   налаштованих переспрямовувачів.\n"
 "\n"
-" Семантика параметра --forwarder-policy:\n"
+" Семантика параметра --forward-policy:\n"
 "   * none — вимкнути переспрямовування для вказаної зони.\n"
 "   * first — спрямувати усі запити до налаштованих переспрямовувачів. Якщо\n"
 "   це не спрацює, виконати перетворення за допомогою кореневих серверів "

From 2c3db7b6645868d8d41b8bf6039228214f4f6660 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sun, 16 Apr 2017 18:21:10 +0200
Subject: [PATCH 3/3] add empty lines to be consistent

---
 po/uk.po | 4 
 1 file changed, 4 insertions(+)

diff --git a/po/uk.po b/po/uk.po
index 4669fcb..a53068e 100644
--- a/po/uk.po
+++ b/po/uk.po
@@ -7998,7 +7998,9 @@ msgstr "Словник перекладених повідомлень"
 msgid ""
 "\n"
 "ID Views\n"
+"\n"
 "Manage ID Views\n"
+"\n"
 "IPA allows to override certain properties of users and groups per each "
 "host.\n"
 "This functionality is primarily used to allow migration from older systems "
@@ -8007,7 +8009,9 @@ msgid ""
 msgstr ""
 "\n"
 "Перегляди ідентифікаторів\n"
+"\n"
 "Керування переглядами ідентифікаторів\n"
+"\n"
 "IPA надає вам змогу перевизначити певні властивості записів користувачів та "
 "груп для окремих вузлів.\n"
 "В основному, ці функціональні можливості призначено для уможливлення "
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/20] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/20] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/20] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/20] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/19] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/19] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/19] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/19] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/18] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/18] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/18] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/18] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/17] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/17] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/17] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/17] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/16] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/16] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/16] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/16] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/15] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/15] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/15] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/15] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to

[Freeipa-devel] [freeipa PR#716][synchronized] Fix minor typos

2017-04-16 Thread realsobek 
   URL: https://github.com/freeipa/freeipa/pull/716
Author: realsobek
 Title: #716: Fix minor typos
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/716/head:pr716
git checkout pr716
From 703691c605b39e08ce3aff4623c90edafa0bca53 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 13:52:44 +0200
Subject: [PATCH 01/14] fix minor typo in ipa-adtrust-install.1

---
 install/tools/man/ipa-adtrust-install.1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/man/ipa-adtrust-install.1 b/install/tools/man/ipa-adtrust-install.1
index ef3c23b..464bbfa 100644
--- a/install/tools/man/ipa-adtrust-install.1
+++ b/install/tools/man/ipa-adtrust-install.1
@@ -26,7 +26,7 @@ Adds all necessary objects and configuration to allow an IPA server to create a
 trust to an Active Directory domain. This requires that the IPA server is
 already installed and configured.
 
-Please note you will not be able to estabilish an trust to an Active Directory
+Please note you will not be able to establish an trust to an Active Directory
 domain unless the realm name of the IPA server matches its domain name.
 
 ipa\-adtrust\-install can be run multiple times to reinstall deleted objects or

From 9226f80f8c397eecc76654f6f74faf53bc0076a2 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:09:32 +0200
Subject: [PATCH 02/14] fix output of `ipa-managed-entries --help`

New string for '-l, --list' option taken from `man 1 ipa-managed-entries`.

Branches 4-4 and 4-5 are affected too. Shall I create separate pull requests for them?
---
 install/tools/ipa-managed-entries | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries
index 731dcc3..e1ca9da 100755
--- a/install/tools/ipa-managed-entries
+++ b/install/tools/ipa-managed-entries
@@ -43,7 +43,7 @@ def parse_options():
   help="DN for the Managed Entry Definition")
 parser.add_option("-l", "--list", dest="list_managed_entries",
   action="store_true",
-  help="DN for the Managed Entry Definition")
+  help="List available Managed Entries")
 parser.add_option("-p", "--password", dest="dirman_password",
   help="Directory Manager password")
 

From 926206023dde0cb7b49fb8060f651682bec26731 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:14:19 +0200
Subject: [PATCH 03/14] fix minor typo in install/html/browserconfig.html

---
 install/html/browserconfig.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index 9c5cf68..d3c8d1c 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -91,7 +91,7 @@
 Browser successfully configured
 Configuration aborted
 Configuration was not successful, extension isn't installed or is disabled. Please proceed to step 2.
-Configuration was not successful, unknown error uccured.
+Configuration was not successful, unknown error occurred.
 
 
 

From c944abf98952c9a44f598131f4e3e91c2d859d76 Mon Sep 17 00:00:00 2001
From: realsobek 
Date: Sat, 15 Apr 2017 15:19:39 +0200
Subject: [PATCH 04/14] rewording of firefox browser configuration

on CentOS 7 with firefox-52.0-4 authentication with 'example.com.' did not work; with 'example.com' it did
---
 install/html/browserconfig.html | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/install/html/browserconfig.html b/install/html/browserconfig.html
index d3c8d1c..f1ab764 100644
--- a/install/html/browserconfig.html
+++ b/install/html/browserconfig.html
@@ -98,16 +98,16 @@
 Step 2
 
 
-In the address bar of Firefox, type about:config to display the list of current configuration options.
+In the address bar of Firefox, type about:config and press enter to display the list of current configuration options.
 
 
-In the Filter field, type negotiate to restrict the list of options.
+In the Filter field, type network.negotiate-auth.trusted-uris to restrict the list of options.
 
 
-Double-click the network.negotiate-auth.trusted-uris entry to display the Enter string value dialog box.
+Double-click the entry to display the "Enter string value" dialog box.
 
 
-Enter the name of the domain against which you want to