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

Changing cert-find to go through the proxy instead of using the port 8080 #874

Closed
wants to merge 1 commit into from

Conversation

felipevolpone
Copy link
Member

@felipevolpone felipevolpone commented Jun 14, 2017

The cert-find command now uses the proxy to reach Dogtag, instead of using the port 8080. In order to accomplish that, it's necessary to change the proxy configuration adding the service URL.

Ticket: https://pagure.io/freeipa/issue/6966

@rcritten
Copy link
Contributor

When modifying the Apache config files you need to bump VERSION so the change is applied to existing installs.

…8080

The cert-find command now uses the proxy to reach Dogtag, instead of using
the port 8080. In order to accomplish that, it's necessary to change the
proxy configuration including the URL called.

https://pagure.io/freeipa/issue/6966
@felipevolpone
Copy link
Member Author

@rcritten I forget that. Thanks for reminding me. Done.

@frasertweedale frasertweedale self-assigned this Jun 16, 2017
@frasertweedale frasertweedale self-requested a review June 16, 2017 05:06
@frasertweedale frasertweedale removed their assignment Jun 16, 2017
@frasertweedale frasertweedale added the ack Pull Request approved, can be merged label Jun 16, 2017
@frasertweedale
Copy link
Contributor

All good. Nice work @felipevolpone 👍

@nicki-krizek
Copy link
Contributor

master:

  • 3653203 Changing cert-find to go through the proxy instead of using the port 8080

ipa-4-5:

  • 960b9a3 Changing cert-find to go through the proxy instead of using the port 8080

@nicki-krizek nicki-krizek added the pushed Pull Request has already been pushed label Jun 16, 2017
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
4 participants