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

[master, ipa-4-3] Raise error when running ipa-adtrust-install with empty netbios--name #24

Closed
wants to merge 1 commit into from

Conversation

mirielka
Copy link
Contributor

@mirielka mirielka commented Aug 26, 2016

When running ipa-adtrust-install, a netbios-name option must be specified.
Currently if an invalid netbios name in form of empty string is specified, the
installation proceeds, but changes the invalid value to a netbios name
determined from domain name without any notification.

Fixing this so that any attempt to supply empty string as netbios name fails
with error in case of unattended installation, or to request input of valid
netbios name from command line during normal installation.

https://fedorahosted.org/freeipa/ticket/6120

Note - newly intended behaviour of ipa-adtrust-install:

  • unattended installation with --netbios-name="" fails
  • unattended installation without --netbios-name specification proceeds and uses default value derived from domain name
  • normal installation in both cases requests inserting of valid netbios name

When running ipa-adtrust-install, a netbios-name option must be specified.
Currently if an invalid netbios name in form of empty string is specified, the
installation proceeds, but changes the invalid value to a netbios name
determined from domain name without any notification.

Fixing this so that any attempt to supply empty string as netbios name fails
with error in case of unattended installation, or to request input of valid
netbios name from command line during normal installation.

https://fedorahosted.org/freeipa/ticket/6120
@tkrizek
Copy link
Contributor

tkrizek commented Aug 30, 2016

Works as described.

@tkrizek tkrizek added the ack Pull Request approved, can be merged label Aug 30, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Aug 30, 2016
@mirielka mirielka deleted the master_adtrust branch August 30, 2016 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants