On 06/25/2013 12:22 PM, Martin Kosek wrote:
On 06/24/2013 07:00 PM, Tomas Babej wrote:
On 06/24/2013 06:13 PM, Petr Vobornik wrote:
Those resources are needed by page which has to use http(browser config)
prior to acceptance of CA cert.

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


_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

I think you (technically) need to update the version in the first commented 
line.

Thanks for pointing this out Tomas. I see you learned a lesson last time :-)


(Probably would not be an issue for anybody, since we haven't done a release
since it was last changed)

Yeah, this is not critical. But it may still make sense to bump the number, if
just for a sake of consistency if someone would do some "git blame"
investigation based on this line.

Martin


Version bumped, updated patch attached.
--
Petr Vobornik
From bc60575a75ceed8dbc2f3791fad095ef62e00a42 Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvobo...@redhat.com>
Date: Mon, 24 Jun 2013 17:44:15 +0200
Subject: [PATCH] Do not redirect to https in /ipa/ui on non-HTML files

Those resources are needed by page which has to use http(browser config) prior to acceptance of CA cert.

https://fedorahosted.org/freeipa/ticket/3748
---
 install/conf/ipa-rewrite.conf | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/install/conf/ipa-rewrite.conf b/install/conf/ipa-rewrite.conf
index 552682b05bb8a911c8b735a3e7252cba24288f9b..37661b8200ed7ab4e316093de90b811b8e6fa3cf 100644
--- a/install/conf/ipa-rewrite.conf
+++ b/install/conf/ipa-rewrite.conf
@@ -1,4 +1,4 @@
-# VERSION 5 - DO NOT REMOVE THIS LINE
+# VERSION 6 - DO NOT REMOVE THIS LINE
 
 RewriteEngine on
 
@@ -15,6 +15,7 @@ RewriteRule ^/ipa/(.*)      http://$FQDN/ipa/$$1 [L,R=301]
 # configuration.
 RewriteCond %{SERVER_PORT}  !^443$$
 RewriteCond %{REQUEST_URI}  !^/ipa/(errors|config|crl)
+RewriteCond %{REQUEST_URI}  !^/ipa/[^\?]+(\.js|\.css|\.png|\.gif|\.ico|\.woff|\.svg|\.ttf|\.eot)$$
 RewriteRule ^/ipa/(.*)      https://$FQDN/ipa/$$1 [L,R=301,NC]
 
 # Rewrite for plugin index, make it like it's a static file
-- 
1.8.1.4

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to