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

Always fetch forest info from root DCs when establishing two-way trust #46

Closed
wants to merge 3 commits into from

Conversation

martbab
Copy link
Contributor

@martbab martbab commented Sep 1, 2016

Prior To Windows Server 2012R2, the netr_DsRGetForestTrustInformation calls
performed against non-root forest domain DCs were automatically routed to the
root domain DCs to resolve trust topology information.

This is no longer the case, so the dcerpc.fetch_domains function must
explicitly contact root domain DCs even in the case when an external two-way
trust to non-root domain is requested.

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

Prior To Windows Server 2012R2, the `netr_DsRGetForestTrustInformation` calls
performed against non-root forest domain DCs were automatically routed to the
root domain DCs to resolve trust topology information.

This is no longer the case, so the `dcerpc.fetch_domains` function must
explicitly contact root domain DCs even in the case when an external two-way
trust to non-root domain is requested.

https://fedorahosted.org/freeipa/ticket/6057
@abbra
Copy link
Contributor

abbra commented Sep 1, 2016

The change is incomplete: we need also to handle oddjobd helper because it directly calls to dcerpc.fetch_domains() with explicitly set trusted domain name.

This allows for re-use of this method in cases where the caller can not or
wishes not to instantiate local Samba domain to retrieve information about
remote ones.

https://fedorahosted.org/freeipa/ticket/6057
flatname = trustconfig_show()['result']['ipantflatname'][0]

remote_domain = dcerpc.retrieve_remote_domain(
api.env.host, flatname, trusted_domain)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you are using api_instance, use it, not 'api'.

Prior To Windows Server 2012R2, the `netr_DsRGetForestTrustInformation` calls
performed against non-root forest domain DCs were automatically routed to
the root domain DCs to resolve trust topology information.

This is no longer the case, so the `com.redhat.idm.trust-fetch-domains` oddjob
helper used to establish one-way needs to explicitly contact root domain DCs
even in the case when an external trust to non-root domain is requested.

https://fedorahosted.org/freeipa/ticket/6057
@abbra
Copy link
Contributor

abbra commented Sep 2, 2016

ACK.

@abbra abbra added the ack Pull Request approved, can be merged label Sep 2, 2016
@martbab martbab added the pushed Pull Request has already been pushed label Sep 5, 2016
@martbab martbab closed this Sep 5, 2016
@martbab martbab deleted the t_6057 branch September 8, 2016 15:44
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
2 participants