Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

csrgen: Allow some certificate fields to be specified by the user #433

Closed
wants to merge 3 commits into from

Conversation

LiptonB
Copy link
Contributor

@LiptonB LiptonB commented Feb 4, 2017

These patches allow CSR generation rules to contain a "prompt," which will cause data to be requested from the user and interpolated into the CSR.

The second commit runs the prompt through gettext. As I asked about here, I'm not sure if this is useful because the prompt strings in the rule files won't be recognized as translatable. But I decided to include the commit for discussion.

@MartinBasti
Copy link
Contributor

************* Module ipaclient.csrgen
ipaclient/csrgen.py:376: [E1101(no-member), CSRGenerator.get_user_prompts] Module 'ipalib.errors' has no 'CertificateMappingError' member)
ipaclient/csrgen.py:380: [E1101(no-member), CSRGenerator.get_user_prompts] Module 'ipalib.errors' has no 'CertificateMappingError' member)
ipaclient/csrgen.py:385: [E1101(no-member), CSRGenerator.get_user_prompts] Module 'ipalib.errors' has no 'CertificateMappingError' member)
ipaclient/csrgen.py:367: [W0612(unused-variable), CSRGenerator.get_user_prompts] Unused variable 'syntax_rules')
************* Module ipatests.test_ipaclient.test_csrgen
ipatests/test_ipaclient/test_csrgen.py:322: [W0612(unused-variable), test_rule_handling.test_userdata_included] Unused variable 'script')
ipatests/test_ipaclient/test_csrgen.py:324: [W0612(unused-variable), test_rule_handling.test_userdata_included] Unused variable 'expected_script'm)

@LiptonB
Copy link
Contributor Author

LiptonB commented Feb 9, 2017

Sorry for submitting this with lint errors - fixed now.

@tiran
Copy link
Member

tiran commented Mar 14, 2017

@LiptonB please resolve conflicts

@LiptonB
Copy link
Contributor Author

LiptonB commented Mar 15, 2017

Rebased, thanks.

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
Currently doesn't change anything because the strings are not
translated.  Need to find a way to include them in the translation
files.

https://fedorahosted.org/freeipa/ticket/4899
@rcritten
Copy link
Contributor

Closing due to inactivity.

@rcritten rcritten closed this May 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants