Skip Menu |

This queue is for tickets about the RT-BugTracker-Public CPAN distribution.

Report information
The Basics
Id: 78894
Status: resolved
Priority: 0/
Queue: RT-BugTracker-Public

People
Owner: Nobody in particular
Requestors: IDN [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



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
Thanks, applied. https://github.com/bestpractical/rt-bugtracker- public/commit/1f84c79625964a7fb870e3436fee31c2db855fdf