Improve comments and documentation

Author: thelinuxkid

README.rst

@@ -75,7 +75,7 @@ not be used for anything other than exploring and testing.
 Creating and sending the AuthnRequest to the identity provider
 --------------------------------------------------------------
 The identification flow starts when a user requests a resource from the service
-provider which, in this case, is implemented in our example app. In our example,
+provider which, in this case, is implemented in our example app. In our example
 the user requests a resource by going to the root path of our app, i.e,::
 
     http://localhost:7070/
@@ -117,7 +117,7 @@ Receiving and verifying the response
 ------------------------------------
 The user will then be redirected to the OneLogin login page where they will
 enter their credentials in order to verify their identity. After OneLogin has
-verified their identity, it will redirect the user to the
+verified their identity it will redirect the user to the
 assertion_consumer_service_url--http://localhost:7070/example/saml/consume.
 
 Our example app then verifies the SAML Response from OneLogin using the fingerprint

example.cfg

@@ -12,11 +12,11 @@ assertion_consumer_service_url =
 idp_sso_target_url =
 # The x.509 certificate fingerprint from OneLogin, found under Security->SAML,
 # as a file path. Only one of idp_cert_file and idp_cert_fingerprint will be
-# used by the example app, with idp_cert_file being having priority if both are
+# used by the example app, with idp_cert_file having priority if both are
 # defined.
 idp_cert_file =
 # The x.509 certificate fingerprint from OneLogin, found under Security->SAML,
 # as a string. Only one of idp_cert_file and idp_cert_fingerprint will be
-# used by the example app, with idp_cert_file being having priority if both are
+# used by the example app, with idp_cert_file having priority if both are
 # defined.
 idp_cert_fingerprint =

onelogin/saml/Response.py

@@ -81,7 +81,7 @@ def _get_name_id(self):
 
     name_id = property(
         fget=_get_name_id,
-        doc="The identity provider to which the user belongs",
+        doc="The value requested in the name_identifier_format, e.g., the user's email address",
         )
 
     def is_valid(