On 12/7/18 11:46 AM, Mikolaj Izdebski wrote:
> It's almost good, but the patch is not entirely correct.
> 


:) I think I'm going to borrow "It's almost good", as the sub-header for
my website.

I looked at the original ticket and inferred too much possibly. Or
perhaps I'm just not clear on how this redirect works. I see now this a
for an apache mod_proxy config? Maybe I was thinking this was for a
mod_rewrite?

Either way; new patch attached without the '$1'. Thanks for reviewing!

-Zach #aikdiouke


diff --git a/playbooks/include/proxies-redirects.yml b/playbooks/include/proxies-redirects.yml
index 3a108eda3..0bfcd543f 100644
--- a/playbooks/include/proxies-redirects.yml
+++ b/playbooks/include/proxies-redirects.yml
@@ -62,7 +62,7 @@
   - role: httpd/redirect
     shortname: jenkins
     website: jenkins.fedorainfracloud.org
-    target: https://jenkins-fedora-infra.apps.ci.centos.org
+    target: https://jenkins-fedora-infra.apps.ci.centos.org/
 
   - role: httpd/redirectmatch
     shortname: fpaste

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
infrastructure mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]

Reply via email to