Re: Patch for issue 7378

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux