[
https://issues.apache.org/jira/browse/TC-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16138647#comment-16138647
]
ASF GitHub Bot commented on TC-540:
-----------------------------------
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/859
[BACKPORT][TC-540] fix postinstall ldap.conf keys; prompt for base_url
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol 2.1-ldapfix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-trafficcontrol/pull/859.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #859
----
commit d8c0d89e2d6f86b37bff528bb0a789ca9c55ae1f
Author: Dan Kirkwood <[email protected]>
Date: 2017-08-18T18:44:33Z
required keys in ldap.conf; check host form; add base_url to defaults and
q's
(cherry picked from commit df979b164af2575159cd339036200a72215c63c2)
commit d1075d7d938925c5df164bb45ed91f99b12f0d13
Author: Dan Kirkwood <[email protected]>
Date: 2017-08-22T20:57:22Z
updates base_url
(cherry picked from commit 35abdcc64d9dbe70a1fd118bc51538d6fa10be66)
----
> TO postinstall writes incorrect keys to ldap.conf
> -------------------------------------------------
>
> Key: TC-540
> URL: https://issues.apache.org/jira/browse/TC-540
> Project: Traffic Control
> Issue Type: Bug
> Components: Traffic Ops
> Affects Versions: 2.1.0
> Reporter: Dan Kirkwood
> Assignee: Dan Kirkwood
> Priority: Blocker
> Fix For: 2.1.0
>
>
> I think this needs to be fixed for 2.1.
> postinstall writes incorrect keys to /opt/traffic_ops/app/conf/ldap.conf --
> they should be:
> {quote}
> "host"
> "admin_dn"
> "admin_pass"
> "search_base"
> {quote}
> but the keys that show up are:
> {quote}
> "password"
> "search_base"
> "hostname"
> "admin_dn"
> "setupLdap"
> {quote}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
