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

Test: basic kerberos over http functionality #200

Closed
wants to merge 2 commits into from
Closed

Test: basic kerberos over http functionality #200

wants to merge 2 commits into from

Conversation

ofayans
Copy link
Contributor

@ofayans ofayans commented Nov 1, 2016

@apophys apophys self-assigned this Nov 10, 2016
Copy link
Contributor

@apophys apophys left a comment

Choose a reason for hiding this comment

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

This commit should precede the test itself.

@apophys
Copy link
Contributor

apophys commented Nov 25, 2016

Please rebase the commits in the right order.

What will happen when the hosts in the topology have both IPv4 and IPv6 stacks when you disable ports for only one? Is the IPA server serving on both network stacks?

Oleg Fayans added 2 commits November 26, 2016 10:59
In some cases we need to check the result of kinit and print out the error
message. Therefore we need it to return the result.
@ofayans
Copy link
Contributor Author

ofayans commented Nov 26, 2016

Commits were swapped. The traffic blocking is now performed on a client for both ipv4 and ipv6

@apophys
Copy link
Contributor

apophys commented Nov 29, 2016

Thank you for rebasing the commits. The test looks good.

@apophys apophys added the ack Pull Request approved, can be merged label Nov 29, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Nov 30, 2016
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