Subject: | Documentation error in RT::BugTracker::Public |
Hi Guys,
Tracked this issue down with trs on IRC:
17:28 < trs> idn: try commenting on $WebNoAuthRegex
17:30 < idn> trs: Yup, that does it.
17:30 < trs> the doc is wrong.
17:30 < idn> Okay
17:32 < idn> trs: What's the right syntax or is that defunct?
17:33 < trs> WebNoAuthRegex is not defunct, but setting it to match
/Public/ is wrong
17:33 < trs> despite what the docs say
17:33 < trs> so a patch for the docs would be a great first step :)
17:33 < trs> you don't need to set it to anything, RT's default is fine.
17:33 < idn> k, I'll take that out and drop a patch in
Patch attached.
Thanks, Ian.
Subject: | 0083-Removing-mention-of-WebNoAuthRegex-the-defaults-are-.patch |
From b198ed7d9a8a47576c296ee897fcf7993105e14d Mon Sep 17 00:00:00 2001
From: root <root@iann-laptop.(none)>
Date: Fri, 10 Aug 2012 17:37:35 +0100
Subject: [PATCH 83/83] Removing mention of WebNoAuthRegex, the defaults are fine so we don't need to make changes to this.
---
README | 12 ------------
lib/RT/BugTracker/Public.pm | 12 ------------
2 files changed, 0 insertions(+), 24 deletions(-)
diff --git a/README b/README
index 66eb2fe..bcfe8bb 100644
--- a/README
+++ b/README
@@ -7,18 +7,6 @@ CONFIGURATION
of configuration and sane defaults. Add require in the main
RT_SiteConfig.pm or define options there.
- NoAuthRegexp
- As public shouldn't require authentication then you should add
- '/Public/' path to NoAuthRegexp option, something like:
-
- Set($WebNoAuthRegex, qr!^(?:/+NoAuth/|
- /+Public/|
- /+REST/\d+\.\d+/NoAuth/)!x );
-
- Note that if you have multiple extensions that want to change this
- option then you have to merge them manually into one perl regular
- expression.
-
WebPublicUser
Make sure to create the public user in your RT system and add the line
below to your RT_SiteConfig.pm.
diff --git a/lib/RT/BugTracker/Public.pm b/lib/RT/BugTracker/Public.pm
index 1ce7164..6110d65 100644
--- a/lib/RT/BugTracker/Public.pm
+++ b/lib/RT/BugTracker/Public.pm
@@ -64,18 +64,6 @@ You can find F<local/etc/BugTracker-Public/RT_SiteConfig.pm> with example of
configuration and sane defaults. Add require in the main F<RT_SiteConfig.pm> or
define options there.
-=head2 NoAuthRegexp
-
-As public shouldn't require authentication then you should add '/Public/' path
-to NoAuthRegexp option, something like:
-
- Set($WebNoAuthRegex, qr!^(?:/+NoAuth/|
- /+Public/|
- /+REST/\d+\.\d+/NoAuth/)!x );
-
-Note that if you have multiple extensions that want to change this option then
-you have to merge them manually into one perl regular expression.
-
=head2 WebPublicUser
Make sure to create the public user in your RT system and add the line below
--
1.7.4.1